[Test] Warning Suppression?

Hi, I keep seeing c:\boost\boost/test/impl/execution_monitor.ipp(214) : warning C4535: calling _set_se_translator() requires /EHa Is the warning correct? If so I think we need to add the appropriate stuff to the build system to generate /EHa in this case. If not I think we need to add the appropriate warning suppression. -- Dave Abrahams Boost Consulting www.boost-consulting.com

David Abrahams wrote:
Hi,
I keep seeing
c:\boost\boost/test/impl/execution_monitor.ipp(214) : warning C4535: calling _set_se_translator() requires /EHa
Is the warning correct? If so I think we need to add the appropriate stuff to the build system to generate /EHa in this case. If not I think we need to add the appropriate warning suppression.
I think so. AFAIK, if BOOST_MS_STRCTURED_EXCEPTION_HANDLING is defined, exection_monitor.cpp would use _set_se_translator to catch win32 exceptions and rethrow a ms_se_exception instead, so that execution_monitor::execute can detect such situation and output some error message. _set_se_translator is only supported for /EHa Hope this helps. -- Yi Zhang

On 5/20/05, ATField <field.zhang@163.com> wrote:
I think so. AFAIK, if BOOST_MS_STRCTURED_EXCEPTION_HANDLING is defined, exection_monitor.cpp would use _set_se_translator to catch win32
This seems to be defined by default.
exceptions and rethrow a ms_se_exception instead, so that execution_monitor::execute can detect such situation and output some error message. _set_se_translator is only supported for /EHa
Hope this helps.
I don't think it does. This is a problem that needs to be fixed in Boost.Test, not code that uses Boost.Test. IMHO of course. -- Caleb Epstein caleb dot epstein at gmail dot com

"Caleb Epstein" <caleb.epstein@gmail.com> wrote in message news:989aceac0505200906439dd46d@mail.gmail.com...
On 5/20/05, ATField <field.zhang@163.com> wrote:
I think so. AFAIK, if BOOST_MS_STRCTURED_EXCEPTION_HANDLING is defined, exection_monitor.cpp would use _set_se_translator to catch win32
This seems to be defined by default.
It does. For the configurations that supports it. Note though that behavior of _set_se_translator is runtime configurable. If you set catch_system_errors=no (using unit test framework) translator invokes debugger/causing immidiate crash.
exceptions and rethrow a ms_se_exception instead, so that execution_monitor::execute can detect such situation and output some error message. _set_se_translator is only supported for /EHa
Hope this helps.
I don't think it does. This is a problem that needs to be fixed in Boost.Test, not code that uses Boost.Test. IMHO of course.
I don't see it as a problem, rather as a feature. And /EHa flag needt obe added to build scripts to support it (unless someone always using catch_system_errors=no always). Gennadiy

"Gennadiy Rozental" <gennadiy.rozental@thomson.com> writes:
I don't think it does. This is a problem that needs to be fixed in Boost.Test, not code that uses Boost.Test. IMHO of course.
I don't see it as a problem, rather as a feature. And /EHa flag needt obe added to build scripts to support it
That would be a way of fixing the problem in Boost.Test. Are you going to do that? -- Dave Abrahams Boost Consulting www.boost-consulting.com
participants (4)
-
ATField
-
Caleb Epstein
-
David Abrahams
-
Gennadiy Rozental