kryskool / cefpython

Automatically exported from code.google.com/p/cefpython
0 stars 1 forks source link

Issues with size_t => int conversions when using cef_values.h API on Win x64 platform #165

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
Issue reported here:
https://code.google.com/p/chromiumembedded/issues/detail?id=1491

    [compile.bat] Building libcefpythonapp vcproj
    Build started: Project: libcefpythonapp_py27_64bit, Configuration: Release|x64
    Compiling...
    cefpython_app.cpp
    v8utils.cpp
    1>.\v8utils.cpp(40) : warning C4267: 'argument' : conversion from 'size_t' to 'int', possible loss of data
    1>.\v8utils.cpp(42) : warning C4267: 'argument' : conversion from 'size_t' to 'int', possible loss of data
    1>.\v8utils.cpp(44) : warning C4267: 'argument' : conversion from 'size_t' to 'int', possible loss of data
    1>.\v8utils.cpp(49) : warning C4267: 'argument' : conversion from 'size_t' to 'int', possible loss of data
    1>.\v8utils.cpp(51) : warning C4267: 'argument' : conversion from 'size_t' to 'int', possible loss of data
    1>.\v8utils.cpp(60) : warning C4267: 'argument' : conversion from 'size_t' to 'int', possible loss of data
    1>.\v8utils.cpp(62) : warning C4267: 'argument' : conversion from 'size_t' to 'int', possible loss of data
    1>.\v8utils.cpp(71) : warning C4267: 'argument' : conversion from 'size_t' to 'int', possible loss of data
    1>.\v8utils.cpp(81) : warning C4267: 'argument' : conversion from 'size_t' to 'int', possible loss of data
    1>.\v8utils.cpp(83) : warning C4267: 'argument' : conversion from 'size_t' to 'int', possible loss of data
    1>.\v8utils.cpp(91) : warning C4267: 'argument' : conversion from 'size_t' to 'int', possible loss of data
    1>.\v8utils.cpp(93) : warning C4267: 'argument' : conversion from 'size_t' to 'int', possible loss of data
    1>.\v8utils.cpp(230) : warning C4267: 'initializing' : conversion from 'size_t' to 'int', possible loss of data

Original issue reported on code.google.com by czarek.t...@gmail.com on 16 Jan 2015 at 1:20

GoogleCodeExporter commented 9 years ago
There are also warnings when cythoning:

    cefpython.cpp(36438) : warning C4244: 'argument' : conversion from 'Py_ssize_t' to 'int',
        possible loss of data
    cefpython.cpp(36462) : warning C4244: 'argument' : conversion from 'Py_ssize_t' to 'int', 
        possible loss of data

Fixes to both v8utils.cpp and process_message_utils.pyx will be to cast to an 
int the result from GetSize().

Original comment by czarek.t...@gmail.com on 16 Jan 2015 at 3:55

GoogleCodeExporter commented 9 years ago
Upstream CEF does not support VS2008 anymore, so this won't be fixed. 
Apparently it compiles fine for 64bit with the latest VS2013.

Original comment by czarek.t...@gmail.com on 23 Jan 2015 at 2:22