Re: [PATCH v2 0/3] MR615: vkd3d-shader: Fix a couple of memory leaks.

[Francisco Casas (＠fcasas)]

Francisco Casas (@fcasas) commented about programs/vkd3d-compiler/main.c:
>  int ret;
>
>  if (!parse_command_line(argc, argv, &options))
> -
>    return 1;
> -
>    goto done;
>
> if (options.print_help)
>    {
>        print_usage(argv[0]);
> -
>    return 0;
> -
>    fail = 0;

It may be just me, but I think that it is more intuitive to set `fail = 1` on errors rather than `fail = 0` on regular code paths.