clustc / google-breakpad

Automatically exported from code.google.com/p/google-breakpad
0 stars 0 forks source link

can not build with visual studio 2013 preview #540

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
gmock-all.cc
c:\program files (x86)\microsoft visual studio 12.0\vc\include\xtree(604): 
error C2220

gtest-all.cc
c:\program files (x86)\microsoft visual studio 12.0\vc\include\xtree(1826): 
error C2220

minidump.cc
..\..\..\processor\minidump.cc(2043): error C2220

Original issue reported on code.google.com by sunbaoji...@gmail.com on 30 Jul 2013 at 1:59

GoogleCodeExporter commented 9 years ago
Same issue.

Original comment by Ghla...@gmail.com on 27 Nov 2013 at 10:14

GoogleCodeExporter commented 9 years ago
Any movement on this? Would be nice to kill this off.

Original comment by Ghla...@gmail.com on 24 Dec 2013 at 8:02

GoogleCodeExporter commented 9 years ago
Sure. I'll take a look after I upgrade to VS 2013.

Original comment by thestig@chromium.org on 3 Jan 2014 at 11:45

GoogleCodeExporter commented 9 years ago
This is because we disable exceptions and this sets _HAS_EXCEPTIONS=0 
triggering this issue:

http://connect.microsoft.com/VisualStudio/feedback/details/809962/has-exceptions
-0-triggers-unreachable-code-warning-in-xtree

because of the unreachable code

the current 'solution' here is to disable warning 4702 as has already been done 
in Chromium.  I'll upload a CL.  Then we can roll deps on gyp as well.

Original comment by wfh@chromium.org on 25 Mar 2014 at 3:21

GoogleCodeExporter commented 9 years ago
Build now works fine with VS2013 - set GYP_MSVS_VERSION=2013 before running gyp.

Original comment by wfh@chromium.org on 23 Apr 2014 at 5:34

GoogleCodeExporter commented 9 years ago
The CL fixing this was https://breakpad.appspot.com/1284002/

Original comment by wfh@chromium.org on 23 Apr 2014 at 5:35