[Piglit] [PATCH 1/3] PNG files, write mode correction

Lionel Landwerlin lionel.g.landwerlin at intel.com
Tue Jan 10 14:48:21 UTC 2017


The man page of fopen says :

"This is strictly for compatibility with C89 and has no effect; the 'b' 
is ignored on all POSIX conforming systems, including Linux."

I don't know where you're running this, but sure :)

Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>

On 10/01/17 14:38, Sebastian Olender wrote:
> Signed-off-by: Sebastian Olender <sebastian.d.olender at intel.com>
> ---
>   tests/util/piglit-util-png.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/util/piglit-util-png.c b/tests/util/piglit-util-png.c
> index 4835ed6..c63ec7b 100644
> --- a/tests/util/piglit-util-png.c
> +++ b/tests/util/piglit-util-png.c
> @@ -87,7 +87,7 @@ piglit_write_png(const char *filename,
>   		break;
>   	}
>   
> -	fp = fopen(filename, "w");
> +	fp = fopen(filename, "wb");
>   	if (!fp)
>   		abortf("failed to open `%s'", filename);
>   




More information about the Piglit mailing list