[PATCH 1/4] cli: fix use of uninitialized variable in "notmuch reply"
David Bremner
david at tethera.net
Fri Jan 6 03:57:41 PST 2012
On Fri, 06 Jan 2012 10:11:42 +0200, Jani Nikula <jani at nikula.org> wrote:
> IMHO the first is cleanest and unaffected by changes in
> notmuch_show_params_t, though might be surprising if you're not (yet)
> used to C99 designated initializers.
Obviously not ;). It would nice to have a consensus about the use of C99
features in the notmuch code. Personally I think it's fine, but I know
we have had some back and forth about e.g. mid-block variable declarations.
> In any case, in the current implementation only .part and .cryptoctx are
> initialized, and the rest of the fields are random. This needs to be
> fixed one way or another.
Agreed.
d
More information about the notmuch
mailing list