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

Already on GitHub? Sign in to your account

Additional fix for issue 6905 #1028

Merged
merged 1 commit into from Dec 24, 2012

Conversation

Projects
None yet
3 participants
Member

9rnsr commented Dec 24, 2012

http://d.puremagic.com/issues/show_bug.cgi?id=6905

For the inference of return value ref-ness, we should use 'auto ref' instead of 'ref' or 'const ref'

Required: D-Programming-Language/dmd#1401

Additional fix for issue 6905
For the inference of return value ref-ness, we should use 'auto ref' instead of 'ref' or 'const ref'

jmdavis added a commit that referenced this pull request Dec 24, 2012

Merge pull request #1028 from 9rnsr/fix6905
Additional fix for issue 6905

@jmdavis jmdavis merged commit 03a6e29 into dlang:master Dec 24, 2012

1 check passed

default Pass: 10
Details
Member

jmdavis commented Dec 24, 2012

Merged.

Member

repeatedly commented Dec 24, 2012

Wait.

Kenji describes this pull requires dmd#1401, but dmd#1401 isn't merged yet.

Member

9rnsr commented Dec 24, 2012

No, dmd#1401 requires this, but this does not require that.

Member

repeatedly commented Dec 24, 2012

Oops. Sorry, It's my mistake.
Required, not Requires ;(

Member

jmdavis commented Dec 24, 2012

It passed the autotester before it was merged, or it wouldn't have been merged. And the integration of the autotester with github has definitely made it quicker to check for that. I wasn't expecting that to be as useful as it's turned out to be.

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