MetzNet
/
dfhack
3
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Should use "break" in this construct, not "continue"

develop
Quietust 2012-10-04 20:20:35 -05:00
parent 6b3d85eb0f
commit ddcc2ee90d
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
plugins/fastdwarf.cpp
Unescape Escape View File

@ -61,21 +61,21 @@ DFhackCExport command_result plugin_onupdate ( color_ostream &out )
{ {
// don't do anything if the dwarf isn't going anywhere // don't do anything if the dwarf isn't going anywhere
if (unit->path.dest.x == -30000) if (unit->path.dest.x == -30000)
continue; break;
// skip dwarves that are dragging creatures or being dragged // skip dwarves that are dragging creatures or being dragged
if ((unit->relations.draggee_id != -1) || (unit->relations.dragger_id != -1)) if ((unit->relations.draggee_id != -1) || (unit->relations.dragger_id != -1))
continue; break;
// skip dwarves that are following other units // skip dwarves that are following other units
if (unit->relations.following != 0) if (unit->relations.following != 0)
continue; break;
old_block = Maps::getTileBlock(unit->pos.x, unit->pos.y, unit->pos.z); old_block = Maps::getTileBlock(unit->pos.x, unit->pos.y, unit->pos.z);
new_block = Maps::getTileBlock(unit->path.dest.x, unit->path.dest.y, unit->path.dest.z); new_block = Maps::getTileBlock(unit->path.dest.x, unit->path.dest.y, unit->path.dest.z);
// make sure source and dest map blocks are valid // make sure source and dest map blocks are valid
if (!old_block || !new_block) if (!old_block || !new_block)
continue; break;
// clear appropriate occupancy flags at old tile // clear appropriate occupancy flags at old tile
if (unit->flags1.bits.on_ground) if (unit->flags1.bits.on_ground)