Nice indeed! ```StringWriterDetach()``` makes more sense as a name, which
is already the case for FileWriter.


On Thu, Jan 23, 2014 at 9:05 PM, Sigurd Teigen <notifications@github.com>wrote:

> In libutils/writer.h:
>
> > @@ -54,9 +54,11 @@
> >  void WriterClose(Writer *writer);
> >
> >  /* Returns modifiable string and destroys itself */
> > -char *StringWriterClose(Writer *writer);
> > +char *StringWriterClose(Writer *writer) FUNC_WARN_UNUSED_RESULT;
>
> nice
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/cfengine/core/pull/1333/files#r9127472>
> .
>