5 more simple patches for usr.bin/make
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.
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));
#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 * !
More information about the Submit