549766 - IPDL: Suppress MSVC warning 4530 in generated C++

Status: RESOLVED INCOMPLETE

(Core :: IPC, defect)

Description

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

We get a bunch of warnings like

C:\Program Files\Microsoft Visual Studio 8\VC\INCLUDE\ostream(520) : warning C4530: C++ exception handler used, but unwind semantics are not enabled. Specify /EHsc
Elem,_Traits>::put(_Elem)'
        with
        [
            _Elem=wchar_t,
            _Traits=std::char_traits<wchar_t>
        ]
        C:\Program Files\Microsoft Visual Studio 8\VC\INCLUDE\ostream(603) : see reference to class template instantiation 'std::basic_ostream<_Elem,_Traits>' being compiled
        with
        [
            _Elem=wchar_t,
            _Traits=std::char_traits<wchar_t>
        ]

because mozilla code is built with -fno-exceptions, but includes chromium code that includes STL code that uses try/catch, apparently.

Comment 1

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

See http://pastebin.org/100068 for more.

Comment 2

[:Cykesiopka]

Fixed by config/msvc-stl-wrapper.template.h from Bug 551254...?
(Not very sure though, sorry)

Comment 3

[:Cykesiopka]

cjones: You fixed Bug 551254, so it seems like you would be in the best position to answer the question in Comment 2.

Thanks ahead of time...

Comment 4

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

No, I don't think that patch would have made this warning go away.  But, are we sure that this warning is still generated?  I'm not sure that any gecko code still uses std::ostream.

Comment 5

[:Cykesiopka]

(In reply to Chris Jones [:cjones] mostly inactive; ni?/f?/r? if you need me from comment #4)
> No, I don't think that patch would have made this warning go away.

Ah. Thanks!