On 20/07/11 17:53, Tarmigan Casebolt wrote:
> * src/mktemp.c: The comma syntax of the previous code (introduced in
> be6120b5) was unusual and hard for a human to parse.  The underlying
> logic is simple even if it does take a few more lines of code, so make
> the code easier for a human to read.

We discussed this previously and decided adding 4 lines
was not warranted to avoid the compiler warning.

> In addition, (though this case is likely fine) some compilers may have
> unexplored corner cases with comma separated statements.  See
>     http://blog.regehr.org/archives/558
> for a further discussion.

Thanks for the info though.

cheers,
Pádraig.

Reply via email to