Skip to content

Comments

fix Issue 12523 - wrong type deduced for foreach ref argument#3424

Merged
9rnsr merged 1 commit intodlang:masterfrom
MartinNowak:fix12523
Apr 6, 2014
Merged

fix Issue 12523 - wrong type deduced for foreach ref argument#3424
9rnsr merged 1 commit intodlang:masterfrom
MartinNowak:fix12523

Conversation

@MartinNowak
Copy link
Member

  • No need to change the storage class because the var type
    already has inferred modifiers.

Issue 12523 – wrong foreach argument type with ref and inout

@MartinNowak MartinNowak restored the fix12523 branch April 5, 2014 10:09
src/statement.c Outdated
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure why setting the storage class is necessary when the var type already had all the modifiers.

- No need to change the storage class because the var type
  already has inferred modifiers.
@9rnsr
Copy link
Contributor

9rnsr commented Apr 6, 2014

LGTM.

@9rnsr
Copy link
Contributor

9rnsr commented Apr 6, 2014

Auto-merge toggled on

9rnsr added a commit that referenced this pull request Apr 6, 2014
fix Issue 12523 - wrong type deduced for foreach ref argument
@9rnsr 9rnsr merged commit 6d10394 into dlang:master Apr 6, 2014
@MartinNowak
Copy link
Member Author

Thanks!

@MartinNowak MartinNowak deleted the fix12523 branch April 6, 2014 12:44
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