[PATCH] include/obstack.h: do not hide _obstack_newchunk
Bernhard Reutner-Fischer
rep.dot.nop at gmail.com
Mon Jun 24 15:51:28 UTC 2013
On 24 June 2013 15:25:20 "Anthony G. Basile" <basile at opensource.dyc.edu> wrote:
> On 06/23/2013 05:25 PM, Anthony G. Basile wrote:
> > On 06/23/2013 02:16 PM, Bernhard Reutner-Fischer wrote:
> >> On 23 June 2013 18:14:08 "Anthony G. Basile" <basile at opensource.dyc.edu>
> >> wrote:
> >>> From: "Anthony G. Basile" <blueness at gentoo.org>
> >>>
> >>> Programs using obstacks, like the suite of utilities provided by
> >>> coreutils, need _obstack_newchunk so we unhide the symbol.
> >>
> >> Eh, that sounds pretty broken. Are you sure this is the right thing to
> >> do? They should most likely obstack_*grow() instead.
> >>
> >> Please fix coreutils instead, or better yet, use Busybox.
> >> Thanks,
> >
>
> >
> > Let me do some poking around and see what's going on.
> >
> >
>
> Hi Bernhard,
>
> Actually its pretty clear what's going on here. Many obstack functions are
> implemented as macros around _obstack_newchunk, so "functions" like
> obstack_*grow() are decls which expand to calls to _obstack_newchunk. Read
> <obstack.h> around line 292. Here's a simple poc:
Right, completely forgot this.
Thanks for the fix!
Sent with AquaMail for Android
http://www.aqua-mail.com
More information about the uClibc
mailing list