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

MPR#7048: ocamldoc -latex, don't escape Latin-1 accented vowels #1420

Merged
merged 2 commits into from Oct 9, 2017

Conversation

xavierleroy
Copy link
Contributor

This causes trouble when the doc comments are actually in UTF8 and
the user is providing their own preamble with an UTF8 inputenc.

The default preamble still contains \usepackage[latin1]{inputenc},
so the Latin-1 accents will still display fine.

Original report: https://caml.inria.fr/mantis/view.php?id=7048

This causes trouble when the doc comments are actually in UTF8 and
the user is providing their own preamble with an UTF8 inputenc.

The default preamble still contains \usepackage[latin1]{inputenc},
so the Latin-1 accents will still display fine.
Copy link
Member

@gasche gasche left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is clearly correct.

Copy link
Member

@Octachron Octachron left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It also looks good to me.

Changes Outdated
@@ -349,6 +349,9 @@ Release branch for 4.06:
when no -title has been provided
(Pierre Boutillier)

- MPR#7048: ocamldoc, in -latex mode, don't escape Latin-1 accented vowels
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Utterly minor nitpicking: ç is not a vowel.

@xavierleroy
Copy link
Contributor Author

ç is not a vowel.

Touché! I fixed the Changes file and will now merge. Thanks for the quick feedback.

@xavierleroy xavierleroy merged commit 875ff82 into trunk Oct 9, 2017
@xavierleroy xavierleroy deleted the MPR7048 branch October 9, 2017 17:18
@xavierleroy
Copy link
Contributor Author

Cherry-pick [4.06 6a8c0ec]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants