cpp/openttd-patchpack/source Changeset - r26561:25f3270fbf0e

Changeset - r26561:25f3270fbf0e

Parent rev.

Child rev.

[Not reviewed]

master

0 1 0

Peter Nelson - 23 months ago 2022-10-01 21:47:57
peter1138@openttd.org

Fix: Off-by-one in GetAlignedPosition().

1 file changed with 2 insertions and 2 deletions:

src/widget.cpp

0 comments (0 inline, 0 general)

src/widget.cpp

➞

Show inline comments

@@ @@ -39,13 +39,13 @@ static inline Point GetAlignedPosition(c @@
 	switch (align & SA_HOR_MASK) {
 		case SA_LEFT:       p.x = r.left; break;
 		case SA_HOR_CENTER: p.x = CenterBounds(r.left, r.right, d.width); break;
 		case SA_RIGHT:      p.x = r.right - d.width; break;
+		case SA_RIGHT:      p.x = r.right + 1 - d.width; break;
 		default: NOT_REACHED();
+	}
 	switch (align & SA_VERT_MASK) {
 		case SA_TOP:         p.y = r.top; break;
 		case SA_VERT_CENTER: p.y = CenterBounds(r.top, r.bottom, d.height); break;
 		case SA_BOTTOM:      p.y = r.bottom - d.height; break;
+		case SA_BOTTOM:      p.y = r.bottom + 1 - d.height; break;
 		default: NOT_REACHED();
+	}
 	return p;

0 comments (0 inline, 0 general)