Re: [Xen-devel] [PATCH RFC V2 05/10] libxl_json: introduce parser functions for builtin types

[Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>]

Ian Campbell writes ("Re: [PATCH RFC V2 05/10] libxl_json: introduce parser 
functions for builtin types"):
> Hrm, I'm not sure what we expect the caller to do with the output of a
> failed operation. I think I would expect that the failing operation
> would clean up any partial result. Ian?

I agree.  I think that unless there's a compelling reason to do
otherwise, a failing operation should not try to pass ownership of
anything to the caller.  It should, however, explictly set the result
variable to 0 so that the caller can safely free it.

Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel