RFC: size-saving boolean type for busybox

Denis Vlasenko vda.linux at googlemail.com
Tue Jan 2 15:38:01 PST 2007


On Tuesday 02 January 2007 18:40, David Daney wrote:
> > #if defined(i386) /* || defined(...) as needed */
> > typedef unsigned char bbool;
> > #else
> > typedef unsigned bbool;
> > #endif
>
> Did you do a size comparison?

After some thought I decided to call it "small[u]int" and converted
fsck_minix. Result:

check_zones2                                         230     231      +1
check_zones                                          213     214      +1
add_zone_ind2                                         80      81      +1
add_zone_ind                                          80      81      +1
add_zone_dind2                                        80      81      +1
version2                                               8       5      -3
verbose                                               24      21      -3
termios_set                                            4       1      -3
show                                                   4       1      -3
repair                                                 4       1      -3
read_block                                           232     229      -3
list                                                 393     390      -3
force                                                  4       1      -3
errors_uncorrected                                     4       1      -3
check_zone_nr2                                       122     119      -3
changed                                                4       1      -3
automatic                                              4       1      -3
write_block                                          600     594      -6
ask                                                  184     178      -6
add_zone2                                            225     216      -9
add_zone                                             224     215      -9
fsck_minix_main                                     4120    4096     -24
recursive_check                                     1206    1179     -27
recursive_check2                                    1397    1361     -36
------------------------------------------------------------------------------
(add/remove: 0/0 grow/shrink: 9/24 up/down: 37/-177)         Total: -140 bytes


More information about the busybox mailing list