Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Store format_set in a threadsafe way #27

Merged
merged 3 commits into from
Feb 9, 2019
Merged

Store format_set in a threadsafe way #27

merged 3 commits into from
Feb 9, 2019
+52 −15

Conversation

timdiggins
Copy link
Contributor

This allows to safely use Timeliness.use_us_formats and Timeliness.use_euro_formats in threads.

(This is based on @andruby's #23.)

Closes #21 and closes #23.

@timdiggins timdiggins mentioned this pull request Oct 9, 2018
Merged
@adzap adzap merged commit dd8d6b3 into adzap:master Feb 9, 2019
@timdiggins timdiggins deleted the multithread branch February 9, 2019 08:19
@timdiggins timdiggins restored the multithread branch February 9, 2019 08:19
@fsateler fsateler mentioned this pull request Jun 14, 2019
Closed
@timdiggins timdiggins deleted the multithread branch June 15, 2019 06:38
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Mention that Timeliness is not thread safe.
3 participants
@timdiggins @adzap @andruby