U++ forum: Welcome to the forum

Search on this site

Search in forums

Home » U++ Library support » U++ Libraries and TheIDE: i18n, Unicode and Internationalization » CtrlCore: X11App.cpp should #include <locale.h> [BUG][FIXED]

Show: Today's Messages :: Show Polls :: Message Navigator
E-mail to friend

CtrlCore: X11App.cpp should #include <locale.h> [BUG][FIXED] [message #2393]

Mon, 10 April 2006 22:38

hojtsy
Messages: 241
Registered: January 2006
Location: Budapest, Hungary

Experienced Member

The file CtrlCore/X11App.cpp uses constants (LC_ALL) and functions from the locale.h header but it does not include it. I suppose that in some environment it is included indirectly by some other header file, but on a Linux I have tried, it was required to include this header directly to X11App.cpp. Please correct this.

[Updated on: Tue, 02 May 2006 14:46] by Moderator

Report message to a moderator

Re: X11App.cpp should #include <locale.h> [message #2595 is a reply to message #2393]

Tue, 18 April 2006 15:55

hojtsy
Messages: 241
Registered: January 2006
Location: Budapest, Hungary

Experienced Member

I see that this has been corrected in 604.dev2. But would it be possible to move the two #include directives of X11App.cpp into the body of #ifndef PLATFORM_X11 ? It would speed up windows compilation a bit.

Report message to a moderator

Re: X11App.cpp should #include <locale.h> [message #2598 is a reply to message #2595]

Tue, 18 April 2006 16:16

mirek
Messages: 14105
Registered: November 2005

Ultimate Member

Two? I see just #include <locale.h> - yes, a good idea, moved.

Mirek

Report message to a moderator

Re: X11App.cpp should #include <locale.h> [message #2600 is a reply to message #2598]

Tue, 18 April 2006 16:50

hojtsy
Messages: 241
Registered: January 2006
Location: Budapest, Hungary

Experienced Member

Two:
#include "CtrlCore.h"
#include <locale.h>

[Updated on: Tue, 18 April 2006 16:50]

Report message to a moderator

Re: X11App.cpp should #include <locale.h> [message #2603 is a reply to message #2600]

Tue, 18 April 2006 18:52

mirek
Messages: 14105
Registered: November 2005

Ultimate Member

"PLATFORM_X11" is defined by #include "CtrlCore.h".....

Mirek

Report message to a moderator

Re: X11App.cpp should #include <locale.h> [message #2604 is a reply to message #2603]

Tue, 18 April 2006 19:45

hojtsy
Messages: 241
Registered: January 2006
Location: Budapest, Hungary

Experienced Member

luzr wrote on Tue, 18 April 2006 12:52

"PLATFORM_X11" is defined by #include "CtrlCore.h".....

Oh I thought that this macro is defined on the command line by the builder if LINUX option is paired with GUI option.

Report message to a moderator

Re: X11App.cpp should #include <locale.h> [message #2605 is a reply to message #2604]

Tue, 18 April 2006 20:20

mirek
Messages: 14105
Registered: November 2005

Ultimate Member

No. The logic is that each compilation flag XXX is reflected as "flagXXX" define in C++. It is then tested in Core.h (in quite ugly and fuzzy preprocessing) and combined with other macros identifying compilers etc... to produce PLATFORM_*, CPU_* and COMPILER_* (reminds me I should finally find time to document them....)

Mirek

Report message to a moderator

Previous Topic:	Locale troubles with UPP
Next Topic:	MultiByte Coding: one half of Chinese chars displayed [BUG?]

Goto Forum:

-=] Back to Top [=-

[ Syndicate this forum (XML) ] [

] [

]

Current Time: Fri Nov 01 01:29:15 CET 2024

Total time taken to generate the page: 0.02185 seconds