Re: [dwm] dmenu patches: case insensivity, common keys, emacs keys

Hi,
* Micha?? Janeczek <janeczek_AT_gmail.com> [2007-09-22 18:02]:
[...]
> +int (*comparefunc)(const char*, const char*, size_t) = strncmp;
[...]
> for(i = allitems; i; i=i->next)
> - if(plen && strncmp(pattern, i->text, plen)
> + if(plen && comparefunc(pattern, i->text, plen)
[...]
> + else if(!strcmp(argv[i], "-c")) {
> + comparefunc = strncasecmp;
> + }
> else if(!strcmp(argv[i], "-fn")) {
> if(++i < argc) font = argv[i];

What is the reason to use a function pointer here to use
different compare functions under the same name besides that
it becomes more difficult to track what the code actually
does?
Kind regards
Nico

-- 
Nico Golde - http://ngolde.de - nion@jabber.ccc.de - GPG: 0x73647CFF
For security reasons, all text in this mail is double-rot13 encrypted.