On Wed, Jul 23, 2014 at 11:45 PM, Nathan Sidwell <nat...@acm.org> wrote:
> This second patch cleans up some more global names.  the three functions
> should not be visible to the user.  I'm not sure if they're attempting to
> export an interface to the user but they're (a) undocumented and (b) don't
> follow existing gcov naming.
>
> I'm a little confused as to why the gcov library was split into
> libgcov-driver and libgcov-interface, as each subsection's protected by an
> appropriate L_<SYMBOL> #ifdef, this appears to serve no purpose.
>

libgcov-interfaces.c define public APIs which user can invoke
programmatically in their program. One example use case is through
http handlers for server programs.  On the other hand,
liggcov-driver.c defines internal functions and actual implementations

Before the refactoring, libgcov.c was huge and hard to read/maintain.

> I have another cleanup coming, and then I'll consider David's patch -- I can
> see how to do what it wants without adding another interface (hurrah!)

Ok. For cleanup code, Teresa can help with review.

thanks,

David
>
> nathan

Reply via email to