On Sun, 29 Sep 2013 at 13:22:49 +0200, Christophe wrote: > From: Christophe CURIS <[email protected]> > > The name will be a problem with next change because it uses a too > common word which will be useful at a better place. > Took the opportunity to rewitre the check in the human logical > thinking order to make it easier to read. > --- > src/WindowMaker.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/WindowMaker.h b/src/WindowMaker.h > index ec8d746..a200c85 100644 > --- a/src/WindowMaker.h > +++ b/src/WindowMaker.h > @@ -238,7 +238,7 @@ typedef enum { > } wprog_state; > > > -#define WCHECK_STATE(state) (state == WProgramState) > +#define WCHECK_STATE(chk_state) (w_global.program.state == (chk_state))
Unfortunately this patch doesn't compile because you define w_global only in the next patch. Does it make sense if I rebase your series and put patch 2 after patch 3? I haven't seen the rest yet, but I guess I can do that. -- To unsubscribe, send mail to [email protected].
