Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 31 Oct 2002 15:34:30 -0500 (EST)
From:      Daniel Eischen <eischen@pcnet1.pcnet.com>
To:        Doug Rabson <dfr@nlsystems.com>
Cc:        ak03@gte.com, tlambert2@mindspring.com, current@freebsd.org
Subject:   Re: [PATCH: libc]Re: gnome on current
Message-ID:  <Pine.GSO.4.10.10210311522180.15812-100000@pcnet1.pcnet.com>
In-Reply-To: <20021031193158.B13073-100000@herring.nlsystems.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, 31 Oct 2002, Doug Rabson wrote:

> On Thu, 31 Oct 2002, Daniel Eischen wrote:
> 
> > This is better, although it has two stubs for each routine.  Adding
> > a weak definition from pthread_foo() to _pthread_foo() (note the
> > lack of _stub) doesn't do the same thing?
> 
> Ok, this version works and seems to be a reasonably tidy solution, at
> least until/if the jump-table version a-la solaris is available. I
> tweaked it slightly to move all the __weak_reference() calls together. Can
> I commit this?

Yes, it looks OK.  I started a slightly different version that
automatically added both references with one macro.  Something
like:

#define	WEAK_REF(sym, alias)	__weak_reference(sym, alias)

#define	DUAL_WEAK_STUB(sym)	\
	WEAK_REF(__CONCAT(sym, _stub), __CONCAT(_, sym)); \
	WEAK_REF(__CONCAT(sym, _stub), sym)

DUAL_WEAK_STUB(pthread_mutex_lock);
...

or something like that.

Go ahead and commit yours if you don't care to fiddle any
more with it.  I won't be able to get to the jump table
thing for a few days.

-- 
Dan Eischen


To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-current" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?Pine.GSO.4.10.10210311522180.15812-100000>