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_,,_

_______________________________________________
busybox mailing list
busybox@busybox.net
http://lists.busybox.net/mailman/listinfo/busybox

Reply via email to