Re: [hackers] [farbfeld] Fix 2 little things in jpg2ff(1) || FRIGN

From: FRIGN <dev_AT_frign.de>
Date: Sun, 6 Mar 2016 20:14:54 +0100

On Sun, 6 Mar 2016 07:47:38 +0100
k0ga_AT_shike2.com wrote:

Hey Roberto,

> > - if (argc > 1) {
> > + if (argc) {
> > fprintf(stderr, "usage: %s\n", argv0);
> > return 1;
> > }

> recommendation about style; Use argc == 0 here and use the form
> without relational operator for logic or pointers values.

You know how it is with style, but I respect your opinion on this
matter as a senior programmer.
I was wondering, don't you mean "argc != 0"? :)

Cheers

FRIGN

-- 
FRIGN <dev_AT_frign.de>
Received on Sun Mar 06 2016 - 20:14:54 CET

This archive was generated by hypermail 2.3.0 : Sun Mar 06 2016 - 20:24:19 CET