Ricardo Mestre wrote:
> There's no need to have a handrolled function for a simple compare that
> is just used once throughout all the source, or should it be kept for
> code readability?

>       if ((info->search_type & SRCH_NO_REGEX) == 0)
> -             return (!is_null_pattern(info->compiled));
> +             return (info->compiled != NULL);
>       return (info->text != NULL);

nope. this function is silly. why no is_null_text function for the next line?
applied. thanks.

Reply via email to