5 more simple patches for usr.bin/make

Matthew Dillon dillon at apollo.backplane.com
Sat Nov 13 11:48:50 PST 2004


: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

    Well, remember that a string assignment to a char[] array includes
    the terminating \0.. so that will not work quite the way you wrote 
    it.

    Just do this:

	const char * const msg = "BLA";

	write(fd, msg, strlen(msg));

    GCC will calculate the strlen() for you at compile-time.

					-Matt
					Matthew Dillon 
					<dillon at xxxxxxxxxxxxx>





More information about the Submit mailing list