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

Re: provideruri redundancy in syncrepl code

Pierangelo Masarati wrote:

        char                            *si_provideruri;
        BerVarray                       si_provideruri_bv;

I don't see any need for having a string and a BerVarray of provideruri
in syncrepl; wouldn't a single berval suffice?  I'm fixing this, unless
there's any specific reason.

There were a lot of places using BerVarray's for no apparent reason, when only the first element of the array was ever referenced. I didn't have the time to fix them all. I think it would be good to clean up all of them.

 -- Howard Chu
 Chief Architect, Symas Corp.       Director, Highland Sun
 http://www.symas.com               http://highlandsun.com/hyc
 Symas: Premier OpenSource Development and Support