summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorEon S. Jeon2013-08-11 06:42:30 -0400
committerRoberto E. Vargas Caballero2013-08-30 07:50:05 +0200
commitf3d438b1015a031bc543bb2d65c81cc2329787d4 (patch)
tree5eb7150d9f6029b40a7ecff809f1e76f9f551519
parent587bc939f63b49c6155c4f1fdf97e73380207446 (diff)
downloadst-f3d438b1015a031bc543bb2d65c81cc2329787d4.tar.gz
st-f3d438b1015a031bc543bb2d65c81cc2329787d4.tar.xz
st-f3d438b1015a031bc543bb2d65c81cc2329787d4.zip
Regarding commit 7e3cff3
Hello. I reviewed and tested commit 7e3cff3, and made a patch that fixes some problems in it. 1. There's a semicolon after an if statement, which is obviously a typo. 2. The current way of calculating text position in "xdraws" yields inconsistent results in some cases. This is due to the use of "font->width", which varies. Instead, "xw.cw" has to be used as the character width. Sincerely, Eon
-rw-r--r--st.c6
1 files changed, 3 insertions, 3 deletions
diff --git a/st.c b/st.c
index 9733a9b..c751aa1 100644
--- a/st.c
+++ b/st.c
@@ -3111,7 +3111,7 @@ xdraws(char *s, Glyph base, int x, int y, int charlen, int bytelen) {
winy + font->ascent,
(FcChar8 *)u8fs,
u8fblen);
- xp += CEIL(font->width * cwscale * u8fl);
+ xp += xw.cw * u8fl;
}
break;
@@ -3121,7 +3121,7 @@ xdraws(char *s, Glyph base, int x, int y, int charlen, int bytelen) {
u8fblen += u8cblen;
}
if(doesexist) {
- if (oneatatime);
+ if (oneatatime)
continue;
break;
}
@@ -3184,7 +3184,7 @@ xdraws(char *s, Glyph base, int x, int y, int charlen, int bytelen) {
xp, winy + frc[i].font->ascent,
(FcChar8 *)u8c, u8cblen);
- xp += CEIL(font->width * cwscale);
+ xp += xw.cw;
}
/*