zhangqd / chromiumembedded

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

Build failure on Windows x64 after revision 1574 #1184

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
After revision 1574, I am seeing the following error on WIn 64 trunk:

FAILED: ninja -t msvc -e environment.x64 -- "C:\Program Files (x86)\Microsoft 
Visual Studio 10.0\VC\bin\amd64\cl.exe" /nologo /showIncludes /FC 
@obj\cef\libcef_dll\wrapper\libcef_dll_wrapper.cef_message_router.obj.rsp /c 
..\..\cef\libcef_dll\wrapper\cef_message_router.cc 
/Foobj\cef\libcef_dll\wrapper\libcef_dll_wrapper.cef_message_router.obj 
/Fdobj\cef\libcef_dll_wrapper.cc.pdb 
c:\cef\workspace\cef3-windows-trunk-64\download\chromium\src\cef\libcef_dll\wrap
per\cef_message_router.cc(225) : error C2220: warning treated as error - no 
'object' file generated
c:\cef\workspace\cef3-windows-trunk-64\download\chromium\src\cef\libcef_dll\wrap
per\cef_message_router.cc(225) : warning C4267: 'return' : conversion from 
'size_t' to 'int', possible loss of data
c:\cef\workspace\cef3-windows-trunk-64\download\chromium\src\cef\libcef_dll\wrap
per\cef_message_router.cc(227) : warning C4267: 'return' : conversion from 
'size_t' to 'int', possible loss of data
c:\cef\workspace\cef3-windows-trunk-64\download\chromium\src\cef\libcef_dll\wrap
per\cef_message_router.cc(779) : warning C4267: 'return' : conversion from 
'size_t' to 'int', possible loss of data
c:\cef\workspace\cef3-windows-trunk-64\download\chromium\src\cef\libcef_dll\wrap
per\cef_message_router.cc(781) : warning C4267: 'return' : conversion from 
'size_t' to 'int', possible loss of data
ninja: build stopped: subcommand failed.

Original issue reported on code.google.com by ol...@adobe.com on 28 Jan 2014 at 4:10

GoogleCodeExporter commented 9 years ago
After fixing this error, I get a few similar errors in 
message_router_unittest.cc.
I fixed them all by static_cast to int.

Original comment by ol...@adobe.com on 28 Jan 2014 at 4:29

Attachments:

GoogleCodeExporter commented 9 years ago
Thanks, fixed in trunk revision 1576.

Original comment by magreenb...@gmail.com on 28 Jan 2014 at 4:59