[ho-tex] A patch for the doi package (was: (fwd) Feedback for /author/oberdiek?lang=en)

[David Carlisle]

Heiko,
Thanks

OK I'll do that (ctan can you ignore upload I just made)

David

On Sun, 9 Sep 2018 at 15:58, Heiko Oberdiek
<heiko.oberdiek at googlemail.com> wrote:
>
> Hello,
>
> On 2018-09-09 11:59, David Carlisle wrote:
> > I put a possible updated version here
> >
> > https://github.com/ho-tex/doi
> >
> > but I won't upload to CTAN  unless Heiko or Maarten say that's OK...
>
> Two remarks:
>
> * I suggest a minor change by putting the slash to the URL base part:
>
>      \newcommand{\doiurl}{https://doi.org/}
>
>    Then, the complete URL cleanly consists of the base macro and
>    the doi identifier:
>
>      \href{\doiurl#1}
>
>    And the comment in the change log becomes:
>
>      %%     https://doi.org/ URL but may be set to old http://dx.doi.org/
>
> * A bug fix, the code uses the *global* scratch register \toks1
>    instead of the *local* \toks2; the fix:
>
>      \edef\x{\toks2={\noexpand\href{\doiurl/#1}}}%
>      \x
>      \edef\x{\endgroup\doitext\the\toks2 \the\toks0}%
>      \x
>
> Best regards
>    Heiko