[exim-dev] [Bug 721] [PATCH] open libspf dynamicaly

Top Page
Delete this message
Reply to this message
Author: Robert Millan
Date:  
To: exim-dev
Subject: [exim-dev] [Bug 721] [PATCH] open libspf dynamicaly
------- You are receiving this mail because: -------
You are on the CC list for the bug.

http://bugs.exim.org/show_bug.cgi?id=721




--- Comment #8 from Robert Millan <rmh@???> 2008-06-24 12:24:54 ---
(In reply to comment #7)
> On Sunday 15 June 2008, Robert Millan wrote:
> > Does this one look better? Haven't tested it in runtime yet, but I will
> > tomorrow.
>
> I would even go as far as getting rid of dynamic.c altogether and putting
> everything as macros or possibly inline functions (it's experimental after
> all) in dynamic.h. I think we can afford that. And LOAD_FN should be there
> too - it's not specific to SPF. You don't need the libspf_ prefix of
> libspf_dlname and libspf_so_handle.


Ok.

> I would also make the function pointers static variables inside the respective
> stubs.


Are you sure you want that? Then we can't avoid calling dlsym() every time one
of our stubs is used. I'm not sure, but I think dlsym() involves a filesystem
read access.


--
Configure bugmail: http://bugs.exim.org/userprefs.cgi?tab=email