diff options
author | Jesse Luehrs <doy@tozt.net> | 2014-08-02 20:06:34 -0400 |
---|---|---|
committer | Jesse Luehrs <doy@tozt.net> | 2014-08-02 20:10:40 -0400 |
commit | 4e1b44a4905cccd1a8509a91f9d10d7a06f956e7 (patch) | |
tree | 62b76b8e00325f1f7a96c646f10b5a5fe62919e6 /crawl-ref/source/mon-place.cc | |
parent | 756871af9b64cee576375e9ed5f73cf3cfa1b453 (diff) | |
download | crawl-ref-4e1b44a4905cccd1a8509a91f9d10d7a06f956e7.tar.gz crawl-ref-4e1b44a4905cccd1a8509a91f9d10d7a06f956e7.zip |
allow iterating over branches in a non-enum order (8742)
This allows us to have a consistent and logical ordering of branches
without requiring the branch enum itself to be reordered (which could
have save compatibility implications). The new ordering of branches just
moves Depths to the place in the ordering that it already is planned to
go on the next major save compat bump, but other changes are possible,
if desired. All places in the code that iterate over branches have been
updated to use the new iterator except for code dealing with save files,
which still uses enum order, so that we can change the display ordering
without affecting saves.
Diffstat (limited to 'crawl-ref/source/mon-place.cc')
-rw-r--r-- | crawl-ref/source/mon-place.cc | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/crawl-ref/source/mon-place.cc b/crawl-ref/source/mon-place.cc index 5acede199e..d94a0a5776 100644 --- a/crawl-ref/source/mon-place.cc +++ b/crawl-ref/source/mon-place.cc @@ -538,16 +538,16 @@ static bool _find_mon_place_near_stairs(coord_def& pos, } // Is it a branch stair? - for (int i = 0; i < NUM_BRANCHES; ++i) + for (branch_iterator it; it; ++it) { - if (branches[i].entry_stairs == feat) + if (it->entry_stairs == feat) { - place = static_cast<branch_type>(i); + place = it->id; break; } - else if (branches[i].exit_stairs == feat) + else if (it->exit_stairs == feat) { - place = brentry[i]; + place = brentry[it->id]; // This can happen on D:1 and in wizmode with random spawns on the // first floor of a branch that didn't generate naturally. if (!place.is_valid()) |