[prev in list] [next in list] [prev in thread] [next in thread] 

List:       busybox
Subject:    Re: add support for PATH in crontabs
From:       Paul Fox <pgf () foxharp ! boston ! ma ! us>
Date:       2022-03-08 21:12:54
Message-ID: 20220308211254.E21A92E1177 () grass ! foxharp ! boston ! ma ! us
[Download RAW message or body]

peter0x44 wrote:
 > > I'm afraid I can't remember how to generate the cool automatic size
 > > diffs. The code change seems to add fewer than 100 bytes, but I also
 > > ifdefed a very old level 5 debug loop in Parsefield(), which saved
 > > over 100. So I think there's been a net shrinkage of about 30 bytes.
 > 
 > Hi Paul,
 > 
 > The size diffs are generated using `make bloatcheck`
 > 

Thanks Peter -- That confirms what I thought:

function                                             old     new   delta
load_crontab                                        1043    1117     +74
.rodata                                             3509    3515      +6
bb_putchar_stderr                                     38       -     -38
ParseField                                           507     437     -70
------------------------------------------------------------------------------
(add/remove: 0/1 grow/shrink: 2/1 up/down: 80/-108)           Total: -28 bytes
   text    data     bss     dec     hex filename
  78292    2767    1528   82587   1429b busybox_old
  78264    2767    1528   82559   1427f busybox_unstripped




=----------------------
paul fox, pgf@foxharp.boston.ma.us (arlington, ma)

_______________________________________________
busybox mailing list
busybox@busybox.net
http://lists.busybox.net/mailman/listinfo/busybox
[prev in list] [next in list] [prev in thread] [next in thread]