Page MenuHomeFreeBSD
Differential D53908

libc: Simplify __get_locale()
ClosedPublic
Actions

Authored by des on Nov 25 2025, 7:54 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Aug 19, 7:41 PM
Unknown Object (File)
Thu, Aug 13, 12:17 AM
Unknown Object (File)
Wed, Aug 12, 1:46 AM
Unknown Object (File)
Tue, Aug 11, 12:05 AM
Unknown Object (File)
Mon, Aug 10, 6:59 PM
Unknown Object (File)
Mon, Aug 10, 6:42 PM
Unknown Object (File)
Mon, Aug 10, 12:45 PM
Unknown Object (File)
Sun, Aug 9, 8:53 PM
View All 67 Files
Subscribers
imp

Details

Reviewers
kevans
fuz
Commits
rG8787b699b036: libc: Simplify __get_locale()
rG844f76c2426d: libc: Simplify __get_locale()
rG5af240c54bd2: libc: Simplify __get_locale()
Summary

MFC after: 1 week
Sponsored by: Klara, Inc.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 68842
Build 65725: arc lint + arc unit

Event Timeline

des created this revision.Nov 25 2025, 7:54 PM
Herald added a subscriber: imp. · View Herald TranscriptNov 25 2025, 7:54 PM
des requested review of this revision.Nov 25 2025, 7:54 PM
Harbormaster completed remote builds in B68842: Diff 167095.Nov 25 2025, 7:54 PM
des added a comment.EditedNov 25 2025, 7:55 PM

The only explanation I can think of for why the code was the way it was is either:

  1. It just incrementally got that way and it didn't occur to anybody to rewrite it, or
  2. It actually matters for some deep branch prediction / speculative execution magic reasons

Can anyone shed some light on this?

Note that __has_thread_locale is global while __thread_locale is thread-local.

fuz added a comment.Nov 25 2025, 8:02 PM

Looks reasonable. Should do the same thing as before, literally.

lib/libc/locale/xlocale_private.h
205
fuz accepted this revision.Nov 25 2025, 8:03 PM

LGTM.

This revision is now accepted and ready to land.Nov 25 2025, 8:03 PM
Closed by commit rG5af240c54bd2: libc: Simplify __get_locale() (authored by des). · Explain WhyNov 26 2025, 4:57 PM
This revision was automatically updated to reflect the committed changes.
des added a commit: rG5af240c54bd2: libc: Simplify __get_locale().
des added a commit: rG844f76c2426d: libc: Simplify __get_locale().Dec 5 2025, 3:09 PM
des added a commit: rG8787b699b036: libc: Simplify __get_locale().Dec 5 2025, 6:29 PM

Revision Contents
Changeset List

PathSize
lib/
libc/
locale/
6 lines

Diff 167095

View Options

lib/libc/locale/xlocale_private.h

Loading...