[h323plus] Cosmetic issues PTLib v2.0.1 with VS2008-SP1

[Marcos Fábio Jardini]

Hello
Reporting some cosmetic issues during compiling PTLib on VS2008-SP1. The
files:

vsdl.cxx
pvfiledev.cxx
pstun.cxx

Have the macro '#define P_FORCE_STATIC_PLUGIN', the VS compiler outputs:

1>..\..\ptclib\{one of above} : warning C4603: 'P_FORCE_STATIC_PLUGIN' :
macro is not defined or definition is different after precompiled header use
1>        Add macro to precompiled header instead of defining here
1>        ..\..\ptclib\{one of above} : use of precompiled header


But the <ptlib/pluginmgr.h> includes <ptlib/plugin.h>, the later already
define 'P_FORCE_STATIC_PLUGIN' when compiling under _WIN32.
So remove or comment the

#define P_FORCE_STATIC_PLUGIN

from:

vsdl.cxx
pvfiledev.cxx
pstun.cxx

//============================
sockets.cxx

The '#include <ptlib.h>' is the pre-compiled header and MUST come 1st
so, changing '#include <ptlib.h>' to come before '#ifdef __NUCLEUS_PLUS__'

>>>before:
#ifdef __NUCLEUS_PLUS__
#include <ConfigurationClass.h>
#endif


#include <ptlib.h>

>>>after:
#include <ptlib.h>

#ifdef __NUCLEUS_PLUS__
#include <ConfigurationClass.h>
#endif

//============================
Error: sphelper.h(1170)

GetUserDefaultUILanguage(): identifier not found


Configure is using c:\program files\microsoft sdks\windows\v6.0a\
for Located Speech API
compile without speech support.

-- 
-----
Jogue RPG, alimente o lado criativo!
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.packetizer.com/pipermail/h323plus/attachments/20080909/c0d205bd/attachment-0004.html>