]> granicus.if.org Git - python/commitdiff
Don't abuse volatile keyword in pytime.c
authorVictor Stinner <victor.stinner@gmail.com>
Thu, 3 Sep 2015 14:33:16 +0000 (16:33 +0200)
committerVictor Stinner <victor.stinner@gmail.com>
Thu, 3 Sep 2015 14:33:16 +0000 (16:33 +0200)
Only use it on the most important number. This change fixes also a compiler
warning on modf().

Python/pytime.c

index cea3cf5d433a915cbdcf066f6f605e453df61183..d226389a7cda1e7f78d6a647a632089792e6beb9 100644 (file)
@@ -135,8 +135,9 @@ int
 _PyTime_ObjectToTime_t(PyObject *obj, time_t *sec, _PyTime_round_t round)
 {
     if (PyFloat_Check(obj)) {
+        double intpart, err;
         /* volatile avoids optimization changing how numbers are rounded */
-        volatile double d, intpart, err;
+        volatile double d;
 
         d = PyFloat_AsDouble(obj);
         if (round == _PyTime_ROUND_HALF_UP)
@@ -257,8 +258,9 @@ static int
 _PyTime_FromFloatObject(_PyTime_t *t, double value, _PyTime_round_t round,
                         long to_nanoseconds)
 {
+    double err;
     /* volatile avoids optimization changing how numbers are rounded */
-    volatile double d, err;
+    volatile double d;
 
     /* convert to a number of nanoseconds */
     d = value;