Remove debug output, fix assert (hopefully) and exercise signedness issues a bit...

diff --git a/Lib/test/test_import.py b/Lib/test/test_import.py
index 412e6b81039a75f4098b2672f7bd7470bfae17d8..e426290df3a76e8e51ce67a2319469dd574a69e2 100644 (file)
--- a/Lib/test/test_import.py
+++ b/Lib/test/test_import.py
@@ -288,7 +288,7 @@ class ImportTests(unittest.TestCase):
             with open(source, 'w') as f:
                 pass
             try:
-                os.utime(source, (2 ** 33, 2 ** 33))
+                os.utime(source, (2 ** 33 - 5, 2 ** 33 - 5))
             except OverflowError:
                 self.skipTest("cannot set modification time to large integer")
             except OSError as e:

diff --git a/Python/import.c b/Python/import.c
index f9f3cdbbfbd0ac1cbf3aba539838a639ad155690..2cac9b530bb27259cc4a524afa017b10fda85873 100644 (file)
--- a/Python/import.c
+++ b/Python/import.c
@@ -905,14 +905,9 @@ write_compiled_module(PyCodeObject *co, char *cpathname, struct stat *srcstat)
         (void) unlink(cpathname);
         return;
     }
-    /* Now write the true mtime */
+    /* Now write the true mtime (as a 32-bit field) */
     fseek(fp, 4L, 0);
-    if (mtime >= LONG_MAX) {
-        fprintf(stderr, "** sizes=(%ld, %ld), mtime=%I64d >= %ld\n", sizeof(time_t), sizeof(srcstat->st_mtime), mtime, LONG_MAX);
-        assert(0);
-        /* can't get here */
-    }
-    assert(mtime < LONG_MAX);
+    assert(mtime <= 0xFFFFFFFF);
     PyMarshal_WriteLongToFile((long)mtime, fp, Py_MARSHAL_VERSION);
     fflush(fp);
     fclose(fp);