[v2] jdupes 1.11.1 port to BusyBox
Jody Bruchon
jody at jodybruchon.com
Wed Dec 19 12:25:29 UTC 2018
On 12/19/2018 7:10 AM, Xabier Oneca -- xOneca wrote:
> Hi,
>
>> The redundant unknown option text is there so the big 'case' has a default block. It should never be reached, but it's there for safety just in case. I suppose the fprintf() could be changed out to the new unsupported_option() function for a tiny space savings, though it's probably not worth a whole new submission just to swap that one line out and the only gain would be reduction of the string size by a few bytes.
> I meant the other way around: leave the default case, and remove each
> disabled case. So you have only one fprintf for all disabled options.
Ah, I see. I could probably do that.
> Cheers,
>
> Xabier Oneca_,,_
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/busybox/attachments/20181219/a9213b5e/attachment.html>
More information about the busybox
mailing list