fix Issue 6652 - 2. Deprecate modifying non ref variable. #1009

Merged
merged 1 commit into from Jan 24, 2013

Conversation

Projects
None yet
2 participants
Member

9rnsr commented Jun 15, 2012

This pull is a part of phases for deprecating/changing code behavior. See my comment in bugzilla.

Member

9rnsr commented Jun 20, 2012

Also contains a fix for the key in foreach(i, e; array).

9rnsr referenced this pull request Sep 25, 2012

Closed

Bug6652 #377

Member

MartinNowak commented Jan 17, 2013

Looks like you need to add -de to REQUIRED_ARGS nowadays.
As the last stage (#1009) is 6 month old I'd like to merge this soon.

Member

9rnsr commented Jan 18, 2013

I updated test cases. Thanks for your notice.

  • Add -de to make deprecated features compilation error.
  • Merge four test cases into two, by using PERMUTE_ARGS: -w.

@MartinNowak MartinNowak added a commit that referenced this pull request Jan 24, 2013

@MartinNowak MartinNowak Merge pull request #1009 from 9rnsr/fix6652_2
fix Issue 6652 - 2. Deprecate modifying non ref variable.
ae95b0f

@MartinNowak MartinNowak merged commit ae95b0f into dlang:master Jan 24, 2013

1 check was pending

default Pass: 8, Pending: 2
Details
Member

9rnsr commented Jan 25, 2013

Thanks!

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