[Python-checkins] r75881 - in python/branches/py3k: Modules/_io/_iomodule.h Modules/_io/bufferedio.c
mark.dickinson
python-checkins at python.org
Tue Oct 27 22:49:48 CET 2009
Author: mark.dickinson
Date: Tue Oct 27 22:49:48 2009
New Revision: 75881
Log:
Merged revisions 75879 via svnmerge from
svn+ssh://pythondev@svn.python.org/python/trunk
........
r75879 | mark.dickinson | 2009-10-27 21:48:20 +0000 (Tue, 27 Oct 2009) | 3 lines
Silence gcc warnings when trying to print an off_t using "lld", on platforms
where off_t has type long (e.g., 64-bit Linux).
........
Modified:
python/branches/py3k/ (props changed)
python/branches/py3k/Modules/_io/_iomodule.h
python/branches/py3k/Modules/_io/bufferedio.c
Modified: python/branches/py3k/Modules/_io/_iomodule.h
==============================================================================
--- python/branches/py3k/Modules/_io/_iomodule.h (original)
+++ python/branches/py3k/Modules/_io/_iomodule.h Tue Oct 27 22:49:48 2009
@@ -70,6 +70,14 @@
* Offset type for positioning.
*/
+/* Printing a variable of type off_t correctly and without producing
+ compiler warnings is surprisingly painful. We identify an integer
+ type whose size matches off_t and then: (1) cast the off_t to that
+ integer type and (2) use the appropriate conversion specification
+ for printf. The cast is necessary: gcc complains about formatting
+ a long with "%lld" even when both long and long long have the same
+ precision. */
+
#if defined(MS_WIN64) || defined(MS_WINDOWS)
/* Windows uses long long for offsets */
@@ -79,7 +87,7 @@
# define PY_OFF_T_MAX PY_LLONG_MAX
# define PY_OFF_T_MIN PY_LLONG_MIN
# define PY_PRIdOFF "lld" /* format to use in printf with type off_t */
-
+# define PY_OFF_T_COMPAT long long /* standard type compatible with off_t */
#else
/* Other platforms use off_t */
@@ -90,18 +98,21 @@
# define PY_OFF_T_MAX PY_LLONG_MAX
# define PY_OFF_T_MIN PY_LLONG_MIN
# define PY_PRIdOFF "lld"
+# define PY_OFF_T_COMPAT long long
#elif (SIZEOF_OFF_T == SIZEOF_LONG)
# define PyLong_AsOff_t PyLong_AsLong
# define PyLong_FromOff_t PyLong_FromLong
# define PY_OFF_T_MAX LONG_MAX
# define PY_OFF_T_MIN LONG_MIN
# define PY_PRIdOFF "ld"
+# define PY_OFF_T_COMPAT long
#elif (SIZEOF_OFF_T == SIZEOF_SIZE_T)
# define PyLong_AsOff_t PyLong_AsSsize_t
# define PyLong_FromOff_t PyLong_FromSsize_t
# define PY_OFF_T_MAX PY_SSIZE_T_MAX
# define PY_OFF_T_MIN PY_SSIZE_T_MIN
# define PY_PRIdOFF "zd"
+# define PY_OFF_T_COMPAT Py_ssize_t
#else
# error off_t does not match either size_t, long, or long long!
#endif
Modified: python/branches/py3k/Modules/_io/bufferedio.c
==============================================================================
--- python/branches/py3k/Modules/_io/bufferedio.c (original)
+++ python/branches/py3k/Modules/_io/bufferedio.c Tue Oct 27 22:49:48 2009
@@ -581,7 +581,7 @@
if (!PyErr_Occurred())
PyErr_Format(PyExc_IOError,
"Raw stream returned invalid position %" PY_PRIdOFF,
- n);
+ (PY_OFF_T_COMPAT)n);
return -1;
}
self->abs_pos = n;
@@ -614,7 +614,7 @@
if (!PyErr_Occurred())
PyErr_Format(PyExc_IOError,
"Raw stream returned invalid position %" PY_PRIdOFF,
- n);
+ (PY_OFF_T_COMPAT)n);
return -1;
}
self->abs_pos = n;
More information about the Python-checkins
mailing list