Re: SVN::GetRootAndHead

--0000000000006873670617612047
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

Den m=C3=A5n 29 apr. 2024 kl 19:28 skrev Stefan via TortoiseSVN-dev <
[email protected]>:

> would it work if we do both?
>
>             rev =3D cachedProperties.GetHeadRevision(uuid, path);
>             if ((rev =3D=3D NO_REVISION) || (m_err !=3D nullptr))
>             {
>                 svn_error_clear(m_err);
>                 rev =3D cachedProperties.GetHeadRevision(uuid, url);
>             }
> I don't have a setup ready to test this. At least it would be better than
> before:
> - if the path doesn't exist anymore, we try again with the root
> - if root works, fine
> - if root doesn't work either, it's no worse than before
>

> Stefan
>
>
Agree with you that it's no worse than before. It seems to work in my
testing. I've committed as r29691.

I've also asked on the subversion dev@ mailing list.

Kind regards,
Daniel

--=20
You received this message because you are subscribed to the Google Groups "=
TortoiseSVN-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an e=
mail to [email protected].
To view this discussion on the web visit https://groups.google.com/d/msgid/=
tortoisesvn-dev/CAMHy98OpkNREhjqyd4yfiY8NtM-H0Gye9FqFzc4kJ4f33v7mBg%40mail.=
gmail.com.

--0000000000006873670617612047
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr"><div dir=3D"ltr"><div dir=3D"ltr">Den m=C3=A5n 29 apr. 202=
4 kl 19:28 skrev Stefan via TortoiseSVN-dev &lt;<a href=3D"mailto:tortoises=
[email protected]">[email protected]</a>&gt;:<br></div=
><div class=3D"gmail_quote"><blockquote class=3D"gmail_quote" style=3D"marg=
in:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1e=
x">would it work if we do both?<div><br></div><div>=C2=A0 =C2=A0 =C2=A0 =C2=
=A0 =C2=A0 =C2=A0 rev =3D cachedProperties.GetHeadRevision(uuid, path);<br>=
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 if ((rev =3D=3D NO_REVISION) || (=
m_err !=3D nullptr))<br>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 {<br>=C2=
=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 svn_error_clear(m_err)=
;<br>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 rev =3D cached=
Properties.GetHeadRevision(uuid, url);<br>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=
=A0 =C2=A0 }</div><div>I don&#39;t have a setup ready to test this. At leas=
t it would be better than before:</div><div>- if the path doesn&#39;t exist=
 anymore, we try again with the root</div><div>- if root works, fine</div><=
div>- if root doesn&#39;t work either, it&#39;s no worse than before</div><=
/blockquote><blockquote class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0=
.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><br></di=
v><div>Stefan</div>

<p></p>

</blockquote><div><br></div><div>Agree with you that it&#39;s no worse than=
 before. It seems to work in my testing. I&#39;ve committed as r29691.</div=
><div><br></div><div>I&#39;ve also asked on the subversion dev@ mailing lis=
t.</div><div><br></div><div>Kind regards,</div><div>Daniel</div><div><br></=
div></div></div></div>

<p></p>

-- <br />
You received this message because you are subscribed to the Google Groups &=
quot;TortoiseSVN-dev&quot; group.<br />
To unsubscribe from this group and stop receiving emails from it, send an e=
mail to <a href=3D"mailto:[email protected]">tor=
[email protected]</a>.<br />
To view this discussion on the web visit <a href=3D"https://groups.google.c=
om/d/msgid/tortoisesvn-dev/CAMHy98OpkNREhjqyd4yfiY8NtM-H0Gye9FqFzc4kJ4f33v7=
mBg%40mail.gmail.com?utm_medium=3Demail&utm_source=3Dfooter">https://groups=
.google.com/d/msgid/tortoisesvn-dev/CAMHy98OpkNREhjqyd4yfiY8NtM-H0Gye9FqFzc=
4kJ4f33v7mBg%40mail.gmail.com</a>.<br />

--0000000000006873670617612047--