Eric Blake <ebl...@redhat.com> writes: > On 05/02/2014 06:44 AM, Markus Armbruster wrote: >> When visit_start_struct() succeeds, visit_end_struct() must not be > > s/succeeds/fails/ (this really confused me on my first read, until I saw > the code and the subject line and determined the typo)
D'oh! Will fix both here an in 10/13/ >> called. rtc_get_date() and balloon_stats_all() call it anyway. As >> far as I can tell, they're only used with the string output visitor, >> which doesn't care. Fix them anyway. >> >> Signed-off-by: Markus Armbruster <arm...@redhat.com> >> --- >> hw/timer/mc146818rtc.c | 23 +++++++++++++++-------- >> hw/virtio/virtio-balloon.c | 25 +++++++++++++++++++------ >> 2 files changed, 34 insertions(+), 14 deletions(-) > > With commit message fixed, > > Reviewed-by: Eric Blake <ebl...@redhat.com> Thanks!
