cpp/openttd-patchpack/source Changeset - r14928:db3712149038

Changeset - r14928:db3712149038

Parent rev.

Child rev.

[Not reviewed]

master

0 1 0

terkhen - 14 years ago 2010-04-02 12:28:08
terkhen@openttd.org

(svn r19541) -Feature: Sort the list of refit options by cargo class / name.

1 file changed with 35 insertions and 1 deletions:

src/vehicle_gui.cpp

0 comments (0 inline, 0 general)

src/vehicle_gui.cpp

➞

Show inline comments

@@ @@ -234,6 +234,11 @@ struct RefitOption { @@
+	{
 		return other.cargo != this->cargo || other.value != this->value;
+	}
 	FORCEINLINE bool operator == (const RefitOption &other) const
+	{
 		return other.cargo == this->cargo && other.value == this->value;
+	}
 };
 typedef SmallVector<RefitOption, 32> RefitList;
@@ @@ -258,7 +263,7 @@ static void BuildRefitList(const Vehicle @@
 		/* Loop through all cargos in the refit mask */
 		const CargoSpec *cs;
-		FOR_ALL_CARGOSPECS(cs) {
+		FOR_ALL_SORTED_CARGOSPECS(cs) {
 			CargoID cid = cs->Index();
 			/* Skip cargo type if it's not listed */
 			if (!HasBit(cmask, cid)) continue;
@@ @@ -380,6 +385,35 @@ struct RefitWindow : public Window { @@
 		this->vscroll.SetCount(this->list.Length());
+	}
 	virtual void OnInit()
+	{
 		if (this->cargo != NULL) {
 			/* Store the RefitOption currently in use. */
 			RefitOption current_refit_option = *(this->cargo);
 			/* Rebuild the refit list */
 			BuildRefitList(Vehicle::Get(this->window_number), &this->list);
 			this->vscroll.SetCount(this->list.Length());
 			this->sel = -1;
 			this->cargo = NULL;
 			for (uint i = 0; i < this->list.Length(); i++) {
 				if (this->list[i] == current_refit_option) {
 					this->sel = i;
 					this->cargo = &this->list[i];
 					this->vscroll.ScrollTowards(i);
 					break;
+				}
+			}
 			/* If the selected refit option was not found, scroll the window to the initial position. */
 			if (this->sel == -1) this->vscroll.ScrollTowards(0);
 		} else {
 			/* Rebuild the refit list */
 			BuildRefitList(Vehicle::Get(this->window_number), &this->list);
 			this->vscroll.SetCount(this->list.Length());
+		}
+	}
 	virtual void OnPaint()
+	{
 		Vehicle *v = Vehicle::Get(this->window_number);

0 comments (0 inline, 0 general)