On 27/11/2015 03:48, Peter Xu wrote:
> +##
> +# @DUMP_COMPLETED
> +#
> +# Emitted when background dump has completed
> +#
> +# Since: 2.6
> +##
> +{ 'event': 'DUMP_COMPLETED' ,
> +  'data': { 'msg': 'str' } }

This makes it hard to understand whether there was an error or not.  I
suggest using '*error': 'str' instead, and omitting the error if the
dump is successful.

Paolo

Reply via email to