-
Notifications
You must be signed in to change notification settings - Fork 116
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
NormalizerNFKC: rename unify_katakana_g_sounds
to unify_katakana_gu_small_sounds
#1513
Conversation
Would you review this when you have time_ |
Could you use |
9030661
to
1eeaa11
Compare
Sure, I have renamed. |
lib/normalizer.c
Outdated
@@ -1707,7 +1707,7 @@ grn_nfkc_normalize_unify_katakana_di_sound(grn_ctx *ctx, | |||
} | |||
|
|||
static const unsigned char * | |||
grn_nfkc_normalize_unify_katakana_g_sounds(grn_ctx *ctx, | |||
grn_nfkc_normalize_unify_katakana_gu_small_sounds(grn_ctx *ctx, | |||
const unsigned char *start, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Indent...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need our formatter configuration and a lint CI job...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed.
unify_katakana_g_sounds
to unify_katakana_gu_sounds
unify_katakana_g_sounds
to unify_katakana_gu_small_sounds
It's for consistency with
unify_katakana_du_small_sounds
.