Re: ja.tmac doesn't seem to work

"G. Branden Robinson" <[email protected]>
Newsgroups gmane.comp.printing.groff.general
Message-ID <20260121053608.csbg3hor3q27y6mr@illithid>
Hi Kubo-san,

At 2026-01-21T11:26:14+0900, KUBO, Koichi wrote:
> The .class and .cflags requests used by ja.tmac don't seem to work as
> expected.  The steps I've confirmed are as follows:
[...]
> When troff is specified with -mja, it should output \[u3042] ... across
> multiple lines, but it only outputs one line.
[...]
> The results seem to be the same whether or not -mja is specified.
[...]
> The troff version is:
> 
> $ troff -v
> GNU troff (groff) version 1.23.0.4049-1828
[...]
> I think the cause is the following code in void charinfo::get_flags()
> in src/roff/troff/input.cpp:
> 
>     if (ci->contains(get_unicode_mapping() >= 0)) {
> 
> Isn't this code a typo of:
> 
>     if (ci->contains(get_unicode_mapping())) {
> 
> Could you please check this?

Yes.  I introduced this bug on 4 September and fixed it on 1 December.

commit c545d5192dcf7beb643c8ce362fa3b17fcf1d033
Author: G. Branden Robinson <[email protected]>
Date:   Mon Dec 1 17:44:45 2025 -0600

    [troff]: Fix Savannah #67570.

    * src/roff/troff/input.cpp (charinfo::get_flags): Fix inadvertent
      regression caused by fumbled refactoring, wherein I inexplicably
      mistook a Boolean-valued return type for an `int`-valued one.
      (`contains()` sounds Boolean to me now, in the cold light of day.)

    Fixes <https://savannah.gnu.org/bugs/?67570>.  Thanks to Dave Kemper for
    the report.  Problem introduced by me in commit 77d8f728d, 4 September.

Please consider trying the groff 1.24.0.rc1 release candidate.

https://lists.gnu.org/archive/html/info-groff/2026-01/msg00000.html

Regards,
Branden
signature.asc (application/pgp-signature, 833 B)
-----BEGIN PGP SIGNATURE-----

iQIzBAABCAAdFiEEh3PWHWjjDgcrENwa0Z6cfXEmbc4FAmlwZcgACgkQ0Z6cfXEm
bc7KahAAmAnzzYBhQysabxJ0tkuRp2SYqiiyzwDAqygOjtqWmTxBP7Bj0j1gFHGK
kp2MSOyCCPOBJYHzLt3ZeRVZfvQJC31e1r50PWz1IwlN4Lvz6oBR1NYIssKaa2Q/
3kXokJlr/ADpCDIXqlRgEjQW2HCfkQZK5hvf4zjNB90xakPCZLhr2FZlbOusZQTf
qVlc1/P75wKUs2kZikURGA/aBPUqnWopZS632iW63gpPHhTRO/5S17KMOo4tf2O3
adYDXUBKPwjVumNrgPFnl10MYQisQAfLcYyqbJtV0a7idecL1C35Mf5l4dfz5zq7
vq+pvmOqypfmA3pm3OjjNvnxuNb8RrEM6uvQP63/kyjuDKBmen7G31nMWhnE6inp
sA+2KSM52W17h15efH/kFk1TghiUQmKj3vHzR+HUGZcBbeypzEpZMmSNLncBtki4
W9wv4NSrBzZJNc23mo9tkmBbtlEute16drIwi6ACXiZmX7UzceURf1r0wGzwbxiT
6rYk0W6HKuB0sJMUfE/2vy6opwCpoxvDeKJThNuh9jd3Wo2b5021DGymI9X9cyWF
75pvoop8tCDufSta25bWq1oOry9WhhAH0KcecLKdC2e5/tQGEZO0fd1WPaG9kRH8
MqOJcKHBdBIaK9cx+uGQGw1NqWJxcrYgktrW9DiTFkWHpY31WjU=
=U60w
-----END PGP SIGNATURE-----
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.