ldb_handler_fold: public or private?

Jelmer Vernooij jelmer at samba.org
Sat Mar 19 09:00:02 MDT 2011


ldb_handler_fold is a convenience function that is used in
source4/lib/ldb/common/attrib_handlers.c. It is currently public, but
not declared in any header files. As far as I can tell it's just a
trivial convenience function that was never intended to be public.

However, as it is public it is technically part of our ABI. 

We can:

 1) make it static (so the symbol is no longer exported)
 2) keep it public and add it to one of the headers
 3) keep it public but add a comment to remove it whenever we get a
chance

Is ldb_handler_fold useful for external users? How harmful would it be
to remove it now, given that it's not in any public header and nobody
should be using it?

Cheers,

Jelmer
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20110319/f936bdc1/attachment.pgp>


More information about the samba-technical mailing list