cpp/openttd-patchpack/source Changeset - r4884:efc306240cec

Changeset - r4884:efc306240cec

Parent rev.

Child rev.

[Not reviewed]

master

0 3 0

Darkvater - 18 years ago 2006-10-18 14:20:10
darkvater@openttd.org

(svn r6821) -Codechange: For the measurement tool do not show the tooltip when the selection
is only a single tile. Seems this was a bit annoying to some ;)

3 files changed with 49 insertions and 43 deletions:

misc_gui.c

viewport.c

window.h

0 comments (0 inline, 0 general)

misc_gui.c

➞

Show inline comments

@@ @@ -665,18 +665,18 @@ static void TooltipsWndProc(Window *w, W @@
 * @param str String to be displayed
 * @param params (optional) up to 5 pieces of additional information that may be
 * added to a tooltip; currently only supports parameters of {NUM} (integer) */
 void GuiShowTooltipsWithArgs(StringID str, uint paramcount, uint32 params[])
+void GuiShowTooltipsWithArgs(StringID str, uint paramcount, const uint32 params[])
+{
 	char buffer[512];
 	BoundingRect br;
 	Window *w;
 	uint i;
 	int x, y;
 	Window *w = FindWindowById(WC_TOOLTIPS, 0);
 	if (w != NULL) DeleteWindow(w);
 	DeleteWindowById(WC_TOOLTIPS, 0);
 	/* We only show measurement tooltips with shift pressed down */
 	if (paramcount != 0 && !_patches.measure_tooltip) return;
 	/* We only show measurement tooltips with patch setting on */
 	if (str == STR_NULL || (paramcount != 0 && !_patches.measure_tooltip)) return;
 	for (i = 0; i != paramcount; i++) SetDParam(i, params[i]);
 	GetString(buffer, str);

viewport.c

➞

Show inline comments

@@ @@ -2110,6 +2110,8 @@ static int CalcHeightdiff(HighLightStyle @@
 	return (int)(h1 - h0) * 50;
+}
 static const measure_strings_length[] = {STR_NULL, STR_MEASURE_LENGTH, STR_MEASURE_LENGTH_HEIGHTDIFF};
 // while dragging
 static void CalcRaildirsDrawstyle(TileHighlightData *thd, int x, int y, int method)
+{
@@ @@ -2215,23 +2217,23 @@ static void CalcRaildirsDrawstyle(TileHi @@
 		TileIndex t0 = TileVirtXY(thd->selstart.x, thd->selstart.y);
 		TileIndex t1 = TileVirtXY(x, y);
 		uint distance = DistanceManhattan(t0, t1) + 1;
-		int heightdiff = CalcHeightdiff(b, distance, t0, t1);
+		byte index = 0;
 		uint params[2];
 		/* If we are showing a tooltip for horizontal or vertical drags,
 		 * 2 tiles have a length of 1. To bias towards the ceiling we add
 		 * one before division. It feels more natural to count 3 lengths as 2 */
 		if ((b & HT_DIR_MASK) != HT_DIR_X && (b & HT_DIR_MASK) != HT_DIR_Y) {
 			distance = (distance + 1) / 2;
 		if (distance != 1) {
 			int heightdiff = CalcHeightdiff(b, distance, t0, t1);
 			/* If we are showing a tooltip for horizontal or vertical drags,
 			 * 2 tiles have a length of 1. To bias towards the ceiling we add
 			 * one before division. It feels more natural to count 3 lengths as 2 */
 			if ((b & HT_DIR_MASK) != HT_DIR_X && (b & HT_DIR_MASK) != HT_DIR_Y) {
 				distance = (distance + 1) / 2;
+			}
 			params[index++] = distance;
 			if (heightdiff != 0) params[index++] = heightdiff;
+		}
 		params[0] = distance;
 		if (heightdiff == 0) {
 			GuiShowTooltipsWithArgs(STR_MEASURE_LENGTH, 1, params);
 		} else {
 			params[1] = heightdiff;
 			GuiShowTooltipsWithArgs(STR_MEASURE_LENGTH_HEIGHTDIFF, 2, params);
+		}
 		GuiShowTooltipsWithArgs(measure_strings_length[index], index, params);
+	}
 	thd->selend.x = x;
@@ @@ -2294,21 +2296,22 @@ calc_heightdiff_single_direction:; @@
 				TileIndex t0 = TileVirtXY(sx, sy);
 				TileIndex t1 = TileVirtXY(x, y);
 				uint distance = DistanceManhattan(t0, t1) + 1;
 				/* With current code passing a HT_LINE style to calculate the height
 				 * difference is enough. However if/when a point-tool is created
 				 * with this method, function should be called with new_style (below)
 				 * instead of HT_LINE | style case HT_POINT is handled specially
 				 * new_style := (_thd.next_drawstyle & HT_RECT) ? HT_LINE | style : _thd.next_drawstyle; */
 				int heightdiff = CalcHeightdiff(HT_LINE | style, 0, t0, t1);
 				byte index = 0;
 				uint params[2];
 				params[0] = distance;
 				if (heightdiff == 0) {
 					GuiShowTooltipsWithArgs(STR_MEASURE_LENGTH, 1, params);
 				} else {
 					params[1] = heightdiff;
 					GuiShowTooltipsWithArgs(STR_MEASURE_LENGTH_HEIGHTDIFF, 2, params);
 				if (distance != 1) {
 					/* With current code passing a HT_LINE style to calculate the height
 					 * difference is enough. However if/when a point-tool is created
 					 * with this method, function should be called with new_style (below)
 					 * instead of HT_LINE | style case HT_POINT is handled specially
 					 * new_style := (_thd.next_drawstyle & HT_RECT) ? HT_LINE | style : _thd.next_drawstyle; */
 					int heightdiff = CalcHeightdiff(HT_LINE | style, 0, t0, t1);
 					params[index++] = distance;
 					if (heightdiff != 0) params[index++] = heightdiff;
+				}
 				GuiShowTooltipsWithArgs(measure_strings_length[index], index, params);
 			} break;
 		case VPM_X_AND_Y_LIMITED: { /* drag an X by Y constrained rect area */
@@ @@ -2316,13 +2319,17 @@ calc_heightdiff_single_direction:; @@
 			x = sx + clamp(x - sx, -limit, limit);
 			y = sy + clamp(y - sy, -limit, limit);
 			/* Fallthrough */
-		case VPM_X_AND_Y: { /* drag an X by Y area */
 		case VPM_X_AND_Y: /* drag an X by Y area */
 			if (_patches.measure_tooltip) {
 				static const StringID measure_strings_area[] = {
 					STR_NULL, STR_NULL, STR_MEASURE_AREA, STR_MEASURE_AREA_HEIGHTDIFF
 				};
 				TileIndex t0 = TileVirtXY(sx, sy);
 				TileIndex t1 = TileVirtXY(x, y);
 				uint dx = abs(TileX(t0) - TileX(t1)) + 1;
 				uint dy = abs(TileY(t0) - TileY(t1)) + 1;
-				int heightdiff;
+				byte index = 0;
 				uint params[3];
 				/* If dragging an area (eg dynamite tool) and it is actually a single
@@ @@ -2336,18 +2343,17 @@ calc_heightdiff_single_direction:; @@
+					}
+				}
 				heightdiff = CalcHeightdiff(style, 0, t0, t1);
 				params[0] = dx;
 				params[1] = dy;
 				if (heightdiff == 0) {
 					GuiShowTooltipsWithArgs(STR_MEASURE_AREA, 2, params);
 				} else {
 					params[2] = heightdiff;
 					GuiShowTooltipsWithArgs(STR_MEASURE_AREA_HEIGHTDIFF, 3, params);
 				if (dx != 1 || dy != 1) {
 					int heightdiff = CalcHeightdiff(style, 0, t0, t1);
 					params[index++] = dx;
 					params[index++] = dy;
 					if (heightdiff != 0) params[index++] = heightdiff;
+				}
 				GuiShowTooltipsWithArgs(measure_strings_area[index], index, params);
+			}
-		} break;
 		break;
+		}
 		default: NOT_REACHED();

window.h

➞

Show inline comments

@@ @@ -807,7 +807,7 @@ void CDECL SetWindowWidgetsHiddenState(W @@
 void CDECL SetWindowWidgetsLoweredState(Window *w, bool lowered_stat, int widgets, ...);
 /* misc_gui.c*/
 void GuiShowTooltipsWithArgs(StringID str, uint paramcount, uint params[]);
+void GuiShowTooltipsWithArgs(StringID str, uint paramcount, const uint params[]);
 static inline void GuiShowTooltips(StringID str)
+{
 	GuiShowTooltipsWithArgs(str, 0, NULL);

0 comments (0 inline, 0 general)