cpp/openttd-patchpack/source Changeset - r20439:d34496be8ad3

Changeset - r20439:d34496be8ad3

Parent rev.

Child rev.

[Not reviewed]

master

0 11 0

rubidium - 11 years ago 2013-06-24 18:39:19
rubidium@openttd.org

(svn r25454) -Remove/Fix/Cleanup: SETX(Y) does not work at all with other than default fonts, so get rid of it

11 files changed with 47 insertions and 192 deletions:

src/aircraft_gui.cpp

src/build_vehicle_gui.cpp

src/gfx.cpp

145

src/gfx_func.h

src/newgrf_text.cpp

src/roadveh_gui.cpp

src/ship_gui.cpp

src/string.cpp

src/strings.cpp

src/table/control_codes.h

src/train_gui.cpp

0 comments (0 inline, 0 general)

src/aircraft_gui.cpp

➞

Show inline comments

@@ @@ -39,7 +39,7 @@ void DrawAircraftDetails(const Aircraft @@
 			SetDParam(0, u->engine_type);
 			SetDParam(1, u->build_year);
 			SetDParam(2, u->value);
-			DrawString(left, right, y, STR_VEHICLE_INFO_BUILT_VALUE, TC_FROMSTRING, SA_LEFT | SA_STRIP);
 			DrawString(left, right, y, STR_VEHICLE_INFO_BUILT_VALUE);
 			SetDParam(0, u->cargo_type);
 			SetDParam(1, u->cargo_cap);

src/build_vehicle_gui.cpp

➞

Show inline comments

@@ @@ -911,7 +911,7 @@ void DrawEngineList(VehicleType type, in @@
 		const uint num_engines = GetGroupNumEngines(_local_company, selected_group, engine);
 		SetDParam(0, engine);
-		DrawString(text_left, text_right, y + normal_text_y_offset, STR_ENGINE_NAME, engine == selected_id ? TC_WHITE : TC_BLACK, SA_STRIP | (rtl ? SA_RIGHT : SA_LEFT));
 		DrawString(text_left, text_right, y + normal_text_y_offset, STR_ENGINE_NAME, engine == selected_id ? TC_WHITE : TC_BLACK, (rtl ? SA_RIGHT : SA_LEFT));
 		DrawVehicleEngine(l, r, sprite_x, y + sprite_y_offset, engine, (show_count && num_engines == 0) ? PALETTE_CRASH : GetEnginePalette(engine, _local_company), EIT_PURCHASE);
 		if (show_count) {
 			SetDParam(0, num_engines);

src/gfx.cpp

➞

Show inline comments

@@ @@ -437,11 +437,10 @@ static WChar *HandleBiDiAndArabicShapes( @@
  * If the string is truncated, add three dots ('...') to show this.
  * @param *str string that is checked and possibly truncated
  * @param maxw maximum width in pixels of the string
  * @param ignore_setxy whether to ignore SETX(Y) or not
  * @param start_fontsize Fontsize to start the text with
  * @return new width of (truncated) string
  */
-static int TruncateString(char *str, int maxw, bool ignore_setxy, FontSize start_fontsize)
 static int TruncateString(char *str, int maxw, FontSize start_fontsize)
+{
 	int w = 0;
 	FontSize size = start_fontsize;
@@ @@ -464,13 +463,7 @@ static int TruncateString(char *str, int @@
 				return ddd_w;
+			}
 		} else {
 			if (c == SCC_SETX) {
 				if (!ignore_setxy) w = *str;
 				str++;
 			} else if (c == SCC_SETXY) {
 				if (!ignore_setxy) w = *str;
 				str += 2;
 			} else if (c == SCC_TINYFONT) {
 			if (c == SCC_TINYFONT) {
 				size = FS_SMALL;
 				ddd = GetCharacterWidth(size, '.') * 3;
 			} else if (c == SCC_BIGFONT) {
@@ @@ -514,11 +507,6 @@ static int GetStringWidth(const WChar *s @@
 			width += GetCharacterWidth(size, c);
 		} else {
 			switch (c) {
 				case SCC_SETX:
 				case SCC_SETXY:
 					/* At this point there is no SCC_SETX(Y) anymore */
 					NOT_REACHED();
 					break;
 				case SCC_TINYFONT: size = FS_SMALL; break;
 				case SCC_BIGFONT:  size = FS_LARGE; break;
 				case '\n':
@@ @@ -551,135 +539,54 @@ static int GetStringWidth(const WChar *s @@
  */
 static int DrawString(int left, int right, int top, char *str, const char *last, DrawStringParams &params, StringAlignment align, bool underline = false, bool truncate = true)
+{
 	/* We need the outer limits of both left/right */
 	int min_left = INT32_MAX;
 	int max_right = INT32_MIN;
 	int initial_left = left;
 	int initial_right = right;
 	int initial_top = top;
 	if (truncate) TruncateString(str, right - left + 1, (align & SA_STRIP) == SA_STRIP, params.fontsize);
 	/*
 	 * To support SETX and SETXY properly with RTL languages we have to
 	 * calculate the offsets from the right. To do this we need to split
 	 * the string and draw the parts separated by SETX(Y).
 	 * So here we split
 	 */
 	static SmallVector<WChar *, 4> setx_offsets;
 	setx_offsets.Clear();
 	if (truncate) TruncateString(str, right - left + 1, params.fontsize);
 	WChar draw_buffer[DRAW_STRING_BUFFER];
 	WChar *p = draw_buffer;
 	*setx_offsets.Append() = p;
 	char *loc = str;
 	for (;;) {
 		WChar c;
 		/* We cannot use Utf8Consume as we need the location of the SETX(Y) */
 		size_t len = Utf8Decode(&c, loc);
 	const char *text = str;
 	for (WChar c = Utf8Consume(&text); c != '\0'; c = Utf8Consume(&text)) {
 		*p++ = c;
+	}
 	*p++ = '\0';
 		if (c == '\0') break;
 		if (p >= lastof(draw_buffer) - 3) {
 			/* Make sure we never overflow (even if copying SCC_SETX(Y)). */
 			*p = '\0';
 			break;
+		}
 		if (c != SCC_SETX && c != SCC_SETXY) {
 			loc += len;
 			continue;
+		}
 	/* In case we have a RTL language we swap the alignment. */
 	if (!(align & SA_FORCE) && _current_text_dir == TD_RTL && (align & SA_HOR_MASK) != SA_HOR_CENTER) align ^= SA_RIGHT;
 	WChar *to_draw = HandleBiDiAndArabicShapes(draw_buffer);
 	int w = GetStringWidth(to_draw, params.fontsize);
 		if (align & SA_STRIP) {
 			/* We do not want to keep the SETX(Y)!. It was already copied, so
 			 * remove it and undo the incrementing of the pointer! */
 			*p-- = '\0';
 			loc += len + (c == SCC_SETXY ? 2 : 1);
 			continue;
+		}
 		if ((align & SA_HOR_MASK) != SA_LEFT) {
 			DEBUG(grf, 1, "Using SETX and/or SETXY when not aligned to the left. Fixing alignment...");
 	/* right is the right most position to draw on. In this case we want to do
 	 * calculations with the width of the string. In comparison right can be
 	 * seen as lastof(todraw) and width as lengthof(todraw). They differ by 1.
 	 * So most +1/-1 additions are to move from lengthof to 'indices'.
 	 */
 	switch (align & SA_HOR_MASK) {
 		case SA_LEFT:
 			/* right + 1 = left + w */
 			right = left + w - 1;
 			break;
 			/* For left alignment and change the left so it will roughly be in the
 			 * middle. This will never cause the string to be completely centered,
 			 * but once SETX is used you cannot be sure the actual content of the
 			 * string is centered, so it doesn't really matter. */
 			align = SA_LEFT | SA_FORCE;
 			initial_left = left = max(left, (left + right - (int)GetStringBoundingBox(str).width) / 2);
+		}
 		case SA_HOR_CENTER:
 			left  = RoundDivSU(right + 1 + left - w, 2);
 			/* right + 1 = left + w */
 			right = left + w - 1;
 			break;
 		/* We add the begin of the string, but don't add it twice */
 		if (p != draw_buffer) {
 			*setx_offsets.Append() = p;
 			p[-1] = '\0';
 			*p++ = c;
+		}
 		case SA_RIGHT:
 			left = right + 1 - w;
 			break;
 		/* Skip the SCC_SETX(Y) ... */
 		loc += len;
 		/* ... copy the x coordinate ... */
 		*p++ = *loc++;
 		/* ... and finally copy the y coordinate if it exists */
 		if (c == SCC_SETXY) *p++ = *loc++;
 		default:
 			NOT_REACHED();
+	}
 	/* In case we have a RTL language we swap the alignment. */
 	if (!(align & SA_FORCE) && _current_text_dir == TD_RTL && !(align & SA_STRIP) && (align & SA_HOR_MASK) != SA_HOR_CENTER) align ^= SA_RIGHT;
 	for (WChar **iter = setx_offsets.Begin(); iter != setx_offsets.End(); iter++) {
 		WChar *to_draw = *iter;
 		int offset = 0;
 		/* Skip the SETX(Y) and set the appropriate offsets. */
 		if (*to_draw == SCC_SETX || *to_draw == SCC_SETXY) {
 			to_draw++;
 			offset = *to_draw++;
 			if (*to_draw == SCC_SETXY) top = initial_top + *to_draw++;
+		}
 		to_draw = HandleBiDiAndArabicShapes(to_draw);
 		int w = GetStringWidth(to_draw, params.fontsize);
 		/* right is the right most position to draw on. In this case we want to do
 		 * calculations with the width of the string. In comparison right can be
 		 * seen as lastof(todraw) and width as lengthof(todraw). They differ by 1.
 		 * So most +1/-1 additions are to move from lengthof to 'indices'.
 		 */
 		switch (align & SA_HOR_MASK) {
 			case SA_LEFT:
 				/* right + 1 = left + w */
 				left = initial_left + offset;
 				right = left + w - 1;
 				break;
 			case SA_HOR_CENTER:
 				left  = RoundDivSU(initial_right + 1 + initial_left - w, 2);
 				/* right + 1 = left + w */
 				right = left + w - 1;
 				break;
 			case SA_RIGHT:
 				left = initial_right + 1 - w - offset;
 				break;
 			default:
 				NOT_REACHED();
+		}
 		min_left  = min(left, min_left);
 		max_right = max(right, max_right);
 		ReallyDoDrawString(to_draw, left, top, params, !truncate);
 		if (underline) {
 			GfxFillRect(left, top + FONT_HEIGHT_NORMAL, right, top + FONT_HEIGHT_NORMAL, _string_colourremap[1]);
+		}
 	ReallyDoDrawString(to_draw, left, top, params, !truncate);
 	if (underline) {
 		GfxFillRect(left, top + FONT_HEIGHT_NORMAL, right, top + FONT_HEIGHT_NORMAL, _string_colourremap[1]);
+	}
-	return (align & SA_HOR_MASK) == SA_RIGHT ? min_left : max_right;
+	return (align & SA_HOR_MASK) == SA_RIGHT ? left : right;
+}
 /**
@@ @@ -811,8 +718,6 @@ uint32 FormatStringLinebreaks(char *str, @@
 			} else {
 				switch (c) {
 					case '\0': return num + (size << 16);
 					case SCC_SETX:  str++; break;
 					case SCC_SETXY: str += 2; break;
 					case SCC_TINYFONT: size = FS_SMALL; break;
 					case SCC_BIGFONT:  size = FS_LARGE; break;
 					case '\n': goto end_of_inner_loop;
@@ @@ -855,8 +760,6 @@ static int GetMultilineStringHeight(cons @@
 		switch (c) {
 			case 0:            y += fh; if (--num < 0) return maxy; break;
 			case '\n':         y += fh;                             break;
 			case SCC_SETX:     src++;                               break;
 			case SCC_SETXY:    src++; y = (int)*src++;              break;
 			case SCC_TINYFONT: fh = GetCharacterHeight(FS_SMALL);   break;
 			case SCC_BIGFONT:  fh = GetCharacterHeight(FS_LARGE);   break;
 			default:           maxy = max<int>(maxy, y + fh);       break;
@@ @@ -1019,10 +922,6 @@ static int DrawStringMultiLine(int left, @@
 			WChar c = Utf8Consume(&src);
 			if (c == 0) {
 				break;
 			} else if (c == SCC_SETX) {
 				src++;
 			} else if (c == SCC_SETXY) {
 				src += 2;
 			} else if (skip_lines > 0) {
 				/* Skipped drawing, so do additional processing to update params. */
 				if (c >= SCC_BLUE && c <= SCC_BLACK) {
@@ @@ -1111,11 +1010,6 @@ Dimension GetStringBoundingBox(const cha @@
 			br.width += GetCharacterWidth(size, c);
 		} else {
 			switch (c) {
 				case SCC_SETX: br.width = max((uint)*str++, br.width); break;
 				case SCC_SETXY:
 					br.width  = max((uint)*str++, br.width);
 					br.height = max((uint)*str++, br.height);
 					break;
 				case SCC_TINYFONT: size = FS_SMALL; break;
 				case SCC_BIGFONT:  size = FS_LARGE; break;
 				case '\n':
@@ @@ -1229,9 +1123,6 @@ skip_cont:; @@
 		} else if (c == SCC_PREVIOUS_COLOUR) { // revert to the previous colour
 			params.SetPreviousColour();
 			goto switch_colour;
 		} else if (c == SCC_SETX || c == SCC_SETXY) { // {SETX}/{SETXY}
 			/* The characters are handled before calling this. */
 			NOT_REACHED();
 		} else if (c == SCC_TINYFONT) { // {TINYFONT}
 			params.SetFontSize(FS_SMALL);
 		} else if (c == SCC_BIGFONT) { // {BIGFONT}

src/gfx_func.h

➞

Show inline comments

@@ @@ -105,7 +105,6 @@ enum StringAlignment { @@
 	SA_CENTER      = SA_HOR_CENTER | SA_VERT_CENTER, ///< Center both horizontally and vertically.
 	SA_FORCE       = 1 << 4, ///< Force the alignment, i.e. don't swap for RTL languages.
 	SA_STRIP       = 1 << 5, ///< Strip the SETX/SETXY commands from the string
 };
 DECLARE_ENUM_AS_BIT_SET(StringAlignment)

src/newgrf_text.cpp

➞

Show inline comments

@@ @@ -432,8 +432,8 @@ char *TranslateTTDPatchCodes(uint32 grfi @@
 		switch (c) {
 			case 0x01:
 				if (str[0] == '\0') goto string_end;
 				d += Utf8Encode(d, SCC_SETX);
 				*d++ = *str++;
 				d += Utf8Encode(d, ' ');
 				str++;
 				break;
 			case 0x0A: break;
 			case 0x0D:
@@ @@ -447,9 +447,8 @@ char *TranslateTTDPatchCodes(uint32 grfi @@
 			case 0x0F: d += Utf8Encode(d, SCC_BIGFONT); break;
 			case 0x1F:
 				if (str[0] == '\0' || str[1] == '\0') goto string_end;
 				d += Utf8Encode(d, SCC_SETXY);
 				*d++ = *str++;
 				*d++ = *str++;
 				d += Utf8Encode(d, ' ');
 				str += 2;
 				break;
 			case 0x7B:
 			case 0x7C:

src/roadveh_gui.cpp

➞

Show inline comments

@@ @@ -35,7 +35,7 @@ void DrawRoadVehDetails(const Vehicle *v @@
 	SetDParam(0, v->engine_type);
 	SetDParam(1, v->build_year);
 	SetDParam(2, v->value);
-	DrawString(left, right, y + y_offset, STR_VEHICLE_INFO_BUILT_VALUE, TC_FROMSTRING, SA_LEFT | SA_STRIP);
 	DrawString(left, right, y + y_offset, STR_VEHICLE_INFO_BUILT_VALUE);
 	if (v->HasArticulatedPart()) {
 		CargoArray max_cargo;

src/ship_gui.cpp

➞

Show inline comments

@@ @@ -62,7 +62,7 @@ void DrawShipDetails(const Vehicle *v, i @@
 	SetDParam(0, v->engine_type);
 	SetDParam(1, v->build_year);
 	SetDParam(2, v->value);
-	DrawString(left, right, y, STR_VEHICLE_INFO_BUILT_VALUE, TC_FROMSTRING, SA_LEFT | SA_STRIP);
 	DrawString(left, right, y, STR_VEHICLE_INFO_BUILT_VALUE);
 	SetDParam(0, v->cargo_type);
 	SetDParam(1, v->cargo_cap);

src/string.cpp

➞

Show inline comments

@@ @@ -248,16 +248,6 @@ void str_validate(char *str, const char @@
 			/* Replace the undesirable character with a question mark */
 			str += len;
 			if ((settings & SVS_REPLACE_WITH_QUESTION_MARK) != 0) *dst++ = '?';
 			/* In case of these two special cases assume that they really
 			 * mean SETX/SETXY and also "eat" the parameter. If this was
 			 * not the case the string was broken to begin with and this
 			 * would not break much more. */
 			if (c == SCC_SETX) {
 				str++;
 			} else if (c == SCC_SETXY) {
 				str += 2;
+			}
+		}
+	}

src/strings.cpp

➞

Show inline comments

@@ @@ -1003,21 +1003,6 @@ static char *FormatString(char *buff, co @@
 				break;
+			}
 			case SCC_SETX: // {SETX}
 				if (buff + Utf8CharLen(SCC_SETX) + 1 < last) {
 					buff += Utf8Encode(buff, SCC_SETX);
 					*buff++ = *str++;
+				}
 				break;
 			case SCC_SETXY: // {SETXY}
 				if (buff + Utf8CharLen(SCC_SETXY) + 2 < last) {
 					buff += Utf8Encode(buff, SCC_SETXY);
 					*buff++ = *str++;
 					*buff++ = *str++;
+				}
 				break;
 			case SCC_REVISION: // {REV}
 				buff = strecpy(buff, _openttd_revision, last);
 				break;
@@ @@ -2036,14 +2021,7 @@ bool MissingGlyphSearcher::FindMissingGl @@
 		FontSize size = this->DefaultSize();
 		if (str != NULL) *str = text;
 		for (WChar c = Utf8Consume(&text); c != '\0'; c = Utf8Consume(&text)) {
 			if (c == SCC_SETX) {
 				/* SetX is, together with SetXY as special character that
 					* uses the next (two) characters as data points. We have
 					* to skip those, otherwise the UTF8 reading will go haywire. */
 				text++;
 			} else if (c == SCC_SETXY) {
 				text += 2;
 			} else if (c == SCC_TINYFONT) {
 			if (c == SCC_TINYFONT) {
 				size = FS_SMALL;
 			} else if (c == SCC_BIGFONT) {
 				size = FS_LARGE;

src/table/control_codes.h

➞

Show inline comments

@@ @@ -27,8 +27,6 @@ enum StringControlCode { @@
 	SCC_ENCODED = SCC_CONTROL_START,
 	/* Display control codes */
 	SCC_SETX,
 	SCC_SETXY,
 	SCC_TINYFONT,  ///< Switch to small font
 	SCC_BIGFONT,   ///< Switch to large font

src/train_gui.cpp

➞

Show inline comments

@@ @@ -215,12 +215,12 @@ static void TrainDetailsInfoTab(const Ve @@
 	if (RailVehInfo(v->engine_type)->railveh_type == RAILVEH_WAGON) {
 		SetDParam(0, v->engine_type);
 		SetDParam(1, v->value);
-		DrawString(left, right, y, STR_VEHICLE_DETAILS_TRAIN_WAGON_VALUE, TC_FROMSTRING, SA_LEFT | SA_STRIP);
 		DrawString(left, right, y, STR_VEHICLE_DETAILS_TRAIN_WAGON_VALUE);
 	} else {
 		SetDParam(0, v->engine_type);
 		SetDParam(1, v->build_year);
 		SetDParam(2, v->value);
-		DrawString(left, right, y, STR_VEHICLE_DETAILS_TRAIN_ENGINE_BUILT_AND_VALUE, TC_FROMSTRING, SA_LEFT | SA_STRIP);
 		DrawString(left, right, y, STR_VEHICLE_DETAILS_TRAIN_ENGINE_BUILT_AND_VALUE);
+	}
+}

0 comments (0 inline, 0 general)