s...@apache.org wrote on Thu, Sep 09, 2010 at 11:31:13 -0000:
> +  return svn_error_return(
> +           svn_error_create(SVN_ERR_FS_CORRUPT, NULL,
> +                            _("Malformed representation header")));

svn_error_return() isn't necessary here, since svn_error_create()
has always recorded the file-line coordinates.

That's unlike, say, svn_error_compose_create(), which doesn't record the
file-line coordinates; thus,

    err1 = foo();
    err2 = bar();
    return svn_error_return(svn_error_compose_create(err1, err2));

isn't redundant.

Reply via email to