On Tue 16.Mar'10 at 19:22:58 +0100, Carlos R. Mafra wrote:
> 
> Perhaps matchIdentifier() could even be deleted and we could use strmatch()
> in its place, as they seem quite similar (why is that cast to 
> 'WFakeGroupLeader'
> in there?)

There are other similar functions defined exclusively to use as the second
argument of WMFindInArray(). 

Look:

[ma...@pilar:wmaker.git]$ git grep matchWindow
src/dock.c:static int matchWindow(void *item, void *cdata)
src/dock.c:             index = WMFindInArray(scr->fakeGroupLeaders, 
matchWindow, (void *)win);
src/event.c:static int matchWindow(void *item, void *cdata)
src/event.c:            while ((widx = WMFindInArray(scr->fakeGroupLeaders, 
matchWindow, (void *)window)) != WANotFound) {

So perhaps it is worth unifying them all to save a few bytes of code
along the way?


-- 
To unsubscribe, send mail to [email protected].

Reply via email to