diff --git a/ChangeLog b/ChangeLog index b1e55a693..96b15281a 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,13 @@ +2013-06-12 Werner Lemberg . + + Another try on pragmas. + + * include/freetype/internal/ftdebug.h: Move pragmas to... + * include/freetype/internal/internal.h: ... this file since it gets + included by all source files. + * include/freetype/internal/ftserv.h: Remove pragma which has no + effect. + 2013-06-12 Werner Lemberg . * include/freetype/internal/ftdebug.h: Disable MSVC warning C4127. diff --git a/include/freetype/internal/ftdebug.h b/include/freetype/internal/ftdebug.h index ede3872b0..58a3916d7 100644 --- a/include/freetype/internal/ftdebug.h +++ b/include/freetype/internal/ftdebug.h @@ -247,19 +247,6 @@ FT_BEGIN_HEADER FT_BASE( void ) ft_debug_init( void ); - -#if defined( _MSC_VER ) /* Visual C++ (and Intel C++) */ - - /* We disable the warning `conditional expression is constant' here */ - /* in order to compile cleanly with the maximum level of warnings. */ - /* In particular, the warning complains about stuff like `while(0)' */ - /* which is very useful in macro definitions. There is no benefit */ - /* in having it enabled. */ -#pragma warning( disable : 4127 ) - -#endif /* _MSC_VER */ - - FT_END_HEADER #endif /* __FTDEBUG_H__ */ diff --git a/include/freetype/internal/ftserv.h b/include/freetype/internal/ftserv.h index 4ac988bd2..cd5fbd0fa 100644 --- a/include/freetype/internal/ftserv.h +++ b/include/freetype/internal/ftserv.h @@ -34,14 +34,6 @@ FT_BEGIN_HEADER -#if defined( _MSC_VER ) /* Visual C++ (and Intel C++) */ - /* We disable the warning `conditional expression is */ - /* constant' in order to compile cleanly with the maximum */ - /* level of warnings. */ -#pragma warning( push ) -#pragma warning( disable : 4127 ) -#endif /* _MSC_VER */ - /* * @macro: * FT_FACE_FIND_SERVICE @@ -763,10 +755,6 @@ FT_BEGIN_HEADER /* */ -#if defined( _MSC_VER ) -#pragma warning( pop ) -#endif - FT_END_HEADER #endif /* __FTSERV_H__ */ diff --git a/include/freetype/internal/internal.h b/include/freetype/internal/internal.h index f500a651c..262afcfa8 100644 --- a/include/freetype/internal/internal.h +++ b/include/freetype/internal/internal.h @@ -4,7 +4,7 @@ /* */ /* Internal header files (specification only). */ /* */ -/* Copyright 1996-2001, 2002, 2003, 2004 by */ +/* Copyright 1996-2004, 2013 by */ /* David Turner, Robert Wilhelm, and Werner Lemberg. */ /* */ /* This file is part of the FreeType project, and may only be used, */ @@ -48,4 +48,16 @@ #define FT_INTERNAL_AUTOHINT_H +#if defined( _MSC_VER ) /* Visual C++ (and Intel C++) */ + + /* We disable the warning `conditional expression is constant' here */ + /* in order to compile cleanly with the maximum level of warnings. */ + /* In particular, the warning complains about stuff like `while(0)' */ + /* which is very useful in macro definitions. There is no benefit */ + /* in having it enabled. */ +#pragma warning( disable : 4127 ) + +#endif /* _MSC_VER */ + + /* END */