Re: [AD] fshook patch

[ Thread Index | Date Index | More lists.liballeg.org/allegro-developers Archives ]


On 2008-10-07, Thomas Fjellstrom <tfjellstrom@xxxxxxxxxx> wrote:
> > > Index: src/unix/usystem.c
> > > ===================================================================
> > > --- src/unix/usystem.c	(.../4.9)	(revision 11023)
> > > +++ src/unix/usystem.c	(.../4.9-fshook)	(revision 11023)
> > > @@ -21,12 +21,14 @@
> > >  #include <stdlib.h>
> > >  #include <string.h>
> > >  #include <sys/stat.h>
> > > +#include <sys/types.h>
> > > +#include <pwd.h>
> >
> > Might as well wrap with ALLEGRO_HAVE_SYS_TYPES_H
> >
> 
> Do you mind if I just remove that sys/types.h include? the fshook.h include 
> lower down pulls it in, and typedef's off_t if it wasn't seen. I don't really 
> want to put that code in two places, since it'll conflict.

No worries.  It's just something I thought of while browsing through.

Peter





Mail converted by MHonArc 2.6.19+ http://listengine.tuxfamily.org/