From 8b582edb387b4e6559de26d0b5d161975c2c05fb Mon Sep 17 00:00:00 2001 From: alberth Date: Sun, 16 May 2010 18:22:07 +0000 Subject: (svn r19829) -Codechange: Extract assignment from the condition-check in the for statement. --- src/newgrf_gui.cpp | 15 +++++++++------ 1 file changed, 9 insertions(+), 6 deletions(-) (limited to 'src') diff --git a/src/newgrf_gui.cpp b/src/newgrf_gui.cpp index 7c5360ec3..0bbb8deca 100644 --- a/src/newgrf_gui.cpp +++ b/src/newgrf_gui.cpp @@ -715,12 +715,13 @@ struct NewGRFWindow : public Window { break; case SNGRFS_REMOVE: { // Remove GRF - GRFConfig **pc, *c, *newsel; + GRFConfig **pc, *newsel; /* Choose the next GRF file to be the selected file */ newsel = this->sel->next; - for (pc = &this->list; (c = *pc) != NULL; pc = &c->next) { + for (pc = &this->list; *pc != NULL; pc = &(*pc)->next) { + GRFConfig *c = *pc; /* If the new selection is empty (i.e. we're deleting the last item * in the list, pick the file just before the selected file */ if (newsel == NULL && c->next == this->sel) newsel = c; @@ -740,11 +741,12 @@ struct NewGRFWindow : public Window { } case SNGRFS_MOVE_UP: { // Move GRF up - GRFConfig **pc, *c; + GRFConfig **pc; if (this->sel == NULL) break; int pos = 0; - for (pc = &this->list; (c = *pc) != NULL; pc = &c->next, pos++) { + for (pc = &this->list; *pc != NULL; pc = &(*pc)->next, pos++) { + GRFConfig *c = *pc; if (c->next == this->sel) { c->next = this->sel->next; this->sel->next = c; @@ -759,11 +761,12 @@ struct NewGRFWindow : public Window { } case SNGRFS_MOVE_DOWN: { // Move GRF down - GRFConfig **pc, *c; + GRFConfig **pc; if (this->sel == NULL) break; int pos = 1; // Start at 1 as we swap the selected newgrf with the next one - for (pc = &this->list; (c = *pc) != NULL; pc = &c->next, pos++) { + for (pc = &this->list; *pc != NULL; pc = &(*pc)->next, pos++) { + GRFConfig *c = *pc; if (c == this->sel) { *pc = c->next; c->next = c->next->next; -- cgit v1.2.3-70-g09d2