argutil.c:63 reads: for (; *a; a = a++) a = a++ is undefined in C. The function containing that code ( dcc_argv_search ) is not used anywhere, in any case. I have noticed other bits that could be cleaned up, should I do so? Is there a preferred way of invoking diff for submitting patches?