public inbox for libc-alpha@sourceware.org
 help / color / mirror / Atom feed
* [PATCH] manual: Use Unicode instead HTML entities for characters (bug 19737)
@ 2020-07-15  7:04 Florian Weimer
  2020-07-15 19:17 ` Carlos O'Donell
  0 siblings, 1 reply; 2+ messages in thread
From: Florian Weimer @ 2020-07-15  7:04 UTC (permalink / raw)
  To: libc-alpha

Texinfo no longer treats arguments to @set in @ifhtml blocks as
literal HTML, so the & in the entity references was encoded as
@amp; in HTML.  Using the equivalent Unicode characters avoids
this issue.

---
 manual/math.texi | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/manual/math.texi b/manual/math.texi
index dcff7b7f41..376306cfc5 100644
--- a/manual/math.texi
+++ b/manual/math.texi
@@ -1,9 +1,9 @@
 @c We need some definitions here.
 @ifclear mult
 @ifhtml
-@set mult ·
-@set infty ∞
-@set pie π
+@set mult @U{00B7}
+@set infty @U{221E}
+@set pie @U{03C0}
 @end ifhtml
 @iftex
 @set mult @cdot


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] manual: Use Unicode instead HTML entities for characters (bug 19737)
  2020-07-15  7:04 [PATCH] manual: Use Unicode instead HTML entities for characters (bug 19737) Florian Weimer
@ 2020-07-15 19:17 ` Carlos O'Donell
  0 siblings, 0 replies; 2+ messages in thread
From: Carlos O'Donell @ 2020-07-15 19:17 UTC (permalink / raw)
  To: Florian Weimer, libc-alpha

On 7/15/20 3:04 AM, Florian Weimer via Libc-alpha wrote:
> Texinfo no longer treats arguments to @set in @ifhtml blocks as
> literal HTML, so the & in the entity references was encoded as
> @amp; in HTML.  Using the equivalent Unicode characters avoids
> this issue.

OK for 2.32.

Looks good in pdf and html. Thanks.

Reviewed-by: Carlos O'Donell <carlos@redhat.com>
Tested-by: Carlos O'Donell <carlos@redhat.com>

> ---
>  manual/math.texi | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/manual/math.texi b/manual/math.texi
> index dcff7b7f41..376306cfc5 100644
> --- a/manual/math.texi
> +++ b/manual/math.texi
> @@ -1,9 +1,9 @@
>  @c We need some definitions here.
>  @ifclear mult
>  @ifhtml
> -@set mult &middot;
> -@set infty &infin;
> -@set pie &pi;
> +@set mult @U{00B7}
> +@set infty @U{221E}
> +@set pie @U{03C0}
>  @end ifhtml
>  @iftex
>  @set mult @cdot
> 


-- 
Cheers,
Carlos.


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2020-07-15 19:18 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-07-15  7:04 [PATCH] manual: Use Unicode instead HTML entities for characters (bug 19737) Florian Weimer
2020-07-15 19:17 ` Carlos O'Donell

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).