On Fri, 2007-05-04 at 12:20 +0100, Mike Dransfield wrote: > David Reveman wrote: > > On Fri, 2007-05-04 at 00:49 +0100, Mike Dransfield wrote: > > > >> I am fairly sure that this is just a bug in the codes logic, but > >> I thought I should check first > >> > > > > hm, I think the code is currently doing the correct thing. > > > > In that case I think this patch is more appropriate ;) > > Do you think it is clear and correct?
Yes, that's how that function works so please add that comment if it wasn't obvious. I assume that this part was just some local change that really shouldn't be in the patch: @@ -655,6 +655,7 @@ typedef Bool (*ImageToFileProc) (CompDisplay *display, #define NOTIFY_DELETE_MASK (1 << 1) #define NOTIFY_MOVE_MASK (1 << 2) #define NOTIFY_MODIFY_MASK (1 << 3) +#define NOTIFY_OPEN_MASK (1 << 4) typedef void (*FileWatchCallBackProc) (const char *name, void *closure); - David _______________________________________________ compiz mailing list compiz@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/compiz