[Date Prev][Date Next]
[Chronological]
[Thread]
[Top]
ber_bv[r]chr() return type (was: commit: ldap/servers/slapd/overlays valsort.c)
- To: OpenLDAP Commit <openldap-commit2devel@OpenLDAP.org>
- Subject: ber_bv[r]chr() return type (was: commit: ldap/servers/slapd/overlays valsort.c)
- From: Hallvard B Furuseth <h.b.furuseth@usit.uio.no>
- Date: Mon, 14 Nov 2005 15:02:33 +0100
- In-reply-to: <200511141346.jAEDkck1004916@cantor.openldap.org>
- References: <200511141346.jAEDkck1004916@cantor.openldap.org>
Instead of this change (and some others I was about to make),
maybe ber_bvchr() and ber_bvrchr() should be changed to return
char* instead of void*? lutil_memchr() would still return void*.
hallvard@OpenLDAP.org writes:
> valsort.c 1.13 -> 1.14
> Fix void* pointer arithmetic from ber_bvchr()
- end = ber_bvchr( &a->a_vals[i], '}' ) + 1;
+ end = (char *) ber_bvchr( &a->a_vals[i],
--
Hallvard