<html>
<body>
Good catch. I'm not sure what's the desired software architecture, but
cairo-boilerplate.c includes cairo-types-private.h which proudly
announces:<br><br>
<font face="Arial, Helvetica" color="#008000">/* This is the only header
file not including cairoint.h. It only contains<br>
* typedefs.*/<br>
</font><font face="Arial, Helvetica" color="#0000FF">#include</font>
<font face="Arial, Helvetica">
</font><font face="Arial, Helvetica" color="#A31515">
"cairo.h"<br>
</font><font face="Arial, Helvetica" color="#0000FF">#include</font>
<font face="Arial, Helvetica">
</font><font face="Arial, Helvetica" color="#A31515">
"cairo-fixed-private.h"<br><br>
</font>Including cairoint.h in cairo-fixed-private.h results in a bunch
of hard to trace errors, in addition to not respecting the comment above.
<br><br>
We could include cairoint.h in every file (cairo-boilerplate.c,
<font face="Arial, Helvetica">cairo-boilerplate-ps.c,
cairo-boilerplate-svg.c, etc) t</font>hat might include
cairo-fixed-private.h (and then any other file using inline but not
already including cairoint.h), but that's really ugly. Maybe we should
move these platform dependant #define to some other header file? seems
like they're not really related to the rest of cairoint.h
anyway.<br><br>
Antoine<br><br>
At 03:44 PM 1/18/2008, Carl Worth wrote:<br>
<blockquote type=cite class=cite cite="">On Fri, 18 Jan 2008 12:31:37
-0800, Vladimir Vukicevic wrote:<br>
> Hm, at some point there was something like<br>
> #ifdef _MSC_VER<br>
> #define inline __inline<br>
> #endif<br><br>
I still see this in cairoint.h:<br><br>
<x-tab> </x-tab>#ifdef
_MSC_VER<br>
<x-tab> </x-tab>#define
snprintf _snprintf<br>
<x-tab> </x-tab>#undef
inline<br>
<x-tab> </x-tab>#define
inline __inline<br>
<x-tab> </x-tab>#endif<br>
<br>
Is something just failing to include that somewhere?<br><br>
-Carl<br><br>
</blockquote></body>
</html>