Skip to content

Conversation

@IgorStepanov
Copy link

…ation

This is needed for dmd#8378.
The new alias this implemetation fixes 5973 issue and your fix becomes erroneous.
__NewAliasThis is a temporary version which is needed to transition from the old implementation to the new.

If it possible, create a new tag, please.

@CyberShadow
Copy link
Owner

I am OK with a workaround if Walter/Andrei are OK with the DMD PR breaking code. However, I don't think __NewAliasThis is a good idea. It would be better to detect if the compiler supports this functionality or requires the 5973 workaround by detecting it with __traits(complies,...) etc.:

https://github.com/CyberShadow/ae/blob/master/utils/meta/caps.d

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants