[m-rev.] for review: pretty-printing sparse bitsets

Julien Fischer jfischer at opturion.com
Mon Sep 22 19:48:11 AEST 2025


On Mon, 22 Sept 2025 at 18:51, Zoltan Somogyi <zoltan.somogyi at runbox.com> wrote:

> But in the process, I found that the last part of the comment says that
> the default formatter map is thread local. For at least the C backend,
> I don't see that being true, since we store the map in a global variable.
> Do you know the story behind that?

The default formatter map was originally stored in a thread-local mutable,
so the documentation was accurate when this module was written.
Ralph changed that in commit ceb64e8c in an attempt to avoid module
ordering problems when using :- initialise directives to populate the default
formatter map.  (I suspect the removal of the thread-local aspect may have
been inadvertent.)

Julien.


More information about the reviews mailing list