You are viewing a plain text version of this content. The canonical link for it is here.
Posted to users@subversion.apache.org by David Binderman <dc...@hotmail.com> on 2019/01/12 15:32:53 UTC

subversion-1.11.1/subversion/libsvn_client/upgrade.c:353: poor error checking ?

Hello there,

subversion-1.11.1/subversion/libsvn_client/upgrade.c:353] -> [subversion-1.11.1/subversion/libsvn_client/upgrade.c:358]: (style) The if condition is the same as the previous if condition

Source code is

      if (!err)
        externals_parent_url = svn_path_url_add_component2(
                                    externals_parent_repos_root_url,
                                    externals_parent_repos_relpath,
                                    iterpool);
      if (!err)

Maybe better code

      if (!err)
        externals_parent_url = svn_path_url_add_component2(
                                    externals_parent_repos_root_url,
                                    externals_parent_repos_relpath,
                                    iterpool);

      if (externals_parent_url)

Regards

David Binderman

Re: subversion-1.11.1/subversion/libsvn_client/upgrade.c:353: poor error checking ?

Posted by Branko Čibej <br...@apache.org>.
On 12.01.2019 16:32, David Binderman wrote:
> Hello there,
>
> subversion-1.11.1/subversion/libsvn_client/upgrade.c:353] -> [subversion-1.11.1/subversion/libsvn_client/upgrade.c:358]: (style) The if condition is the same as the previous if condition
>
> Source code is
>
>       if (!err)
>         externals_parent_url = svn_path_url_add_component2(
>                                     externals_parent_repos_root_url,
>                                     externals_parent_repos_relpath,
>                                     iterpool);
>       if (!err)


Thanks! Fixed on trunk in r1851180.


> Maybe better code
>
>       if (!err)
>         externals_parent_url = svn_path_url_add_component2(
>                                     externals_parent_repos_root_url,
>                                     externals_parent_repos_relpath,
>                                     iterpool);
>
>       if (externals_parent_url)


Looks better but is unfortunately wrong. :)

-- Brane