On 05.03.2014 14:20, jan i wrote:
On 5 March 2014 14:18, Rob Weir <robw...@apache.org> wrote:

On Wed, Mar 5, 2014 at 7:32 AM, jan i <j...@apache.org> wrote:
On 5 March 2014 12:46, Andre Fischer <awf....@gmail.com> wrote:

On 05.03.2014 12:38, John O'Sullivan wrote:

Use MSVC's WD flag to disable the warning. You'll probably need to hack
wntmsci11.mk...

I wouldn't like to turn off this warning globally.  I only don't want to
see it for system headers that I can't change anyway.


Use #pragma  to disable/enable the warnings. Put the #pragma around the
include statement.

Or use a push/pop set of pragmas, which handles nested inclusion
scenarios safer:

http://msdn.microsoft.com/en-us/library/2c8f766e%28v=vs.90%29.aspx


Yes, I am already doing that. But with my last commit [1] I missed another inclusion of a system header that triggers the warning.

If you use push/pop please check for platform, since this is not supported
by all C++ compilers, whereas #pragma alone is.

That is not a problem here. Warnings are compiler dependent. And when you know for which compiler you want to turn off a warning, you also know if it can handle push/pop.

-Andre


rgds
jan I.


-Rob


[1] http://svn.apache.org/viewvc?view=revision&revision=1574449

rgds
jan I.

-Andre


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@openoffice.apache.org
For additional commands, e-mail: dev-h...@openoffice.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@openoffice.apache.org
For additional commands, e-mail: dev-h...@openoffice.apache.org




---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@openoffice.apache.org
For additional commands, e-mail: dev-h...@openoffice.apache.org

Reply via email to