[sane-devel] saned option handling changes (was Re: [PATCH v4 1/5] saned: parse standalone args in main())
paddy-hack at member.fsf.org
Tue Oct 13 14:36:17 UTC 2015
Matteo Croce writes:
> Hi Olaf,
> great, but I think that commit 45e66aee is wrong because it's
> shadowing the global variable 'bind_addr'.
> Also there is no need to initialize it to NULL because static
> variables are always set to 0.
I write a lot of C++. There static variables are initialized by their
default constructor and I always forget what happens to POD-types. I
normally prefer to be explicit.
Anyway, I've reverted the offending commit.
Hope this helps,
Olaf Meeuwissen, LPIC-2 FSF Associate Member since 2004-01-27
Support Free Software Support the Free Software Foundation
GnuPG key: F84A2DD9/B3C0 2F47 EA19 64F4 9F13 F43E B8A4 A88A F84A 2DD9
More information about the sane-devel