Eric Blake <ebl...@redhat.com> writes: > On 12/17/2015 09:49 AM, Markus Armbruster wrote: >> Cc: Fam Zheng <f...@redhat.com> >> Signed-off-by: Markus Armbruster <arm...@redhat.com> >> --- >> block/vmdk.c | 28 +++++++++++++++------------- >> 1 file changed, 15 insertions(+), 13 deletions(-) > > Could have mentioned what the change was: factoring out a common > next_line() helper to let you drop an end-of-loop label.
Adding: Factor out loop stepping to turn a while-loop with goto into a for-loop with continue. > Reviewed-by: Eric Blake <ebl...@redhat.com> Thanks!