author | erik <erik> | 2007-01-24 19:57:51 (UTC) |
---|---|---|
committer | erik <erik> | 2007-01-24 19:57:51 (UTC) |
commit | 48d9219a96096cf44df8ac24413b36d1b718b1d5 (patch) (unidiff) | |
tree | ef49f0d2b778283221ebbb8634f1a59710ddd108 /noncore/games | |
parent | 89e81059e832ff77c2f0ac8b9db12f80eafa03fc (diff) | |
download | opie-48d9219a96096cf44df8ac24413b36d1b718b1d5.zip opie-48d9219a96096cf44df8ac24413b36d1b718b1d5.tar.gz opie-48d9219a96096cf44df8ac24413b36d1b718b1d5.tar.bz2 |
Each file in this commit has the issue where a function can return a
negative result but the result is used in a context that can only be
positive.
-rw-r--r-- | noncore/games/kpacman/score.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/noncore/games/kpacman/score.cpp b/noncore/games/kpacman/score.cpp index 6878b81..737f03b 100644 --- a/noncore/games/kpacman/score.cpp +++ b/noncore/games/kpacman/score.cpp | |||
@@ -236,17 +236,17 @@ void Score::keyPressEvent(QKeyEvent *k) | |||
236 | // killTimers(); | 236 | // killTimers(); |
237 | emit toggleNew(); | 237 | emit toggleNew(); |
238 | end(); | 238 | end(); |
239 | } | 239 | } |
240 | 240 | ||
241 | if (x != cursor.x || y != cursor.y) { | 241 | if (x != cursor.x || y != cursor.y) { |
242 | if (cursor.x != -1) | 242 | if (cursor.x != -1 && lastScore >= 0) |
243 | cursor.chr = hallOfFame[lastScore].name.at(cursor.x-14); | 243 | cursor.chr = hallOfFame[lastScore].name.at(cursor.x-14); |
244 | scrollRepeat = FALSE; | 244 | scrollRepeat = FALSE; |
245 | repaint(rect(x, y*1.25, cursor.chr), FALSE); | 245 | repaint(rect(x, y*1.25, cursor.chr), FALSE); |
246 | } else | 246 | } else if (lastScore >= 0) |
247 | hallOfFame[lastScore].name.at(cursor.x-14) = cursor.chr; | 247 | hallOfFame[lastScore].name.at(cursor.x-14) = cursor.chr; |
248 | 248 | ||
249 | if (key == UpKey || key == Key_Up || key == DownKey || key == Key_Down) | 249 | if (key == UpKey || key == Key_Up || key == DownKey || key == Key_Down) |
250 | scrollRepeat = TRUE; | 250 | scrollRepeat = TRUE; |
251 | else | 251 | else |
252 | repaint(rect(cursor.x, cursor.y*1.25, cursor.chr), FALSE); | 252 | repaint(rect(cursor.x, cursor.y*1.25, cursor.chr), FALSE); |