[Date Prev][Date Next]
RE: Untested patch: back-tcl used wrong types (ITS#1719)
The nnewRdn and nnewSuperior arguments are the normalized version of the
newRdn and newSuperior. back-perl apparently doesn't care about
normalization. I've never set up back-tcl and I have no idea what it needs
to do. It's been unsupported since about when I joined the team, I think...
-- Howard Chu
Chief Architect, Symas Corp. Director, Highland Sun
Symas: Premier OpenSource Development and Support
> -----Original Message-----
> From: owner-openldap-bugs@OpenLDAP.org
> [mailto:owner-openldap-bugs@OpenLDAP.org]On Behalf Of
> Sent: Wednesday, April 03, 2002 12:37 AM
> To: openldap-its@OpenLDAP.org
> Subject: Untested patch: back-tcl used wrong types (ITS#1719)
> Full_Name: Hallvard B. Furuseth
> Version: HEAD (2002-04-02)
> URL: http://folk.uio.no/hbf/OpenLDAP/tcl.txt
> Submission from: (NULL) (18.104.22.168)
> This patch is untested (except for compiling it); a compiler bug
> prevented me from compiling it completely.
> back-tcl is outdated:
> - It expected be->be_suffix to be a char**, not a struct berval**.
> - It expected the string arguments to the tcl_back_*() routines
> to be const char*, not struct berval.
> - tcl_back_compare() expected an Ava* instead of AttributeAssertion*.
> - tcl_back_modify() expected an LDAPModList* instead of Modifications*.
> - tcl_back_modrdn() expected too few arguments.
> - And finally, back-tcl did printf(%ld, ber_int_t) instead of
> printf(%ld, (long) ber_int_t).
> Note: This patch merely ignores the two new arguments nnewrdn and
> nnewSuperior to tcl_back_modrdn(), because that's what back-perl does.
> I have no idea what these arguments do. Whoever invented them should
> have a look at tcl_modrdn.c. (And maybe back-perl/modrdn.c too.)