[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