2 patches for repository http://darcs.net/screened:

patch a628ba1d4b060c15a153f1de645778b74fe786fe
Author: Ben Franksen <ben.franksen@online.de>
Date:   Wed Apr 27 11:16:59 CEST 2022
  * simplify getDefaultRepo

patch 6362f3254af6238e8b235705dac02315f398a2f5
Author: Ben Franksen <ben.franksen@online.de>
Date:   Wed Apr 27 11:17:36 CEST 2022
  * simplify addRepoSource by using getDefaultRepo

Attachments

• patch-preview.txt
• simplify-getdefaultrepo.dpatch

>   * simplify getDefaultRepo

Why doesn't this call `fixRepoPath` any more?

>  * simplify addRepoSource by using getDefaultRepo

This looks good if the removal of the call to `fixRepoPath` above is
ok.

I think it's okay. Whatever gets written to _darcs/prefs/defaultrepo 
should already be "fixed" i.e. converted to an absolute path unless 
it's a URL. The only place where we write that file is in 
addRepoSource. The value is either its first argument (the local 
repodir) or (for --inherit-default) the defaultrepo of the remote. I 
just checked that in all cases addRepoSource gets the repodir in 
"fixed" form (using fixUrl). BTW, the remaining place where 
fixRepoPath is called is redundant, too.

This could be improved by replacing String with AbsoluteOrRemotePath 
which has this invariant baked in. I have a few improvements for 
D.R.Prefs in my working branch and would like to send them first 
before making such a change.

OK, thanks.