[PATCH] size reduction and clean up swaponoff.c
Bernhard Fischer
rep.nop at aon.at
Sat Sep 17 17:46:46 UTC 2005
On Sat, Sep 17, 2005 at 06:50:03AM -0500, Rob Landley wrote:
>On Saturday 17 September 2005 05:19, Bernhard Fischer wrote:
>> On Fri, Sep 16, 2005 at 12:50:49AM -0400, Mike Frysinger wrote:
>> >On Wednesday 14 September 2005 11:12 am, Rob Landley wrote:
>> >> On Monday 12 September 2005 21:26, Mike Frysinger wrote:
>> >> > unrelated to this patch, and minor overall, but our swap utils
>> >> > different
>> >
>> >if they really care they could just pipe into /dev/null
>>
>> I put CURRENT_TTY and CONSOLE_DEV into their own obj files.
>
>Ouch.
>
>> Should we put DEV_NULL also in it's obj file contained in libbb?
>
>/dev/null, /dev/tty, and /dev/console are all constant strings, right? Living
>at standard defined places?
yes.
>
>If you're saying we should have one instance of these strings to save space
>and can't trust compilers to get this right, fine. But that's a larger
>problem, there's a a lot of strings like that. Putting each one in its
There are indeed more of these.
>own .o file is _not_ a good solution here.
hm, why not?
>
>We need to work out some kind of mechanism for this, but having an
>individual .c file and individual .o file for each string is just... Ick.
We do *not* have individual .c files for them. In another thread i
proposed to put them into messages.c but you objected ¹). I locally have
them now in libbb/static_strings.c but any suggestion for a more
appropriate name or place is welcome. It's really a minor detail, imho,
but i want to resolve it sooner rather than later.
¹) later in said thread, you admitted that you "tend to have a tropism for
_fewer_ files", but ok.
More information about the busybox
mailing list