On Mon, 2006-09-25 at 13:08 +0200, Nico Erfurth wrote:
> + case 'a':
> + if (Ustrncmp(big_buffer, "-acl ", 5) == 0)
> + else if (Ustrncmp(big_buffer, "-aclc ", 6) == 0 ||
> + Ustrncmp(big_buffer, "-aclm ", 6) == 0)
Wouldn't it make sense to switch those two around saving another string
comparison in the common case that only new variables are present, and
possibly even moving the "-acl " part all the way to the end? As-is,
that'll always be executed but never result in a match (on new-style acl
spool files).
johannes
