Title: [233406] trunk/Source/WebCore
Revision
233406
Author
[email protected]
Date
2018-06-30 19:12:26 -0700 (Sat, 30 Jun 2018)

Log Message

Follow-up: Fix clang static analyzer warnings: Garbage return value
<https://webkit.org/b/187224>

Address review feedback from Eric Carlson.

* platform/mediastream/MediaConstraints.h:
(WebCore::NumericConstraint::valueForCapabilityRange const):
- Use brace initialization for more local variables.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (233405 => 233406)


--- trunk/Source/WebCore/ChangeLog	2018-07-01 01:49:50 UTC (rev 233405)
+++ trunk/Source/WebCore/ChangeLog	2018-07-01 02:12:26 UTC (rev 233406)
@@ -1,3 +1,14 @@
+2018-06-30  David Kilzer  <[email protected]>
+
+        Follow-up: Fix clang static analyzer warnings: Garbage return value
+        <https://webkit.org/b/187224>
+
+        Address review feedback from Eric Carlson.
+
+        * platform/mediastream/MediaConstraints.h:
+        (WebCore::NumericConstraint::valueForCapabilityRange const):
+        - Use brace initialization for more local variables.
+
 2018-06-30  Michael Catanzaro  <[email protected]>
 
         Asan false positive: stack use after scope under WebCore::ApplyPropertyBorderImageModifier in WebCore::Length::Length(WebCore::Length&&)

Modified: trunk/Source/WebCore/platform/mediastream/MediaConstraints.h (233405 => 233406)


--- trunk/Source/WebCore/platform/mediastream/MediaConstraints.h	2018-07-01 01:49:50 UTC (rev 233405)
+++ trunk/Source/WebCore/platform/mediastream/MediaConstraints.h	2018-07-01 02:12:26 UTC (rev 233406)
@@ -241,8 +241,8 @@
     ValueType valueForCapabilityRange(ValueType current, ValueType capabilityMin, ValueType capabilityMax) const
     {
         ValueType value { 0 };
-        ValueType min = capabilityMin;
-        ValueType max = capabilityMax;
+        ValueType min { capabilityMin };
+        ValueType max { capabilityMax };
 
         if (m_exact) {
             ASSERT(validForRange(capabilityMin, capabilityMax));
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to