On Tue, May 4, 2010 at 5:57 PM, Johan Kotlinski <kotlinski@gmail.com> wrote: > > if(currentNode->dir_entry != NULL) > > { > > free(currentNode->dir_entry); > > } > > One suggestion: You could remove this unnecessary NULL check. free() > does nothing when passed a null pointer. > > Thank you. Little nuggets like this go a long way. -- Payton Byrd <http://www.paytonbyrd.com> <http://it.toolbox.com/blogs/paytonbyrd> ---------------------------------------------------------------------- To unsubscribe from the list send mail to majordomo@musoftware.de with the string "unsubscribe cc65" in the body(!) of the mail.Received on Wed May 5 01:57:58 2010
This archive was generated by hypermail 2.1.8 : 2010-05-05 01:58:00 CEST