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
Use CUSTOM_LOCALEDIR if specified #3108
Conversation
#ifdef STATIC_LOCALEDIR | ||
if (STATIC_LOCALEDIR[0] != '\0') | ||
path_locale = STATIC_LOCALEDIR; | ||
else |
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.
I suggest removing the "else" to fix the indentation on the following code line.
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.
That would break it, the later line would reset path_locale
.
Is this better?
#ifdef STATIC_LOCALEDIR
path_locale = STATIC_LOCALEDIR[0] ? STATIC_LOCALEDIR : getDataPath("locale");
#else
path_locale = getDataPath("locale");
#endif
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.
Yep, that looks better.
Looks good, although I as well haven't tested it. |
80e5d89
to
fb7e8ed
Compare
@sfan5: This is a bug, as |
May be relevant for merging: |
Should fix #1582 (although I haven't tested it).