-
Notifications
You must be signed in to change notification settings - Fork 45
syntax: handling of empty strings or trailing white spaces #223
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
Comments
Hmm... these examples (and this syntax) promotes a common I18N mistake: the idea of sentence concatenation working equivalently across languages. This example assumes that the user name always occurs after the rest of the sentence. Promoting this kind of thinking even if we know better than to use it ourselves is a bad thing. Ideally the syntax should discourage it as a behavior at all. After all Note too that @langpavel's choice about spaces has implications for translations of the file. A Japanese translator does not want that (hard to see!) space if it is meaningful. In other words, do this:
... and disallow concatenations. It answers the question at the same time :-) |
This syntax can be helpful in Latin alphabet based languages, but not exclusively. It's translator tool, not coder or programmer. Note that new syntax/operators MAY BE used in translations, not in the application consuming it
No, this have nothing to do with placement
You can always write it as @aphillips proposed.
Correct, but much of redundancy, I like this variant, but this may be not best solution always I guess... will think about. |
@aphillips good point about the idea of sentence concatenation working equivalently across languages I do not expect, that |
The |
Take this as an example:
1. Will be good if can be written as:
2. Will be better if can be written as:
The latest
\-
in*[other]
is trimLike this:
\
EOL → emty string or keep spaces before\
-
EOL → greedy trim (like eat all whitespaces around)\
+
EOL → greedy+one space trim (like eat all whitespaces around, keep one if in middle)The text was updated successfully, but these errors were encountered: