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
x/text/message: package level docs about MatchLanguage are unclear #27641
Comments
CC @mpvl |
If I understand your example correctly, you are attempting to match using the language matcher from the default catalog. However, you haven't added any strings (and thus no languages) to the catalog. Something like this works, so I think this is expected behavior (and just an unclear example in the docs):
I have relabeled this as a documentation issue (unless of course @mpvl says otherwise). |
Ah, OK, that makes sense. It's just not very well explained in the documentation, hence I got the wrong impression. But that means basically what I want to do in my scenario is call |
@svip see also this CL https://golang.org/cl/127598 in which this was brought up which I just remembered. I'm trying to think of a better way to clarify this in the docs than using a language tag directly (which, as Marcel points out, is probably rare). |
Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (
go version
)?go1.11 linux/amd64
Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (
go env
)?linux/amd64
What did you do?
I tried to use
message.MatchLanguage("nl")
to obtainlanguage.Dutch
, as per the example on its site.See the following code for an example:
What did you expect to see?
I expected to receive
language.Dutch
, so that when I calledp.Printf("%.2f\n", 5000.00)
, I would get "5.000,00", as Dutch uses "." for thousand separators and "," for decimal separators.What did you see instead?
Instead
message.MatchLanguage("nl")
returned theund Tag
rather than theDutch Tag
. And therefore when I calledp.Printf()
as per the example on the package's description, I got "5,000.00", which ironically matches the example in the description, but the example is wrong.I also tried the third example in opening example, which also did not seem to produce the expected result.
It seems to me that
message.MatchLanguage()
either does not work or does not work as described.Note: Making a new
Printer
withlanguage.Dutch
works as expected.The text was updated successfully, but these errors were encountered: