openttd-mini: comparison window.c

equal deleted inserted replaced

-:0811adaec525
+:eeaefdabfdfd
 {
 	Window *w;
 	DrawPixelInfo bk;
 	_cur_dpi = &bk;
-	for(w=_windows; w!=_last_window; w++) {
+	for (w = _windows; w != _last_window; w++) {
 		if (right > w->left &&
 				bottom > w->top &&
 				left < w->left + w->width &&
 				top < w->top + w->height) {
-				DrawOverlappedWindow(w, left, top, right, bottom);
+			DrawOverlappedWindow(w, left, top, right, bottom);
-			}
+		}
 	}
 }
 void DrawOverlappedWindow(Window *w, int left, int top, int right, int bottom)
 {
 	while (++v != _last_window) {
 		if (right > v->left &&
 				bottom > v->top &&
 				left < v->left + v->width &&
 				top < v->top + v->height) {
 			if (left < (x=v->left)) {
 				DrawOverlappedWindow(w, left, top, x, bottom);
 				DrawOverlappedWindow(w, x, top, right, bottom);
 				return;
 			}
 	WindowNumber wn;
 	ViewPort *vp;
 	Window *v;
 	int count;
-	if (w == NULL)
+	if (w == NULL) return;
-		return;
 	if (_thd.place_mode != 0 && _thd.window_class == w->window_class && _thd.window_number == w->window_number) {
 		ResetObjectToPlace();
 	}
 Window *FindWindowById(WindowClass cls, WindowNumber number)
 {
 	Window *w;
-	for(w=_windows; w!=_last_window; w++) {
+	for (w = _windows; w != _last_window; w++) {
-		if (w->window_class == cls &&
+		if (w->window_class == cls && w->window_number == number) return w;
-			  w->window_number == number) {
-			return w;
-		}
 	}
 	return NULL;
 }
 }
 void DeleteWindowByClass(WindowClass cls)
 {
 	Window *w;
 	for (w = _windows; w != _last_window;) {
 		if (w->window_class == cls) {
 			DeleteWindow(w);
 			w = _windows;
-		} else
+		} else {
 			w++;
+		}
 	}
 }
 Window *BringWindowToFrontById(WindowClass cls, WindowNumber number)
 {
 	Window *v;
 	Window temp;
 	v = _last_window;
 	do {
-		if (--v < _windows)
+		if (--v < _windows) return w;
-			return w;
 	} while (IsVitalWindow(v));
-	if (w == v)
+	if (w == v) return w;
-		return w;
 	assert(w < v);
 	temp = *w;
 	memmove(w, w + 1, (v - w) * sizeof(Window));
 * @return w pointer to the window that is going to be deleted
 */
 static Window *FindDeletableWindow(void)
 {
 	Window *w;
 	for (w = _windows; w < endof(_windows); w++) {
-		if (w->window_class != WC_MAIN_WINDOW && !IsVitalWindow(w) && !(w->flags4 & WF_STICKY) )
+		if (w->window_class != WC_MAIN_WINDOW && !IsVitalWindow(w) && !(w->flags4 & WF_STICKY)) {
-				return w;
+			return w;
+		}
 	}
 	return NULL;
 }
 /** A window must be freed, and all are marked as important windows. Ease the
 * @return w Pointer to the window that is being deleted
 */
 static Window *ForceFindDeletableWindow(void)
 {
 	Window *w;
 	for (w = _windows;; w++) {
 		assert(w < _last_window);
+		if (w->window_class != WC_MAIN_WINDOW && !IsVitalWindow(w)) return w;
-		if (w->window_class != WC_MAIN_WINDOW && !IsVitalWindow(w))
-				return w;
 	}
 }
 bool IsWindowOfPrototype(const Window* w, const Widget* widget)
 {
 void AssignWidgetToWindow(Window *w, const Widget *widget)
 {
 	w->original_widget = widget;
 	if (widget != NULL) {
-		const Widget *wi = widget;
 		uint index = 1;
-		while (wi->type != WWT_LAST) {
+		const Widget* wi;
-			wi++;
-			index++;
+		for (wi = widget; wi->type != WWT_LAST; wi++) index++;
-		}
+		w->widget = realloc(w->widget, sizeof(*w->widget) * index);
-		w->widget = realloc(w->widget, sizeof(Widget) * index);
+		memcpy(w->widget, widget, sizeof(*w->widget) * index);
-		memcpy(w->widget, widget, sizeof(Widget) * index);
+	} else {
-	} else
 		w->widget = NULL;
+	}
 }
 /** Open a new window. If there is no space for a new window, close an open
 * window. Try to avoid stickied windows, but if there is no else, close one of
 * those as well. Then make sure all created windows are below some always-on-top
 	// We have run out of windows, close one and use that as the place for our new one
 	if (w >= endof(_windows)) {
 		w = FindDeletableWindow();
-		if (w == NULL) // no window found, force it!
+		if (w == NULL) w = ForceFindDeletableWindow();
-			w = ForceFindDeletableWindow();
 		DeleteWindow(w);
 		w = _last_window;
 	}
 	if (w == NULL || w->left >= (_screen.width-20) || w->left <= -60 || w->top >= (_screen.height-20)) {
 		return AllocateWindowAutoPlace(width,height,proc,cls,widget);
 	}
 	x = w->left;
-	if (x > _screen.width - width)
+	if (x > _screen.width - width) x = _screen.width - width - 20;
-		x = (_screen.width - width) - 20;
+	return AllocateWindow(x + 10, w->top + 10, width, height, proc, cls, widget);
-	return AllocateWindow(x+10,w->top+10,width,height,proc,cls,widget);
 }
 typedef struct SizeRect {
 	int left,top,width,height;
 	if (left < 0 || top < 22 || right > _screen.width || bottom > _screen.height)
 		return false;
 	// Make sure it is not obscured by any window.
-	for(w=_windows; w!=_last_window; w++) {
+	for (w = _windows; w != _last_window; w++) {
-		if (w->window_class == WC_MAIN_WINDOW)
+		if (w->window_class == WC_MAIN_WINDOW) continue;
-			continue;
 		if (right > w->left &&
-		    w->left + w->width > left &&
+				w->left + w->width > left &&
 				bottom > w->top &&
-				w->top + w->height > top)
+				w->top + w->height > top) {
-					return false;
+			return false;
+		}
 	}
 	return true;
 }
 		return false;
 	if (top < 22 || top > _screen.height - (height>>2))
 		return false;
 	// Make sure it is not obscured by any window.
-	for(w=_windows; w!=_last_window; w++) {
+	for (w = _windows; w != _last_window; w++) {
-		if (w->window_class == WC_MAIN_WINDOW)
+		if (w->window_class == WC_MAIN_WINDOW) continue;
-			continue;
 		if (left + width > w->left &&
-		    w->left + w->width > left &&
+				w->left + w->width > left &&
 				top + height > w->top &&
-				w->top + w->height > top)
+				w->top + w->height > top) {
-					return false;
+			return false;
+		}
 	}
 	return true;
 }
 	_awap_r.width = width;
 	_awap_r.height = height;
 	if (IsGoodAutoPlace1(0, 24)) goto ok_pos;
-	for(w=_windows; w!=_last_window; w++) {
+	for (w = _windows; w != _last_window; w++) {
-		if (w->window_class == WC_MAIN_WINDOW)
+		if (w->window_class == WC_MAIN_WINDOW) continue;
-			continue;
 		if (IsGoodAutoPlace1(w->left+w->width+2,w->top)) goto ok_pos;
 		if (IsGoodAutoPlace1(w->left-   width-2,w->top)) goto ok_pos;
 		if (IsGoodAutoPlace1(w->left,w->top+w->height+2)) goto ok_pos;
 		if (IsGoodAutoPlace1(w->left,w->top-   height-2)) goto ok_pos;
 		if (IsGoodAutoPlace1(w->left-   width-2,w->top+w->height-height)) goto ok_pos;
 		if (IsGoodAutoPlace1(w->left+w->width-width,w->top+w->height+2)) goto ok_pos;
 		if (IsGoodAutoPlace1(w->left+w->width-width,w->top-   height-2)) goto ok_pos;
 	}
-	for(w=_windows; w!=_last_window; w++) {
+	for (w = _windows; w != _last_window; w++) {
-		if (w->window_class == WC_MAIN_WINDOW)
+		if (w->window_class == WC_MAIN_WINDOW) continue;
-			continue;
 		if (IsGoodAutoPlace2(w->left+w->width+2,w->top)) goto ok_pos;
 		if (IsGoodAutoPlace2(w->left-   width-2,w->top)) goto ok_pos;
 		if (IsGoodAutoPlace2(w->left,w->top+w->height+2)) goto ok_pos;
 		if (IsGoodAutoPlace2(w->left,w->top-   height-2)) goto ok_pos;
 	{
 		int left=0,top=24;
 restart:;
-		for(w=_windows; w!=_last_window; w++) {
+		for (w = _windows; w != _last_window; w++) {
 			if (w->left == left && w->top == top) {
 				left += 5;
 				top += 5;
 				goto restart;
 			}
 Window *AllocateWindowDescFront(const WindowDesc *desc, int value)
 {
 	Window *w;
-	if (BringWindowToFrontById(desc->cls, value))
+	if (BringWindowToFrontById(desc->cls, value)) return NULL;
-		return NULL;
 	w = AllocateWindowDesc(desc);
 	w->window_number = value;
 	return w;
 }
 Window *FindWindowFromPt(int x, int y)
 {
 	Window *w;
-	for(w=_last_window; w != _windows;) {
+	for (w = _last_window; w != _windows;) {
 		--w;
 		if (IS_INSIDE_1D(x, w->left, w->width) &&
-		    IS_INSIDE_1D(y, w->top, w->height))
+				IS_INSIDE_1D(y, w->top, w->height)) {
-					return w;
+			return w;
+		}
 	}
 	return NULL;
 }
 	for (w = _last_window; w != _windows;) {
 		--w;
 		if (w->flags4&WF_TIMEOUT_MASK && !(--w->flags4&WF_TIMEOUT_MASK)) {
 			CallWindowEventNP(w, WE_TIMEOUT);
-			if (w->desc_flags & WDF_UNCLICK_BUTTONS)
+			if (w->desc_flags & WDF_UNCLICK_BUTTONS) UnclickWindowButtons(w);
-				UnclickWindowButtons(w);
 		}
 	}
 }
 Window *GetCallbackWnd(void)
 static void HandlePlacePresize(void)
 {
 	Window *w;
 	WindowEvent e;
-	if (_special_mouse_mode != WSM_PRESIZE)
+	if (_special_mouse_mode != WSM_PRESIZE) return;
-		return;
+	w = GetCallbackWnd();
-	if ((w = GetCallbackWnd()) == NULL)
+	if (w == NULL) return;
-		return;
 	e.place.pt = GetTileBelowCursor();
 	if (e.place.pt.x == -1) {
 		_thd.selend.x = -1;
 		return;
 static bool HandleDragDrop(void)
 {
 	Window *w;
 	WindowEvent e;
-	if (_special_mouse_mode != WSM_DRAGDROP)
+	if (_special_mouse_mode != WSM_DRAGDROP) return true;
-		return true;
+	if (_left_button_down) return false;
-	if (_left_button_down)
-		return false;
 	w = GetCallbackWnd();
 	ResetObjectToPlace();
-	if (w) {
+	if (w != NULL) {
 		// send an event in client coordinates.
 		e.event = WE_DRAGDROP;
 		e.dragdrop.pt.x = _cursor.pos.x - w->left;
 		e.dragdrop.pt.y = _cursor.pos.y - w->top;
 		e.dragdrop.widget = GetWidgetFromPos(w, e.dragdrop.pt.x, e.dragdrop.pt.y);
 static bool HandlePopupMenu(void)
 {
 	Window *w;
 	WindowEvent e;
-	if (!_popup_menu_active)
+	if (!_popup_menu_active) return true;
-		return true;
 	w = FindWindowById(WC_TOOLBAR_MENU, 0);
 	if (w == NULL) {
 		_popup_menu_active = false;
 		return false;
 	static Window *last_w = NULL;
 	w = FindWindowFromPt(_cursor.pos.x, _cursor.pos.y);
 	// We changed window, put a MOUSEOVER event to the last window
-	if (last_w && last_w != w) {
+	if (last_w != NULL && last_w != w) {
 		e.event = WE_MOUSEOVER;
 		e.mouseover.pt.x = -1;
 		e.mouseover.pt.y = -1;
-		if (last_w->wndproc)
+		if (last_w->wndproc) last_w->wndproc(last_w, &e);
-			last_w->wndproc(last_w, &e);
 	}
 	last_w = w;
-	if (w) {
+	if (w != NULL) {
 		// send an event in client coordinates.
 		e.event = WE_MOUSEOVER;
 		e.mouseover.pt.x = _cursor.pos.x - w->left;
 		e.mouseover.pt.y = _cursor.pos.y - w->top;
 		if (w->widget != NULL) {
 static bool HandleWindowDragging(void)
 {
 	Window *w;
 	// Get out immediately if no window is being dragged at all.
-	if (!_dragging_window)
+	if (!_dragging_window) return true;
-		return true;
 	// Otherwise find the window...
 	for (w = _windows; w != _last_window; w++) {
 		if (w->flags4 & WF_DRAGGING) {
 			const Widget *t = &w->widget[1]; // the title bar ... ugh
 							(v_right > _screen.width - 13 && nx + t->right > v_right)) {
 						ny = v_bottom;
 					} else {
 						if (nx + t->left > v->left - 13 &&
 								nx + t->right < v_right + 13) {
-							if (w->top >= v_bottom)
+							if (w->top >= v_bottom) {
 								ny = v_bottom;
-							else if (w->left < nx)
+							} else if (w->left < nx) {
 								nx = v->left - 13 - t->left;
-							else
+							} else {
 								nx = v_right + 13 - t->right;
+							}
 						}
 					}
 				}
 			}
 			x = _cursor.pos.x - _drag_delta.x;
 			y = _cursor.pos.y - _drag_delta.y;
 			/* X and Y has to go by step.. calculate it */
-			if (w->resize.step_width > 1)
+			if (w->resize.step_width > 1) x = x - (x % w->resize.step_width);
-				x = x - (x % (int)w->resize.step_width);
+			if (w->resize.step_height > 1) y = y - (y % w->resize.step_height);
-			if (w->resize.step_height > 1)
-				y = y - (y % (int)w->resize.step_height);
 			/* Check if we don't go below the minimum set size */
 			if ((int)w->width + x < (int)w->resize.width)
 				x = w->resize.width - w->width;
 			if ((int)w->height + y < (int)w->resize.height)
 				y = w->resize.height - w->height;
 			/* Window already on size */
-			if (x == 0 && y == 0)
+			if (x == 0 && y == 0) return false;
-				return false;
 			/* Now find the new cursor pos.. this is NOT _cursor, because
 			    we move in steps. */
 			_drag_delta.x += x;
 			_drag_delta.y += y;
 					}
 					wi++;
 				}
 				/* We resized at least 1 widget, so let's rezise the window totally */
-				if (resize_width)
+				if (resize_width)  w->width  = x + w->width;
-					w->width  = x + w->width;
+				if (resize_height) w->height = y + w->height;
-				if (resize_height)
-					w->height = y + w->height;
 			}
 			e.event = WE_RESIZE;
 			e.sizing.size.x = x + w->width;
 			e.sizing.size.y = y + w->height;
 	int i;
 	int pos;
 	Scrollbar *sb;
 	// Get out quickly if no item is being scrolled
-	if (!_scrolling_scrollbar)
+	if (!_scrolling_scrollbar) return true;
-		return true;
 	// Find the scrolling window
-	for(w=_windows; w != _last_window; w++) {
+	for (w = _windows; w != _last_window; w++) {
 		if (w->flags4 & WF_SCROLL_MIDDLE) {
 			// Abort if no button is clicked any more.
 			if (!_left_button_down) {
 				w->flags4 &= ~WF_SCROLL_MIDDLE;
 				SetWindowDirty(w);
 {
 	Window *w;
 	ViewPort *vp;
 	int dx,dy, x, y, sub;
-	if (!_scrolling_viewport)
+	if (!_scrolling_viewport) return true;
-		return true;
 	if (!_right_button_down) {
 stop_capt:;
 		_cursor.fix_at = false;
 		_scrolling_viewport = false;
 		hx = (w->widget[4].right  - w->widget[4].left) / 2;
 		hy = (w->widget[4].bottom - w->widget[4].top ) / 2;
 		hvx = hx * -4 + hy * 8;
 		hvy = hx *  4 + hy * 8;
-		if (x < -hvx) { x = -hvx; sub = 0; }
+		if (x < -hvx) {
-		if (x > (int)MapMaxX() * 16 - hvx) { x = MapMaxX() * 16 - hvx; sub = 0; }
+			x = -hvx;
-		if (y < -hvy) { y = -hvy; sub = 0; }
+			sub = 0;
-		if (y > (int)MapMaxY() * 16 - hvy) { y = MapMaxY() * 16 - hvy; sub = 0; }
+		}
+		if (x > (int)MapMaxX() * 16 - hvx) {
+			x = MapMaxX() * 16 - hvx;
+			sub = 0;
+		}
+		if (y < -hvy) {
+			y = -hvy;
+			sub = 0;
+		}
+		if (y > (int)MapMaxY() * 16 - hvy) {
+			y = MapMaxY() * 16 - hvy;
+			sub = 0;
+		}
 		WP(w,smallmap_d).scroll_x = x;
 		WP(w,smallmap_d).scroll_y = y;
 		WP(w,smallmap_d).subscroll = sub;
 static Window *MaybeBringWindowToFront(Window *w)
 {
 	Window *u;
-	if (w->window_class == WC_MAIN_WINDOW || IsVitalWindow(w) ||
+	if (w->window_class == WC_MAIN_WINDOW ||
-			w->window_class == WC_TOOLTIPS    || w->window_class == WC_DROPDOWN_MENU)
+			IsVitalWindow(w) ||
-				return w;
+			w->window_class == WC_TOOLTIPS ||
+			w->window_class == WC_DROPDOWN_MENU) {
+		return w;
+	}
 	for (u = w; ++u != _last_window;) {
-		if (u->window_class == WC_MAIN_WINDOW || IsVitalWindow(u) ||
+		if (u->window_class == WC_MAIN_WINDOW ||
-			  u->window_class == WC_TOOLTIPS    || u->window_class == WC_DROPDOWN_MENU)
+				IsVitalWindow(u) ||
-				continue;
+				u->window_class == WC_TOOLTIPS ||
+				u->window_class == WC_DROPDOWN_MENU) {
+			continue;
+		}
 		if (w->left + w->width <= u->left ||
 				u->left + u->width <= w->left ||
 				w->top  + w->height <= u->top ||
-				u->top + u->height <= w->top)
+				u->top + u->height <= w->top) {
-					continue;
+			continue;
+		}
 		return BringWindowToFront(w);
 	}
 	return w;
 	we.keypress.ascii = key & 0xFF;
 	we.keypress.keycode = key >> 16;
 	we.keypress.cont = true;
 	// check if we have a query string window open before allowing hotkeys
-	if(FindWindowById(WC_QUERY_STRING, 0)!=NULL || FindWindowById(WC_SEND_NETWORK_MSG, 0)!=NULL || FindWindowById(WC_CONSOLE, 0)!=NULL || FindWindowById(WC_SAVELOAD, 0)!=NULL)
+	if (FindWindowById(WC_QUERY_STRING,     0) != NULL ||
+			FindWindowById(WC_SEND_NETWORK_MSG, 0) != NULL ||
+			FindWindowById(WC_CONSOLE,          0) != NULL ||
+			FindWindowById(WC_SAVELOAD,         0) != NULL) {
 		query_open = true;
+	}
 	// Call the event, start with the uppermost window.
-	for(w=_last_window; w != _windows;) {
+	for (w = _last_window; w != _windows;) {
 		--w;
 		// if a query window is open, only call the event for certain window types
-		if(query_open && w->window_class!=WC_QUERY_STRING && w->window_class!=WC_SEND_NETWORK_MSG && w->window_class!=WC_CONSOLE && w->window_class!=WC_SAVELOAD)
+		if (query_open &&
+				w->window_class != WC_QUERY_STRING &&
+				w->window_class != WC_SEND_NETWORK_MSG &&
+				w->window_class != WC_CONSOLE &&
+				w->window_class != WC_SAVELOAD) {
 			continue;
+		}
 		w->wndproc(w, &we);
-		if (!we.keypress.cont)
+		if (!we.keypress.cont) break;
-			break;
 	}
 	if (we.keypress.cont) {
 		w = FindWindowById(WC_MAIN_TOOLBAR, 0);
 		// When there is no toolbar w is null, check for that
 	ViewPort *vp;
 	DecreaseWindowCounters();
 	HandlePlacePresize();
 	UpdateTileSelection();
-	if (!VpHandlePlaceSizingDrag())
+	if (!VpHandlePlaceSizingDrag())  return;
-		return;
+	if (!HandleDragDrop())           return;
+	if (!HandlePopupMenu())          return;
-	if (!HandleDragDrop())
+	if (!HandleWindowDragging())     return;
-		return;
+	if (!HandleScrollbarScrolling()) return;
+	if (!HandleViewportScroll())     return;
-	if (!HandlePopupMenu())
+	if (!HandleMouseOver())          return;
-		return;
-	if (!HandleWindowDragging())
-		return;
-	if (!HandleScrollbarScrolling())
-		return;
-	if (!HandleViewportScroll())
-		return;
-	if (!HandleMouseOver())
-		return;
 	x = _cursor.pos.x;
 	y = _cursor.pos.y;
 		}
 		return;
 	}
 	w = FindWindowFromPt(x, y);
-	if (w == NULL)
+	if (w == NULL) return;
-		return;
 	w = MaybeBringWindowToFront(w);
 	vp = IsPtInWindowViewport(w, x, y);
 	if (vp != NULL) {
-		if (_game_mode == GM_MENU)
+		if (_game_mode == GM_MENU) return;
-			return;
 		// only allow zooming in-out in main window, or in viewports
-		if ( mousewheel && !(w->flags4 & WF_DISABLE_VP_SCROLL) &&
+		if (mousewheel &&
-			   (w->window_class == WC_MAIN_WINDOW || w->window_class == WC_EXTRA_VIEW_PORT) ) {
+				!(w->flags4 & WF_DISABLE_VP_SCROLL) && (
+					w->window_class == WC_MAIN_WINDOW ||
+					w->window_class == WC_EXTRA_VIEW_PORT
+				)) {
 			ZoomInOrOutToCursorWindow(mousewheel < 0,w);
 		}
 		if (click == 1) {
 			DEBUG(misc, 2) ("cursor: 0x%X (%d)", _cursor.sprite, _cursor.sprite);
 void UpdateWindows(void)
 {
 	Window *w;
 	int t;
+	t = _we4_timer + 1;
-	if ((t=_we4_timer+1) >= 100) {
+	if (t >= 100) {
-		for(w = _last_window; w != _windows;) {
+		for (w = _last_window; w != _windows;) {
 			w--;
 			CallWindowEventNP(w, WE_4);
 		}
 		t = 0;
 	}
 	_we4_timer = t;
-	for(w = _last_window; w != _windows;) {
+	for (w = _last_window; w != _windows;) {
 		w--;
 		if (w->flags4 & WF_WHITE_BORDER_MASK) {
 			w->flags4 -= WF_WHITE_BORDER_ONE;
 			if (!(w->flags4 & WF_WHITE_BORDER_MASK)) {
 				SetWindowDirty(w);
 		}
 	}
 	DrawDirtyBlocks();
-	for(w = _windows; w!=_last_window; w++) {
+	for (w = _windows; w != _last_window; w++) {
-		if (w->viewport != NULL)
+		if (w->viewport != NULL) UpdateViewportPosition(w);
-			UpdateViewportPosition(w);
 	}
 	DrawTextMessage();
 	// Redraw mouse cursor in case it was hidden
 	DrawMouseCursor();
 }
 int GetMenuItemIndex(const Window *w, int x, int y)
 {
 	if ((x -= w->left) >= 0 && x < w->width && (y -= w->top + 1) >= 0) {
 		y /= 10;
-		if (y < WP(w,menu_d).item_count && !HASBIT(WP(w,menu_d).disabled_items, y))
+		if (y < WP(w, const menu_d).item_count &&
+				!HASBIT(WP(w, const menu_d).disabled_items, y)) {
 			return y;
+		}
 	}
 	return -1;
 }
 void InvalidateWindow(byte cls, WindowNumber number)
 {
 	Window *w;
-	for(w=_windows; w!=_last_window; w++) {
+	for (w = _windows; w != _last_window; w++) {
-		if (w->window_class==cls && w->window_number==number)
+		if (w->window_class == cls && w->window_number == number) SetWindowDirty(w);
-			SetWindowDirty(w);
 	}
 }
 void InvalidateWidget(const Window* w, byte widget_index)
 {
 void InvalidateWindowWidget(byte cls, WindowNumber number, byte widget_index)
 {
 	const Window* w;
-	for(w=_windows; w!=_last_window; w++) {
+	for (w = _windows; w != _last_window; w++) {
-		if (w->window_class==cls && w->window_number==number) {
+		if (w->window_class == cls && w->window_number == number) {
 			InvalidateWidget(w, widget_index);
 		}
 	}
 }
 void InvalidateWindowClasses(byte cls)
 {
 	const Window* w;
-	for(w=_windows; w!=_last_window; w++) {
-		if (w->window_class==cls)
+	for (w = _windows; w != _last_window; w++) {
-			SetWindowDirty(w);
+		if (w->window_class == cls) SetWindowDirty(w);
 	}
 }
 void CallWindowTickEvent(void)
 {
 	Window *w;
-	for(w=_last_window; w != _windows;) {
+	for (w = _last_window; w != _windows;) {
 		--w;
 		CallWindowEventNP(w, WE_TICK);
 	}
 }
 void DeleteNonVitalWindows(void)
 {
 	Window *w;
-	for(w=_windows; w!=_last_window;) {
+	for (w = _windows; w != _last_window;) {
 		if (w->window_class != WC_MAIN_WINDOW &&
 				w->window_class != WC_SELECT_GAME &&
 				w->window_class != WC_MAIN_TOOLBAR &&
 				w->window_class != WC_STATUS_BAR &&
 				w->window_class != WC_TOOLBAR_MENU &&
 * then, does a little hacked loop of closing all stickied windows. Note
 * that standard windows (status bar, etc.) are not stickied, so these aren't affected */
 void DeleteAllNonVitalWindows(void)
 {
 	Window *w;
 	// Delete every window except for stickied ones
 	DeleteNonVitalWindows();
 	// Delete all sticked windows
 	for (w = _windows; w != _last_window;) {
 		if (w->flags4 & WF_STICKY) {
 void RelocateAllWindows(int neww, int newh)
 {
 	Window *w;
-	for(w=_windows; w!= _last_window ;w++) {
+	for (w = _windows; w != _last_window; w++) {
 		int left, top;
 		if (w->window_class == WC_MAIN_WINDOW) {
 			ViewPort *vp = w->viewport;
 			vp->width = w->width = neww;
 			if (left + (w->width>>1) >= neww) left = neww - w->width;
 			top = w->top;
 			if (top + (w->height>>1) >= newh) top = newh - w->height;
 		}
-		if (w->viewport) {
+		if (w->viewport != NULL) {
 			w->viewport->left += left - w->left;
 			w->viewport->top += top - w->top;
 		}
 		w->left = left;

changeset 2639	eeaefdabfdfd
parent 2631	2ed0eb408229
child 2643	f1fec6d1559a