changeset:   95367:7907746baa0d
branch:      3.4
parent:      95364:1ad7c0253abe
user:        Serhiy Storchaka <storchaka@gmail.com>
date:        Thu Apr 02 09:47:27 2015 +0300
files:       Lib/test/test_exceptions.py Misc/NEWS Python/errors.c
description:
Issue #22977: Fixed formatting Windows error messages on Wine.
Patch by Martin Panter.


diff -r 1ad7c0253abe -r 7907746baa0d Lib/test/test_exceptions.py
--- a/Lib/test/test_exceptions.py	Thu Apr 02 00:04:06 2015 -0400
+++ b/Lib/test/test_exceptions.py	Thu Apr 02 09:47:27 2015 +0300
@@ -6,10 +6,11 @@
 import pickle
 import weakref
 import errno
+import ctypes
 
 from test.support import (TESTFN, captured_output, check_impl_detail,
                           check_warnings, cpython_only, gc_collect, run_unittest,
-                          no_tracing, unlink)
+                          no_tracing, unlink, get_attribute)
 
 class NaiveException(Exception):
     def __init__(self, x):
@@ -245,6 +246,13 @@
             self.assertEqual(w.strerror, 'foo')
             self.assertEqual(w.filename, None)
 
+    def test_windows_message(self):
+        """Should fill in unknown error code in Windows error message"""
+        windll = get_attribute(ctypes, "windll")
+        code = int.from_bytes(b"\xE0msc", "big")
+        with self.assertRaisesRegex(OSError, hex(code)):
+            windll.kernel32.RaiseException(code, 0, 0, None)
+
     def testAttributes(self):
         # test that exception attributes are happy
 
diff -r 1ad7c0253abe -r 7907746baa0d Misc/NEWS
--- a/Misc/NEWS	Thu Apr 02 00:04:06 2015 -0400
+++ b/Misc/NEWS	Thu Apr 02 09:47:27 2015 +0300
@@ -10,6 +10,9 @@
 Core and Builtins
 -----------------
 
+- Issue #22977: Fixed formatting Windows error messages on Wine.
+  Patch by Martin Panter.
+
 - Issue #23803: Fixed str.partition() and str.rpartition() when a separator
   is wider then partitioned string.
 
diff -r 1ad7c0253abe -r 7907746baa0d Python/errors.c
--- a/Python/errors.c	Thu Apr 02 00:04:06 2015 -0400
+++ b/Python/errors.c	Thu Apr 02 09:47:27 2015 +0300
@@ -491,7 +491,7 @@
                 /* Only ever seen this in out-of-mem
                    situations */
                 s_buf = NULL;
-                message = PyUnicode_FromFormat("Windows Error 0x%X", i);
+                message = PyUnicode_FromFormat("Windows Error 0x%x", i);
             } else {
                 /* remove trailing cr/lf and dots */
                 while (len > 0 && (s_buf[len-1] <= L' ' || s_buf[len-1] == L'.'))
@@ -600,7 +600,7 @@
         NULL);          /* no args */
     if (len==0) {
         /* Only seen this in out of mem situations */
-        message = PyUnicode_FromFormat("Windows Error 0x%X", err);
+        message = PyUnicode_FromFormat("Windows Error 0x%x", err);
         s_buf = NULL;
     } else {
         /* remove trailing cr/lf and dots */