Subject: Re: Replacement for grep(1) (part 2)
To: Assar Westerlund <email@example.com>
From: Dag-Erling Smorgrav <firstname.lastname@example.org>
Date: 07/07/1999 23:01:13
Assar Westerlund <email@example.com> writes:
> Dag-Erling Smorgrav <firstname.lastname@example.org> writes:
> > + realpat = grep_malloc(strlen(pattern) + sizeof("^(")
> > + + sizeof(")$") + 1);
> Why not just use asprintf?
Doesn't matter, thsis code is gone in the latest version. You though
the linux 'kernel of the day' was bad; now you have the FreeBSD 'grep
of the minute' :)
> In this case it doesn't matter but in general this function is wrong.
> malloc(0) can return NULL.
> And besides, I really don't think this is a grep function but actually
> is useful for programs that don't have any strategy for handling out
> of memory errors and might as well die (with a descriptive error
> message, of course). Let's call it emalloc and let's put in somewhere
> where it can be used.
Too simple to warrant that, and other programs will likely want to
handle the error differently.
Dag-Erling Smorgrav - email@example.com