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.

Sign up for GitHub

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

Jump to bottom

[REG2.066a] Issue 13180 - AA get returns const(char[]) instead of string #3801

Merged
merged 1 commit into from
Jul 23, 2014
Merged

[REG2.066a] Issue 13180 - AA get returns const(char[]) instead of string #3801

merged 1 commit into from
Jul 23, 2014

Conversation

9rnsr
Copy link
Contributor

@9rnsr 9rnsr commented Jul 23, 2014

https://issues.dlang.org/show_bug.cgi?id=13180

It's a corner case between enhancement 12290 and improved top-const removing which implemented in #3763.

@WalterBright
Copy link
Member

Auto-merge toggled on

WalterBright added a commit that referenced this pull request Jul 23, 2014
@WalterBright
Merge pull request #3801 from 9rnsr/fix13180
a85f5d7 
[REG2.066a] Issue 13180 - AA get returns const(char[]) instead of string
@WalterBright WalterBright merged commit a85f5d7 into dlang:master Jul 23, 2014
@9rnsr
Copy link
Contributor Author

9rnsr commented Jul 23, 2014

Thanks.

9rnsr pushed a commit to 9rnsr/dmd that referenced this pull request Jul 23, 2014
@WalterBright @9rnsr
Merge pull request dlang#3801 from 9rnsr/fix13180
b5befe0 
[REG2.066a] Issue 13180 - AA get returns const(char[]) instead of string
@9rnsr 9rnsr deleted the fix13180 branch July 23, 2014 13:48
@CyberShadow
Copy link
Member

This pull request introduced a regression:
https://issues.dlang.org/show_bug.cgi?id=13294

9rnsr added a commit to 9rnsr/dmd that referenced this pull request Aug 25, 2014
@9rnsr
Revert "Merge pull request dlang#3801 from 9rnsr/fix13180"
a2798ab 
This reverts commit a85f5d7, reversing
changes made to b4335c9.

Conflicts:
	test/runnable/template9.d
@CyberShadow
Copy link
Member

This pull request introduced a regression:
https://issues.dlang.org/show_bug.cgi?id=14886

ibuclaw pushed a commit to ibuclaw/dmd that referenced this pull request Jul 10, 2022
@RazvanN7
Merge pull request dlang#3801 from ljmf00/add-nogc-asserterror
1cb7499 
core/exception: annotate @nogc on AssertError ctors
Sign up for free to join this conversation on GitHub. Already have an account? Sign in 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.
3 participants
@9rnsr @WalterBright @CyberShadow