Re: [PATCH] wcwidth: adjust manpage input (Re: [PATCH] fix wcwidth to work with gcc 16 sign extension)

--000000000000bb84eb0653e8f48d
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

Patch applied.  Thanks.

-- Jeff J.

On Wed, Jun 10, 2026 at 10:16=E2=80=AFAM Jon Turney <[email protected]=
rg.uk>
wrote:

> On 08/06/2026 22:26, Jeff Johnston wrote:
> > Patch applied.
>   'make info' chokes on this.
>
> >   MAKEINFO ../../../src/newlib/libc/libc.info
> > wcwidth.def:34: misplaced {
> > wcwidth.def:36: misplaced }
> > wcwidth.def:38: misplaced {
> > wcwidth.def:40: misplaced }
> > wcwidth.def:44: misplaced {
> > wcwidth.def:48: misplaced {
> > wcwidth.def:48: misplaced }
> > wcwidth.def:49: misplaced }
>
> I think this is because makedoc does not correctly escape '{' or '}' for
> texinfo, outside of a fixed-width font section (which makedoc expects to
> be marked-up with '.' or '|'at the start of lines -- see courierize in
> makedoc.c)
>
> That's definitely a shortcoming in makedoc, but since this *is* a code
> example, courierize-ing it seems appropriate.
>
> Patch attached.
>

--000000000000bb84eb0653e8f48d
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr"><div class=3D"gmail_default" style=3D"font-family:verdana,=
sans-serif">Patch applied.=C2=A0 Thanks.</div><div class=3D"gmail_default" =
style=3D"font-family:verdana,sans-serif"><br></div><div class=3D"gmail_defa=
ult" style=3D"font-family:verdana,sans-serif">-- Jeff J.</div></div><br><di=
v class=3D"gmail_quote gmail_quote_container"><div dir=3D"ltr" class=3D"gma=
il_attr">On Wed, Jun 10, 2026 at 10:16=E2=80=AFAM Jon Turney &lt;<a href=3D=
"mailto:[email protected]">[email protected]</a>&gt; wr=
ote:<br></div><blockquote class=3D"gmail_quote" style=3D"margin:0px 0px 0px=
 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 08/06/20=
26 22:26, Jeff Johnston wrote:<br>
&gt; Patch applied.<br>
=C2=A0 &#39;make info&#39; chokes on this.<br>
<br>
&gt;=C2=A0 =C2=A0MAKEINFO ../../../src/newlib/libc/<a href=3D"http://libc.i=
nfo" rel=3D"noreferrer" target=3D"_blank">libc.info</a><br>
&gt; wcwidth.def:34: misplaced {<br>
&gt; wcwidth.def:36: misplaced }<br>
&gt; wcwidth.def:38: misplaced {<br>
&gt; wcwidth.def:40: misplaced }<br>
&gt; wcwidth.def:44: misplaced {<br>
&gt; wcwidth.def:48: misplaced {<br>
&gt; wcwidth.def:48: misplaced }<br>
&gt; wcwidth.def:49: misplaced }<br>
<br>
I think this is because makedoc does not correctly escape &#39;{&#39; or &#=
39;}&#39; for <br>
texinfo, outside of a fixed-width font section (which makedoc expects to <b=
r>
be marked-up with &#39;.&#39; or &#39;|&#39;at the start of lines -- see co=
urierize in <br>
makedoc.c)<br>
<br>
That&#39;s definitely a shortcoming in makedoc, but since this *is* a code =
<br>
example, courierize-ing it seems appropriate.<br>
<br>
Patch attached.<br>
</blockquote></div>

--000000000000bb84eb0653e8f48d--