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

Reply via email to