-
Notifications
You must be signed in to change notification settings - Fork 367
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
Is there a plan to switch from wxT("") to wxString::FromUTF8? #796
Comments
I think that wxT should be replaced to wxS, not wxString::FromUTF8. https://docs.wxwidgets.org/3.2.4/group__group__funcmacro__string.html#ga0a06701ea0e6bb00c5171ace1b1f5691 |
I agree, it is possible. Since the documentation recommends not using wxT |
What exactly do you mean by "is everywhere in the code generator"? The wxFB source code itself or the generated code? When i am rewriting wxFB source code, i am removing In generated code you should rarely see this macro, unless you have disabled internationalization which you usually should not do. Although the wxWidgets documentation says you should not use this macro anymore, it also mentions a huge limitation if you do so: you should only use 7 bit characters. While this is not a big issue for the english language, it is for other languages. I encountered many encoding related problems in the past, especially Windows is prone for such issues. Even in these days, the safest way to avoid them is using wide strings. Since |
I only meant code generation. As we wrote above, wxS("") is what wxWidgets recommend in the documentation. |
Hello, wxT("...") is everywhere in the code generator. In version 3.2, they removed it from the documentation on the forum, they wrote that they left it only for backward compatibility and recommend using wxString::FromUTF8 (https://docs.wxwidgets.org/3.2.4/overview_string.html). Is there a replacement planned in wxFormBuilder?
The text was updated successfully, but these errors were encountered: