Skip to content
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

back out using std::chrono for timezone stuff #10963

Merged
merged 2 commits into from Aug 7, 2022

Conversation

shuffle2
Copy link
Contributor

@shuffle2 shuffle2 commented Aug 7, 2022

People really like running old OS

fixes https://bugs.dolphin-emu.org/issues/13007

return seconds.get_local_time().time_since_epoch().count();
#else
// TODO Would really, really like to use std::chrono here, but Windows did not support
// std::chrono::current_zone() until 19H1, and other compilers don't even provide support for
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

19H1? Really. Wow..

@JosJuice JosJuice merged commit d69f38f into dolphin-emu:master Aug 7, 2022
11 checks passed
@shuffle2 shuffle2 deleted the notime branch August 8, 2022 00:38
@dezraj
Copy link

dezraj commented Dec 21, 2022

Dear developers, I am the author of that bug report. Recently I was forced to update my LTSC 1809 installation to the LTSC 21H2 version in order to keep using WhatsApp for desktop for working purposes, so now you could reimplement this feature if you wanted.

Also, I checked my System32 folder and this version of Windows LTSC does come with the icu.dll file. So I am at your disposal if you want me to test this feature again in this version.

Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
4 participants