AuroraMiddleware/protobuf
1
0
Fork 0
Code Issues 1 Pull Requests Projects Releases Wiki Activity

Fix for https://github.com/google/protobuf/issues/758

Browse Source 
Change-Id: I590b34b96c84a3ba6e094a0bd86f153147ade3d3
This commit is contained in:
Josh Haberman 2015-08-28 16:34:48 -07:00
parent 26f0c678f6
commit fc80fad9d5
1 changed files with 17 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

31
python/google/protobuf/pyext/message.cc
View File

@ -1776,22 +1776,25 @@ class PythonFieldValuePrinter : public TextFormat::FieldValuePrinter {
// Python floats to ensure consistency.
string PrintFloat(float value) const { return PrintDouble(value); }
string PrintDouble(double value) const {
// Same as float.__str__()
char* buf = PyOS_double_to_string(
value,
#if PY_MAJOR_VERSION < 3
'g', PyFloat_STR_PRECISION, // Output is rounded to 12 digits.
#else
'r', 0,
#endif
Py_DTSF_ADD_DOT_0, // Trailing .0 is always printed.
NULL);
if (!buf) {
// This implementation is not highly optimized (it allocates two temporary
// Python objects) but it is simple and portable. If this is shown to be a
// performance bottleneck, we can optimize it, but the results will likely
// be more complicated to accommodate the differing behavior of double
// formatting between Python 2 and Python 3.
//
// (Though a valid question is: do we really want to make out output
// dependent on the Python version?)
ScopedPyObjectPtr py_value(PyFloat_FromDouble(value));
if (!py_value.get()) {
return string();
}
string result(buf);
PyMem_Free(buf);
return result;
ScopedPyObjectPtr py_str(PyObject_Str(py_value.get()));
if (!py_str.get()) {
return string();
}
return string(PyString_AsString(py_str.get()));
}
};