latex3 / babel

The babel system for LaTeX, LuaLaTeX and XeLaTeX
LaTeX Project Public License v1.3c
123 stars 34 forks source link

A fix for a couple of bugs in hebrew numerals #280

Closed Udi-Fogiel closed 4 months ago

Udi-Fogiel commented 5 months ago

Consider the following:

\documentclass{article}
\usepackage[bidi=basic, hebrew, provide=*]{babel}

\babelfont{rm}{David CLM}

\begin{document}
\localenumeral{letters.gershayim}{12}
\localenumeral{letters.final}{12}
\localenumeral{letters.plain}{12}

\edef\foo{\localenumeral{letters.gershayim}{12}}\show\foo

\edef\foo{\localenumeral{letters.final}{12}}\show\foo

\edef\foo{\localenumeral{letters.plain}{12}}\show\foo
\end{document}

It presents two problems

I was wondering why shouldn't these features be allowed in pdftex. I've noticed that the following (after the commits) produce an error

% !TEX TS-program = pdflatex
\documentclass[a4paper, 12pt]{article}
\usepackage[NHE8]{fontenc}
\usepackage[bidi=default,hebrew, provide=*]{babel}
% \let\gershayim\hebgershayim
\begin{document}
\localenumeral{letters.gershayim}{12}
\localenumeral{letters.final}{12}
\localenumeral{letters.plain}{12}

\edef\foo{\localenumeral{letters.gershayim}{12}}\show\foo

\edef\foo{\localenumeral{letters.final}{12}}\show\foo

\edef\foo{\localenumeral{letters.plain}{12}}\show\foo
\end{document}

this is due a bug in the encodings, I had a typo... It should be fixed in the following couple of days. In the meantime adding the line \let\gershayim\hebgershayim should fix the problem. If that was the reason for not including pdftex, I think it should be changed.

jbezos commented 4 months ago
  • letters.plain does not work, this is because the internal is named letters.long. I assumed that this is a typo and that the documentation is the correct one.

Thanks. Correct, the name is letters.plain.