[latex3-commits] [git/LaTeX3-latex3-hyperref] testnewkeyval: handle a few more keys (46d38d3)

[Ulrike Fischer]

Repository : https://github.com/latex3/hyperref
On branch  : testnewkeyval
Link       : https://github.com/latex3/hyperref/commit/46d38d3f52ff40a37ae2f9886b30e135d393565a

>---------------------------------------------------------------

commit 46d38d3f52ff40a37ae2f9886b30e135d393565a
Author: Ulrike Fischer <fischer at troubleshooting-tex.de>
Date:   Sat Jul 2 17:01:31 2022 +0200

    handle a few more keys


>---------------------------------------------------------------

46d38d3f52ff40a37ae2f9886b30e135d393565a
 hyperref.dtx | 24 +++++++++++++++++++++++-
 1 file changed, 23 insertions(+), 1 deletion(-)

diff --git a/hyperref.dtx b/hyperref.dtx
index c8ddbdc..75395e5 100644
--- a/hyperref.dtx
+++ b/hyperref.dtx
@@ -4775,7 +4775,11 @@
     raiselinks .legacy_if_set:n = {Hy at raiselinks}
   }
 %    \end{macrocode}
-% Most PDF-creating drivers do not allow links to be broken
+% Most PDF creating drivers nowadays allow links to be broken.
+% and the boolean |\Hy at breaklinks| is actually only used
+% for hdviwind.def and pdfmark.def
+% |\Hy at setbreaklinks| is a rather convoluted method to figure out if the key has
+% been set explicitly to be able to issue a warning if the driver is loaded.
 %    \begin{macrocode}
 \def\Hy at setbreaklinks#1{%
   \csname breaklinks#1\endcsname
@@ -4800,6 +4804,14 @@
   \let\Hy at setbreaklinks\@gobble
 }
 %    \end{macrocode}
+% As the dvips driver checks the value we set the usage to `load'.%
+%    \begin{macrocode}
+\DeclareKeys
+ {
+   ,breaklinks .legacy_if_set:n {Hy at breaklinks}
+   ,breaklinks .usage:n = load
+ }
+%    \end{macrocode}
 %    \begin{macrocode}
 \define at key{Hyp}{localanchorname}[true]{%
   \Hy at WarningNoLine{%
@@ -4808,6 +4820,16 @@
   \Hy at boolkey{localanchorname}{#1}%
 }
 %    \end{macrocode}
+% This key is deprecated, local anchor names are problematic.
+% TODO: add message.
+%    \begin{macrocode}
+\DeclareKeys
+ {
+   ,localanchorname .legacy_if_set:n {Hy at localanchorname}
+   ,localanchorname .usage:n = load
+ }
+%    \end{macrocode}
+%
 % Determines whether an automatic anchor is put on each page
 %    \begin{macrocode}
 \define at key{Hyp}{pageanchor}[true]{%