Eric Blake <ebl...@redhat.com> writes: > programmingk...@gmail.com[*] reported a runtime failure on a > 32-bit Mac OS compilation, where "%"PRId64 expands to "%qd". > Fortunately, we had very few spots that were relying on our > pseudo-printf JSON parsing of int64_t numbers, so it was > easier to just convert callers to stick to safer %lld. > > The remaining uses of pseudo-printf handling are more complex; > there are only 3 users in the released codebased, but LOTS of > users in the testsuite (via wrapper functions like qmp()); I > will be posting a followup series that rips out the remaining > uses of dynamic JSON, but it will be 2.9 material, while > these (first three) patches qualify for 2.8. The fourth patch > is RFC; not intended to be applied now, but shows how I tested > patch 3/4; it will probably reappear in the later 2.9 series. > > [*] git log shows the name John, but the particular email that > sparked this only stated the non-descript name 'G 3', which > makes it a bit hard for me to know which form is preferred > when lending credit.
PATCH 1-3 applied to qapi-next for 2.8 with minor touch-ups. Thanks!
