-
Notifications
You must be signed in to change notification settings - Fork 249
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
Replace capitialize() with capitalize(Locale.ROOT) #340
Comments
Closed
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the bug
This is a suggestion given by android studio to use capitalize(Locale.ROOT)
Implicitly using the default locale is a common source of bugs: Use capitalize(Locale) instead. For strings meant to be internal use Locale.ROOT, otherwise Locale.getDefault().
Component
SDC gallery app
To Reproduce
Steps to reproduce the behavior:
ResourceMapper.kt
line 104
Expected behavior
Not as such behavior change.
Screenshots
N/A
Smartphone (please complete the following information):
Additional context
If this is a valid issue, I can send PR.
The text was updated successfully, but these errors were encountered: