[PATCH gpgme] Fix error handling in _gpgme_parse_status().

Werner Koch wk at gnupg.org
Thu Nov 15 21:52:44 CET 2012


On Thu, 15 Nov 2012 19:20, xi.wang at gmail.com said:

> The storage of an enum is compatible with int, but the signedness is
> very tricky.

Now I see what you are meaning.  The test is optimized out.  I am used
to see something like

  warning: comparison of unsigned expression >= 0 is always true

in such cases but that does not happen with GPGME.  The reason is that,
for unknown reasons, GPGME does not test for gcc flags in configure and
uses way too less warning switches.  This definitely needs to be fixed.

> It's better to fix this by changing the return type to signed int, or

Right that is how I would have solved this too.

Thanks,

  Werner

-- 
Die Gedanken sind frei.  Ausnahmen regelt ein Bundesgesetz.




More information about the Gnupg-devel mailing list