cpp/openttd-patchpack/source Changeset - r21041:10121cda1c19

Changeset - r21041:10121cda1c19

Parent rev.

Child rev.

[Not reviewed]

master

0 3 0

rubidium - 11 years ago 2013-11-26 13:21:25
rubidium@openttd.org

(svn r26121) -Codechange: add some asserts after dynamic casts to show they shouldn't return NULL ever

3 files changed with 10 insertions and 2 deletions:

src/smallmap_gui.cpp

src/widget.cpp

src/widgets/dropdown.cpp

0 comments (0 inline, 0 general)

src/smallmap_gui.cpp

➞

Show inline comments

@@ @@ -1644,6 +1644,7 @@ public: @@
 		bar->SetupSmallestSize(w, init_array);
 		this->smallmap_window = dynamic_cast<SmallMapWindow *>(w);
 		assert(this->smallmap_window != NULL);
 		this->smallest_x = max(display->smallest_x, bar->smallest_x + smallmap_window->GetMinLegendWidth());
 		this->smallest_y = display->smallest_y + max(bar->smallest_y, smallmap_window->GetLegendHeight(smallmap_window->min_number_of_columns));
 		this->fill_x = max(display->fill_x, bar->fill_x);

src/widget.cpp

➞

Show inline comments

@@ @@ -147,7 +147,9 @@ void ScrollbarClickHandler(Window *w, NW @@
 		mi = nw->pos_y;
 		ma = nw->pos_y + nw->current_y;
+	}
 	ScrollbarClickPositioning(w, dynamic_cast<NWidgetScrollbar*>(nw), x, y, mi, ma);
 	NWidgetScrollbar *scrollbar = dynamic_cast<NWidgetScrollbar*>(nw);
 	assert(scrollbar != NULL);
 	ScrollbarClickPositioning(w, scrollbar, x, y, mi, ma);
+}
 /**
@@ @@ -1526,7 +1528,9 @@ void NWidgetMatrix::SetupSmallestSize(Wi @@
+	}
 	/* Reset the widget number. */
 	SB(dynamic_cast<NWidgetCore *>(this->head)->index, 16, 16, 0);
 	NWidgetCore *nw = dynamic_cast<NWidgetCore *>(this->head);
 	assert(nw != NULL);
 	SB(nw->index, 16, 16, 0);
 	this->head->SetupSmallestSize(w, init_array);
 	Dimension padding = {this->pip_pre + this->pip_post, this->pip_pre + this->pip_post};
@@ @@ -1595,6 +1599,7 @@ NWidgetCore *NWidgetMatrix::GetWidgetFro @@
 	if (sub_wid >= this->count) return NULL;
 	NWidgetCore *child = dynamic_cast<NWidgetCore *>(this->head);
 	assert(child != NULL);
 	child->AssignSizePosition(ST_RESIZE,
 			this->pos_x + (rtl ? this->pip_post - widget_col * this->widget_w : this->pip_pre + widget_col * this->widget_w) + base_offs_x,
 			this->pos_y + this->pip_pre + widget_row * this->widget_h + base_offs_y,
@@ @@ -1619,6 +1624,7 @@ NWidgetCore *NWidgetMatrix::GetWidgetFro @@
 	/* Get the appropriate offsets so we can draw the right widgets. */
 	NWidgetCore *child = dynamic_cast<NWidgetCore *>(this->head);
 	assert(child != NULL);
 	int start_x, start_y, base_offs_x, base_offs_y;
 	this->GetScrollOffsets(start_x, start_y, base_offs_x, base_offs_y);

src/widgets/dropdown.cpp

➞

Show inline comments

@@ @@ -475,6 +475,7 @@ int HideDropDownMenu(Window *pw) @@
 		if (w->window_class != WC_DROPDOWN_MENU) continue;
 		DropdownWindow *dw = dynamic_cast<DropdownWindow*>(w);
 		assert(dw != NULL);
 		if (pw->window_class == dw->parent_wnd_class &&
 				pw->window_number == dw->parent_wnd_num) {
 			int parent_button = dw->parent_button;

0 comments (0 inline, 0 general)