Re: [Libguestfs] [PATCH 3/3] src: print contents of structs and struct lists on tracing

On Thu, Feb 25, 2016 at 02:30:48PM +0100, Pino Toscano wrote: > On Thursday 25 February 2016 12:31:30 Richard W.M. Jones wrote: > > On Wed, Feb 24, 2016 at 01:52:04PM +0100, Pino Toscano wrote: > > > It eases the debugging, instead of getting just the name of the struct > > > returned. > > > > The series is fine, so ACK. Definitely improves the trace output :-) > > Thanks -- I got tired of not seeing what was returned in case of > structs. > > > A few things that would be good though: > > > > - Export the functions through the public API. We already have > > functions to copy structs (eg. guestfs_copy_application, > > guestfs_copy_application_list). > > Would be actually worth exporting them for general usage, though? > I thought about this, and I did not find immediate use cases; although > I can change idea if there are enough/good ones. They become part of the API and ABI so we need to be really sure that we want to support the functions forever. But assuming they're well designed and future-proof I don't see why not.