Skip to content
This repository has been archived by the owner on Oct 12, 2022. It is now read-only.

move volatileLoad/Store to new module core.volatile_ #1032

Closed
wants to merge 2 commits into from

Conversation

MartinNowak
Copy link
Member

No description provided.

@MartinNowak
Copy link
Member Author

requires dlang/dmd#4155

@WalterBright
Copy link
Member

Don't care for module names with _ in them. Shouldn't open the door to that.

@dnadlinger
Copy link
Member

@WalterBright: The ball has been in your court for almost two months, and now you discover that you don't like this? What else do you propose? Just removing volatile as a keyword now and going straight with core.volatile? Waiting any longer without a decision is not an option.

@JinShil
Copy link
Contributor

JinShil commented Nov 22, 2014

Don't care for module names with _ in them. Shouldn't open the door to that.

With volatile reaching end of life soon, this should be temporary: http://dlang.org/deprecate.html#volatile

@WalterBright
Copy link
Member

What else do you propose?

I propose my original pull request #892 until volatile becomes available. In the meantime, keyword volatile needs to go through the usual deprecation cycle. I know nobody likes bitop but me, but creating a new temporary file&module named volatile_ sets a bad precedent that we'll be stuck with for years because of backwards compatibility.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants