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!

Reply via email to