From fdc69149267ae4b14c00df0b3a2bdb80018a0fb8 Mon Sep 17 00:00:00 2001 From: PatR Date: Wed, 29 Dec 2021 02:13:34 -0800 Subject: [PATCH] Qt paper doll fix for two-handed weapons Allowing optional arguments can be risky. The recent glyph changes added a new argument to the routine that constructs a mirror image of a tile but not to the call to that routine. Because an existing argument was optional, the compiler didn't complain about the new one being missing. Obsolete optional 'fem' argument ought to removed but this doesn't tackle that. --- win/Qt/qt_glyph.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/win/Qt/qt_glyph.cpp b/win/Qt/qt_glyph.cpp index e4057e59b..6c1df7c0c 100644 --- a/win/Qt/qt_glyph.cpp +++ b/win/Qt/qt_glyph.cpp @@ -98,7 +98,7 @@ void NetHackQtGlyphs::drawGlyph(QPainter& painter, int glyph, int tileidx, painter.drawPixmap(x, y, pm, px, py, width(), height()); } else { // for paper doll; mirrored image for left side of two-handed weapon - painter.drawPixmap(x, y, reversed_pixmap(glyph, fem), + painter.drawPixmap(x, y, reversed_pixmap(glyph, tileidx, fem), 0, 0, width(), height()); } }