You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Greg Stein <gs...@lyra.org> on 2003/07/17 18:54:11 UTC

Re: svn commit: rev 6500 - in trunk/subversion: clients/cmdline include libsvn_client

On Thu, Jul 17, 2003 at 01:31:23PM -0500, sussman@tigris.org wrote:
>...
> +++ trunk/subversion/libsvn_client/repos_diff.c	Thu Jul 17 13:31:20 2003
>...
> +  if (err && err->apr_err == SVN_ERR_WC_NOT_LOCKED)
> +    {
> +      /* ### maybe try to stat the local b->wcpath? */      
> +      /* If the file path doesn't exist, then send a 'skipped' notification. */
> +      if (eb->notify_func)
> +        (*eb->notify_func) (eb->notify_baton, b->wcpath,
> +                            svn_wc_notify_skip, svn_node_file, NULL,
> +                            svn_wc_notify_state_missing, prop_state,
> +                            SVN_INVALID_REVNUM);
> +      
> +      return SVN_NO_ERROR;

You didn't clear the error.

>...

> +      err = get_path_access (&adm_access, eb->adm_access, b->wcpath,
> +                             eb->dry_run, b->pool);
> +
> +      if (err && err->apr_err == SVN_ERR_WC_NOT_LOCKED)
> +        {
> +          /* ### maybe try to stat the local b->wcpath? */          
> +          /* If the path doesn't exist, then send a 'skipped' notification. */
> +          if (eb->notify_func)
> +            (*eb->notify_func) (eb->notify_baton, b->wcpath,
> +                                svn_wc_notify_skip, svn_node_dir, NULL,
> +                                svn_wc_notify_state_missing,
> +                                svn_wc_notify_state_missing,
> +                                SVN_INVALID_REVNUM);
> +      
> +          return SVN_NO_ERROR;

Same here.

Cheers,
-g

-- 
Greg Stein, http://www.lyra.org/

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org