Comment 6 for bug 314048

Revision history for this message
Steven Van Ingelgem (steven-vaningelgem-be) wrote : Re: [Bug 314048] Re: Cut down on compiler warnings

Hi Yury,

I don't think I ever do a "> 1200" because that's bad coding. I generally
include more files.
What I should do is start from my mega-patch and check why it breaks things.
That shouldn't be too difficult I suppose ;-).

Grtz,
Steven

2009/1/12 Yury V. Zaytsev <email address hidden>

> It seems that the reason for those errors are statements like this
> (example taken from cttypes.h):
>
> // FIXME: MSVC++ 6
>
> #if _MSC_VER > 1200
>
> typedef uint16_t WORD;
>
> #endif // _MSC_VER > 1200
>
>
> WORD is not defined in GCC, but obviously does not get defined because
> _MSC_VER is not defined as well. I am to lame to propose a viable solution.
> Is it possible to convert those statements in the following form:
>
> #if (_MSC_VER > 1200) && IS_GCC
>
> where IS_GCC would be defined if the compiler is GCC?
>
> --
> Cut down on compiler warnings
> https://bugs.launchpad.net/bugs/314048
> You received this bug notification because you are a member of Cuneiform
> Linux, which is the registrant for Cuneiform for Linux.
>
> Status in Linux port of Cuneiform: New
>
> Bug description:
> The code currently emits quite a lot of compiler warnings. Getting rid of
> these would be beneficial
>
> Attached is a patch from Steven Van Ingelgem that removes a lot of MSVC
> warnings. Unfortunately it breaks recognition completely on OS X.
>