Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

don't define \noboundary as text command / issue 37 #45

Merged
merged 5 commits into from Sep 29, 2017

Conversation

@u-fischer
Copy link
Contributor

u-fischer commented Sep 29, 2017

Reference #37

This removes the definition of \noboundary as a textcommand. Instead \textEncodingNoboundary is defined, and \noboundary is mappped locally in \pdfstringdef to this new command.

\noboundary is defined in two other files: latex/arabi/puenc-ar.def and
latex/pdfx/l8uenc.def. They should adapt their code too.

The other commands mentioned in the issue are imho not solely hyperrefs problem and so difficult to correct without cooperation of the other packages.

(I messed up the commit history as I forgot to create a branch on the last pull request, sorry ..., but I have no idea if I can repair it without making it more messy)

@@ -1231,6 +1231,9 @@
% There are several commands that prints characters in the
% printable ASCII area that don't obey the NFSS, so they have
% to be redefined here.
% UF 29.09.2017: added a mapping for \noboundary, see issue #37

This comment has been minimized.

Copy link
@davidcarlisle

davidcarlisle Sep 29, 2017

Member

I think there's some tex markup missing here for typeset comments, but I'll take the PR and fix locally

@davidcarlisle davidcarlisle merged commit b312fe2 into latex3:master Sep 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.