[m-rev.] for review: Use format string to build path in io.do_make_temp.

Zoltan Somogyi zoltan.somogyi at runbox.com
Wed Sep 19 10:59:01 AEST 2018



On Wed, 19 Sep 2018 10:35:40 +1000, Peter Wang <novalazy at gmail.com> wrote:
> This should fix Mantis bug #469, where gcc emits a warning for a call to
> strncat, breaking the build.
> 
> library/io.m:
>     Simplify the non-mkstemp branch of io.do_make_temp by building the
>     temporary path with MR_make_string() instead of a combination of
>     strcpy/strcat/strncat. This was already the case in the mkstemp
>     branch.
> 
>     Be more precise in the documentation of io.make_temp_file regarding
>     the truncation of the Prefix argument. Give notification that the
>     behaviour is subject to change.

The diff looks fine.

Zoltan.


More information about the reviews mailing list