Skip to content

Commit

Permalink
locale.c: Change U32 to less specific
Browse files Browse the repository at this point in the history
The previous commits have made this mask only ever use the lowest two
bits; so even a U8 is sufficient.
  • Loading branch information
khwilliamson committed Nov 13, 2023
1 parent eb89238 commit 2be5e64
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion embed.fnc
Expand Up @@ -4411,7 +4411,7 @@ S |HV * |my_localeconv |const int item
S |void |populate_hash_from_localeconv \
|NN HV *hv \
|NN const char *locale \
|const U32 which_mask \
|const PERL_UINT_FAST8_T which_mask \
|NN const lconv_offset_t *strings[2] \
|NULLOK const lconv_offset_t *integers
# endif
Expand Down
4 changes: 2 additions & 2 deletions locale.c
Expand Up @@ -5456,7 +5456,7 @@ S_my_localeconv(pTHX_ const int item)
/* This is a mask, with one bit to tell S_populate_hash_from_localeconv to
* populate the NUMERIC items; another bit for the MONETARY ones. This way
* it can choose which (or both) to populate from. */
U32 index_bits = 0;
PERL_UINT_FAST8_T index_bits = 0;

/* This converts from a locale index to its bit position in the above mask.
* */
Expand Down Expand Up @@ -5714,7 +5714,7 @@ S_populate_hash_from_localeconv(pTHX_ HV * hv,

/* bit mask of which categories to
* populate */
const U32 which_mask,
const PERL_UINT_FAST8_T which_mask,

/* strings[0] points to the numeric
* string fields; [1] to the monetary */
Expand Down
2 changes: 1 addition & 1 deletion proto.h

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit 2be5e64

Please sign in to comment.