10
66
Fork
You've already forked fcft
14

setlocale isues #60

Closed
opened 2023年08月16日 00:30:05 +02:00 by kpckpc · 0 comments

There is a check for the locale in 'base_pattern_from_name'. The 'CHANGES.md' mentions the cause of not setting/restoring is thread safety of setlocale. Just wondering can this not be fixed by using the threadsafe 'uselocale' on linux and mac?

The 'CHANGES.md' file mentions that the user should set it. At the vary least the thread safety aspect and proper use should be mentioned in the manpage of 'fcft_from_name'.

Especially since he use of harfbuzz will be more international audience, proper use of setting/restoring the locale should be shown in the example code of the 'fcft_from_name' manpage and the 'example/main.c' code, this to prevent the code only working in English-speaking countries and then suddenly strange errors elsewhere in the world, also considering the guarding 'assert' might disappear with -DNDEBUG.

There is a check for the locale in 'base_pattern_from_name'. The 'CHANGES.md' mentions the cause of not setting/restoring is thread safety of setlocale. Just wondering can this not be fixed by using the threadsafe 'uselocale' on linux and mac? The 'CHANGES.md' file mentions that the user should set it. At the vary least the thread safety aspect and proper use should be mentioned in the manpage of 'fcft_from_name'. Especially since he use of harfbuzz will be more international audience, proper use of setting/restoring the locale should be shown in the example code of the 'fcft_from_name' manpage and the 'example/main.c' code, this to prevent the code only working in English-speaking countries and then suddenly strange errors elsewhere in the world, also considering the guarding 'assert' might disappear with -DNDEBUG.
Sign in to join this conversation.
No Branch/Tag specified
master
colrv1
releases/3.3
rasterize-utf8
spacing
releases/3.2
releases/3.1
releases/3.0
releases/2.5
releases/2.4
releases/2.3
releases/2.2
releases/2.1
releases/2.0
releases/1.0
3.3.3
3.3.2
3.3.1
3.3.0
3.2.0
3.1.10
3.1.9
3.1.8
3.1.7
3.1.6
3.1.5
3.1.4
3.1.3
3.1.2
3.1.1
3.1.0
3.0.1
3.0.0
2.5.1
2.5.0
2.4.6
2.4.5
2.4.4
2.4.3
2.4.2
2.4.1
2.4.0
2.3.3
2.3.2
2.3.1
2.3.0
2.2.92
2.2.91
2.2.90
2.2.7
2.2.6
2.2.5
2.2.4
2.2.3
2.2.2
2.2.1
2.2.0
2.1.3
2.1.2
2.1.1
2.1.0
2.0.0
1.1.7
1.1.6
1.1.5
1.1.4
1.1.3
1.1.2
1.1.1
1.1.0
1.0.1
1.0.0
0.4.3
0.4.2
0.4.1
0.4.0
0.3.5
0.3.4
0.3.3
0.3.2
0.3.1
0.3.0
0.2.0
0.1.1
0.1.0
0.0.0
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
dnkl/fcft#60
Reference in a new issue
dnkl/fcft
No description provided.
Delete branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?