Denys Vlasenko wrote:
> ...
> function                                             old     new   delta
> telnetd_main                                        1355    1350      -5
> make_new_session                                     532     521     -11
> free_session                                         118     101     -17
> ------------------------------------------------------------------------------
> (add/remove: 0/0 grow/shrink: 0/3 up/down: 0/-33)             Total: -33 bytes
>    text    data     bss     dec     hex filename
>  676328    2538   12104  690970   a8b1a busybox_old
>  676295    2538   12104  690937   a8af9 busybox_unstripped
>
> Please review attached (or see current svn).
It seems fine.

I'm surprised that your changes to telnetd_main removed only 5 bytes. 
The compiler seems to be quite clever already. But with your changes it 
is also obvious to the human reader that kill_session does the same each 
time.

Regards
Ralf Friedl
_______________________________________________
busybox mailing list
busybox@busybox.net
http://busybox.net/cgi-bin/mailman/listinfo/busybox

Reply via email to