> > > Thanks! The v5 patch looks good to me. Let's see if all agree on the
> > > goto duplicate_error approach which could reduce the LOC by ~80.
> >
> > I think the "goto duplicate_error" approach looks good, it avoids
> > duplicating the same error code multiple times.
>
> Thanks. I will mark the v5 patch "ready for committer" if no one has comments.
Hi,
I looked into the patch and noticed a minor thing.
+ return; /* keep compiler quiet */
}
I think we do not need the comment here.
The compiler seems not require "return" at the end of function
when function's return type is VOID.
In addition, it seems better to remove these "return;" like what
commit "3974c4" did.
Best regards,
houzj