[PATCH] scanner: Fix reported executable name to 'wayland-scanner'

Yong Bakos junk at humanoriented.com
Fri Jun 3 23:34:38 UTC 2016


Hi,

On Jun 2, 2016, at 8:16 PM, Jonas Ådahl <jadahl at gmail.com> wrote:
> 
> On Thu, Jun 02, 2016 at 02:42:44PM -0700, Bryce Harrington wrote:
>> 'wayland-scanner -v' (correctly) reports the program as named
>> "wayland-scanner", but 'wayland-scanner -h' was inconsistent, referring
>> to it as './scanner'.
>> 
>> Signed-off-by: Bryce Harrington <bryce at osg.samsung.com>

Bryce, good catch! But...


> I guess we could also pass argv and use argv[0], but this works as well.
> 
> Reviewed-by: Jonas Ådahl <jadahl at gmail.com>


I don't follow this at all, because...


>> ---
>> src/scanner.c | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>> 
>> diff --git a/src/scanner.c b/src/scanner.c
>> index 5f06e8e..ae2326f 100644
>> --- a/src/scanner.c
>> +++ b/src/scanner.c
>> @@ -57,8 +57,8 @@ enum side {
>> static int
>> usage(int ret)
>> {
>> -	fprintf(stderr, "usage: ./scanner [OPTION] [client-header|server-header|code]"
>> -		" [input_file output_file]\n");
>> +	fprintf(stderr, "usage: wayland-scanner [OPTION] [client-header|server-header|code]"
>> +		" [input_file output_file]\n", argv[0]);

argv is out of scope here in `usage`. This won't compile without changing
the parameter list, and the passed args at all call sites.

Also, shouldn't the diff be more like (notice the %s format specifier):

+	fprintf(stderr, "usage: %s [OPTION] [client-header|server-header|code]"
+		" [input_file output_file]\n", argv[0]);


>> 	fprintf(stderr, "\n");
>> 	fprintf(stderr, "Converts XML protocol descriptions supplied on "
>> 			"stdin or input file to client\n"
>> -- 
>> 1.9.1

Regards,
yong




More information about the wayland-devel mailing list