On Fri, Sep 16, 2022 at 1:48 PM Markus Armbruster <arm...@redhat.com> wrote: > > Ani Sinha <a...@anisinha.ca> writes: > > > On Fri, Sep 16, 2022 at 2:13 AM Markus Armbruster <arm...@redhat.com> wrote: > >> > >> The has_FOO for pointer-valued FOO are redundant, except for arrays. > >> They are also a nuisance to work with. Recent commit "qapi: Start to > >> elide redundant has_FOO in generated C" > > > > Commit is referenced using <commit hash 13 chars min> ("commit header") > > Apply my patches in your tree, and your hashes will differ from mine. > Hashes can serve as stable reference only when we git-fetch patches, not > when we git-send-email then.
Has the referenced patch not been pushed upstream already? I thought it was and hence was asking for the reference. > > >> provided the means to elide > >> them step by step. This is the step for qapi/acpi.py. > >> > >> Said commit explains the transformation in more detail. The invariant > >> violations mentioned there do not occur here. > >> > >> Cc: Michael S. Tsirkin <m...@redhat.com> > >> Cc: Igor Mammedov <imamm...@redhat.com> > >> Cc: Ani Sinha <a...@anisinha.ca> > >> Signed-off-by: Markus Armbruster <arm...@redhat.com> >