The unused function list

[Andrew Bartlett]

On Fri, 2012-09-07 at 08:16 +0200, Volker Lendecke wrote:
> On Fri, Sep 07, 2012 at 04:03:51PM +1000, Andrew Bartlett wrote:
> > On Fri, 2012-09-07 at 07:52 +0200, Volker Lendecke wrote:
> > > On Fri, Sep 07, 2012 at 10:36:27AM +1000, Andrew Bartlett wrote:
> > > > Just a reminder that we generate a list of unused functions at:
> > > > 
> > > > https://build.samba.org/lcov/data/coverage/samba_4_0_test/unused-fns.txt
> > > > 
> > > > Some of these functions are from heimdal, pidl or complete an API that
> > > > isn't fully used, but there are others that are just unused, and could
> > > > be removed.  (And of course unused code is very much untested code).
> > > > 
> > > > (This list, and the code coverage functions were out of date for a
> > > > while, but are now generated on a twice daily basis).
> > > 
> > > This list can't be right. For example ctdb_packet_flush is
> > > definitely used. It is a bad API, but in the current code it
> > > is part of the basic code connecting to ctdb. So please do
> > > not remove it and its users, this would completely break our
> > > cluster offering.
> > 
> > On the contrary, this is a genuine bug!  source3/lib/ctdb_packet.c is
> 
> What is the functionality that is broken by this "genuine
> bug"? Is a few kb too large a binary a "genuine bug" these
> days?

I don't mind if this code is compiled or not.  I'll I'm indicating in
response to your suggestion above is that the tool correctly indicates
that in the default waf build, these functions are unused. 

Andrew Bartlett

-- 
Andrew Bartlett                                http://samba.org/~abartlet/
Authentication Developer, Samba Team           http://samba.org