#7166 closed defect (fixed)
DRASCULA: Misleading indentation in animation.cpp
Reported by: | DrMcCoy | Owned by: | criezy |
---|---|---|---|
Priority: | low | Component: | Engine: Drascula |
Version: | Keywords: | ||
Cc: | Game: | Drascula |
Description
GCC 6's -Wmisleading-indentation flags the if statement in engines/drascula/animation.cpp:130 to be misleadingly indented.
Specifically, this if statement is indented as if it belonged inside the for-statement in line 119. However, due to the lack of curly braces on that for-statement, this if-statement is not within the scope of that for-statement.
Ticket imported from: bugs/7166.
Change History (2)
comment:1 by , 8 years ago
Owner: | set to |
---|---|
Resolution: | → fixed |
Status: | new → closed |
comment:2 by , 8 years ago
Note:
See TracTickets
for help on using tickets.
Thank you. That was an interesting one as this if statement should both belong to the for-statement started in line 119 and be outside of it :P
In essence there was one if statement missing, which caused a slight delay when interrupting the intro animation in the first scene with Igor. This is fixed with commit abb33e5.