[Date Prev][Date Next] [Chronological] [Thread] [Top]

Re: (ITS#5764) addpartial patch



dhawes@vt.edu wrote:
> Pierangelo Masarati wrote:
>> dhawes@vt.edu wrote:
>>> Full_Name: David Hawes
>>> Version: 2.4.12
>>> OS: URL: ftp://ftp.openldap.org/incoming/david-hawes-081021.patch
>>> Submission from: (NULL) (128.173.38.164)
>>>
>>>
>>> This patch includes the following changes:
>>>
>>> - -fPIC added to the Makefile for compilation on x86_64 systems.
>>> - No longer use be_search() to retrieve the entry that is to be
>>> compared.  Use
>>> be_entry_get_rw() instead.
>> You should probably use overlay_entry_get_ov() rather than
>> be_entry_get_rw() from inside an overlay (AFAIR).
> 
> Okay, I'll make that change, thanks.
> 
> I notice that most overlays seem to use be_entry_get_rw(), and one
> (translucent) uses both.  What is the reasoning to use one over the other?

New patch that uses overlay_entry_get_ov() available at:

ftp://ftp.openldap.org/incoming/david-hawes-081104.patch

The patch referenced previously can be discarded.  This patch contains
all changes against 2.4.12.