-
Notifications
You must be signed in to change notification settings - Fork 179
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
Remove compilation constant FEATURE_THREAD_CURRENTCULTURE #1802
Remove compilation constant FEATURE_THREAD_CURRENTCULTURE #1802
Conversation
#else | ||
CultureInfo.CurrentCulture = this.originalCulture; | ||
CultureInfo.CurrentUICulture = this.originalUiCulture; | ||
#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.
Any specific reason to keep Thread.CurrentThread.CurrentCulture
instead of CultureInfo.CurrentCulture
? Both work, but CultureInfo.CurrentCulture
has always felt more natural to me.
Doesn't really matter though; if you prefer to keep it this way, I'll merge as is.
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.
Swapped!
I... Did not think. Just removed branch with constant. Will rework.
…On Sat, Nov 21, 2020, 08:50 Thomas Levesque, ***@***.***> wrote:
***@***.**** commented on this pull request.
------------------------------
In tests/FakeItEasy.Tests.TestHelpers/UsingCultureAttribute.cs
<#1802 (comment)>
:
> Thread.CurrentThread.CurrentCulture = this.originalCulture;
Thread.CurrentThread.CurrentUICulture = this.originalUiCulture;
-#else
- CultureInfo.CurrentCulture = this.originalCulture;
- CultureInfo.CurrentUICulture = this.originalUiCulture;
-#endif
Any specific reason to keep Thread.CurrentThread.CurrentCulture instead
of CultureInfo.CurrentCulture? Both work, but CultureInfo.CurrentCulture
has always felt more natural to me.
Doesn't really matter though; if you prefer to keep it this way, I'll
merge as is.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#1802 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAY7YFICHUPREBWOCNKXF5LSQ7ATZANCNFSM4T5YZDFA>
.
|
36d2322
to
b6a9313
Compare
Flipped. I just amended, rather than fixupping. That would've added confusion, I think. |
Thanks @blairconrad! |
Thank you. I like your version better too. |
Thanks for
|
This change has been released as part of FakeItEasy 7.0.0-beta.1. |
This change has also been released as part of FakeItEasy 7.0.0. |
Continues #1801 a bit, as suggested by @thomaslevesque in #1801 (comment)