[v2] jdupes 1.11.1 port to BusyBox

Xabier Oneca -- xOneca xoneca at gmail.com
Wed Dec 19 12:10:03 UTC 2018


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.

Cheers,

Xabier Oneca_,,_


More information about the busybox mailing list