JasonWoof Got questions, comments, patches, etc.? Contact Jason Woofenden
Use int instead of long for color
authorJohannes Hofmann <johannes.hofmann@gmx.de>
Sun, 24 Nov 2013 09:20:45 +0000 (10:20 +0100)
committerRoberto E. Vargas Caballero <k0ga@shike2.com>
Sun, 24 Nov 2013 09:43:35 +0000 (10:43 +0100)
This patch replaces long by int32_t. It saves
some memory on 64bit systems.

st.c

diff --git a/st.c b/st.c
index ee8dc94..fb084b1 100644 (file)
--- a/st.c
+++ b/st.c
@@ -180,8 +180,8 @@ typedef unsigned short ushort;
 typedef struct {
        char c[UTF_SIZ]; /* character code */
        ushort mode;      /* attribute flags */
-       ulong fg;        /* foreground  */
-       ulong bg;        /* background  */
+       uint32_t fg;      /* foreground  */
+       uint32_t bg;      /* background  */
 } Glyph;
 
 typedef Glyph *Line;
@@ -379,7 +379,7 @@ static void tsetdirtattr(int);
 static void tsetmode(bool, bool, int *, int);
 static void tfulldirt(void);
 static void techo(char *, int);
-static long tdefcolor(int *, int *, int);
+static uint32_t tdefcolor(int *, int *, int);
 static void tselcs(void);
 static void tdeftran(char);
 static inline bool match(uint, uint);
@@ -1666,9 +1666,9 @@ tdeleteline(int n) {
        tscrollup(term.c.y, n);
 }
 
-long
+uint32_t
 tdefcolor(int *attr, int *npar, int l) {
-       long idx = -1;
+       int32_t idx = -1;
        uint r, g, b;
 
        switch (attr[*npar + 1]) {
@@ -1717,7 +1717,7 @@ tdefcolor(int *attr, int *npar, int l) {
 void
 tsetattr(int *attr, int l) {
        int i;
-       long idx;
+       int32_t idx;
 
        for(i = 0; i < l; i++) {
                switch(attr[i]) {