On Tue, 13 Apr 2010 at 18:34:52 +0200, Tamas TEVESZ wrote: > On Tue, 13 Apr 2010, Carlos R. Mafra wrote: > > > Plain integer as NULL pointer, non-ANSI function declaration etc. > > to be amended with: > > diff --git a/src/superfluous.c b/src/superfluous.c > index 59ce334..a270b53 100644 > --- a/src/superfluous.c > +++ b/src/superfluous.c > @@ -34,8 +34,8 @@ > > #include "WindowMaker.h" > #include "screen.h" > -#include "superfluous.h" > #include "dock.h" > +#include "superfluous.h" > #include "framewin.h" > #include "window.h" > #include "actions.h" > > :)
Ops, indeed. But what do you think about the solution below? As superflous.h is using "dock" and "screen" stuff, it should better include their headers, no? diff --git a/src/superfluous.h b/src/superfluous.h index 8c9135a..0c6af1c 100644 --- a/src/superfluous.h +++ b/src/superfluous.h @@ -19,6 +19,9 @@ * USA. */ +#include "dock.h" +#include "screen.h" + #define PIECES ((64/ICON_KABOOM_PIECE_SIZE)*(64/ICON_KABOOM_PIECE_SIZE)) extern void DoKaboom(WScreen *scr, Window win, int x, int y); -- To unsubscribe, send mail to [email protected].
