Fix "Dead nested assignment" static analyzer warning in monthFromDayInYear()
authorddkilzer@apple.com <ddkilzer@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 30 Mar 2020 16:58:27 +0000 (16:58 +0000)
committerddkilzer@apple.com <ddkilzer@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 30 Mar 2020 16:58:27 +0000 (16:58 +0000)
<https://webkit.org/b/209718>

Reviewed by Alex Christensen.

* wtf/DateMath.h:
(WTF::monthFromDayInYear): Change the last check so it doesn't
assign a value to `step`, thus avoiding the warning.

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@259202 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WTF/ChangeLog
Source/WTF/wtf/DateMath.h

index ebd293e..0e44007 100644 (file)
@@ -1,3 +1,14 @@
+2020-03-30  David Kilzer  <ddkilzer@apple.com>
+
+        Fix "Dead nested assignment" static analyzer warning in monthFromDayInYear()
+        <https://webkit.org/b/209718>
+
+        Reviewed by Alex Christensen.
+
+        * wtf/DateMath.h:
+        (WTF::monthFromDayInYear): Change the last check so it doesn't
+        assign a value to `step`, thus avoiding the warning.
+
 2020-03-28  Simon Fraser  <simon.fraser@apple.com>
 
         Add a ScrollLatching log channel and improve some logging functionality
index 602f89a..cfd5d75 100644 (file)
@@ -1,6 +1,6 @@
 /*
  * Copyright (C) 1999-2000 Harri Porten (porten@kde.org)
- * Copyright (C) 2006, 2007 Apple Inc. All rights reserved.
+ * Copyright (C) 2006-2020 Apple Inc. All rights reserved.
  * Copyright (C) 2009 Google Inc. All rights reserved.
  * Copyright (C) 2010 Research In Motion Limited. All rights reserved.
  *
@@ -345,7 +345,7 @@ inline int monthFromDayInYear(int dayInYear, bool leapYear)
         return 8;
     if (d < (step += 31))
         return 9;
-    if (d < (step += 30))
+    if (d < step + 30)
         return 10;
     return 11;
 }