← Back to index | Original Bugzilla link

Bug 13679 – foreach_reverse is allowed for AAs

Status: RESOLVED
Resolution: FIXED
Severity: normal
Priority: P1
Component: dmd
Product: D
Version: D2
Platform: All
OS: All
Creation time: 2014-11-04T07:43:00Z
Last change time: 2015-02-18T03:39:13Z
Keywords: pull
Assigned to: nobody
Creator: yebblies
See also: https://issues.dlang.org/show_bug.cgi?id=10009

Comments

Comment #0 by yebblies — 2014-11-04T07:43:09Z

The code below compiles, but foreach_reverse does the same thing as foreach (just like 1553). void main() { int[int] aa; foreach(k, v; aa) {} foreach_reverse(k, v; aa) {} }

Comment #1 by yebblies — 2014-11-04T07:49:03Z

https://github.com/D-Programming-Language/dmd/pull/4110

Comment #2 by bugzilla — 2014-11-05T01:26:57Z

The spec: http://dlang.org/statement.html#ForeachStatement says that: "foreach_reverse for associative arrays is illegal." So this should follow the path of warning/deprecation/error.

Comment #3 by github-bugzilla — 2014-11-10T20:54:35Z

Commits pushed to master at https://github.com/D-Programming-Language/dmd https://github.com/D-Programming-Language/dmd/commit/da85c9745a528a4c521615fc88844a3431db957d Fix Issue 13679 - foreach_reverse is allowed for AAs https://github.com/D-Programming-Language/dmd/commit/7b130d056b7c326c2142cfffeec01e5b1e7afae9 Merge pull request #4110 from yebblies/issue13679 Issue 13679 - foreach_reverse is allowed for AAs

Comment #4 by github-bugzilla — 2015-02-18T03:39:13Z

Commits pushed to 2.067 at https://github.com/D-Programming-Language/dmd https://github.com/D-Programming-Language/dmd/commit/da85c9745a528a4c521615fc88844a3431db957d Fix Issue 13679 - foreach_reverse is allowed for AAs https://github.com/D-Programming-Language/dmd/commit/7b130d056b7c326c2142cfffeec01e5b1e7afae9 Merge pull request #4110 from yebblies/issue13679