[Bug 667290] tools: Include locale.h before any i18n headers.

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=667290
  GStreamer | gstreamer (core) | git

--- Comment #3 from Idar Tollefsen <itollefs at cisco.com> 2012-03-21 12:19:11 UTC ---
Indeed, the commit message should have been better.

The issue was with Solaris. And in fact, it's not locale.h that is the real
issues, but macros defined in header files included by locale.h that needs to
be in before the other i18n stuff. These are the macros for X/Open
level/standard and whether extensions to this is to be used or not. I believe
it's stdlib.h that defines these based on compiler flags and defines set on the
command line or before header inclusion.

In any event, if stdlib.h isn't in before the other i18n include files, you'll
end up with either a error message saying something about incompatible UNIX
standards or it won't find prototypes for functions it needs because the chosen
standard level is too low and/or extensions aren't used.

Moving the include to the header sounds good though, and should achieve the
same thing.

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.