5 more simple patches for usr.bin/make

Joerg Sonnenberger joerg at britannica.bec.de
Sat Nov 13 09:43:01 PST 2004


On Wed, Nov 10, 2004 at 04:02:32PM -0800, Max Okumoto wrote:
> Here are five more independent patches for make.  The description
> of the patch is at the top of the patch files. They are all cosmetic
> and do not change functionality.
> 
> 				Max

One thing you can improve is the use of static strings with write.
E.g. instead of write(2, "BLA", 3); something either like
const char msg_bla[] = "BLA";
write(2, msg_bla, sizeof(msg_bla));

or 

#define WRITE(fd, str) write(fd, str, sizeof(str))

would be useful, the hardwired string length is evil. Be careful though,
the sizeof only works with char[], not char * !

Joerg





More information about the Submit mailing list