diff options
author | Frediano Ziglio <freddy77@gmail.com> | 2020-04-28 10:35:04 +0100 |
---|---|---|
committer | Frediano Ziglio <freddy77@gmail.com> | 2020-05-06 13:36:49 +0100 |
commit | 6a80b4b050e8e19412368890aef4daf93be818a6 (patch) | |
tree | b0ecc4e053db009700f74e59dd4e12368c4facec | |
parent | 37445f47843fadcb718b3a5604e3cfb6889d3b1f (diff) |
Reduce indentation in grow_changed_tiles
Signed-off-by: Frediano Ziglio <freddy77@gmail.com>
-rw-r--r-- | src/scan.c | 42 |
1 files changed, 22 insertions, 20 deletions
@@ -210,26 +210,28 @@ static void grow_changed_tiles(scanner_t *scanner G_GNUC_UNUSED, } for (j = 0; j < NUM_HORIZONTAL_TILES; j++) { - if (!tiles_changed[i][j]) { - int grow = 0; - - /* You get good optimizations from having multiple rows, - so be more aggressive in growing the first and last tile; - just require a neighbor be set */ - if (j == 0 && tiles_changed[i][1]) - grow++; - else if (j == NUM_HORIZONTAL_TILES - 1 && tiles_changed[i][j - 1]) - grow++; - - /* Otherwise, require that growing 'fills' a gap */ - else if (j > 0 && j < (NUM_HORIZONTAL_TILES - 1) && - tiles_changed[i][j - 1] && tiles_changed[i][j + 1]) - grow++; - - if (grow) { - tiles_changed[i][j] = true; - tiles_changed_in_row[i]++; - } + int grow = 0; + + if (tiles_changed[i][j]) { + continue; + } + + /* You get good optimizations from having multiple rows, + so be more aggressive in growing the first and last tile; + just require a neighbor be set */ + if (j == 0 && tiles_changed[i][1]) + grow++; + else if (j == NUM_HORIZONTAL_TILES - 1 && tiles_changed[i][j - 1]) + grow++; + + /* Otherwise, require that growing 'fills' a gap */ + else if (j > 0 && j < (NUM_HORIZONTAL_TILES - 1) && + tiles_changed[i][j - 1] && tiles_changed[i][j + 1]) + grow++; + + if (grow) { + tiles_changed[i][j] = true; + tiles_changed_in_row[i]++; } } |