[hackers] [sent] fix 2 uninitialized variables and bin == NULL check || Markus Teich

[git]

commit 008ff1b88b3e9cdd13e82d0ec4fb573c977f6a3d
Author:     Markus Teich <markus.te...@stusta.mhn.de>
AuthorDate: Wed Dec 9 00:50:05 2015 +0100
Commit:     Markus Teich <markus.te...@stusta.mhn.de>
CommitDate: Wed Dec 9 00:50:05 2015 +0100

    fix 2 uninitialized variables and bin == NULL check

diff --git a/drw.c b/drw.c
index 15797ce..be30400 100644
--- a/drw.c
+++ b/drw.c
@@ -228,7 +228,7 @@ int
 drw_text(Drw *drw, int x, int y, unsigned int w, unsigned int h, const char 
*text, int invert) {
        char buf[1024];
        int tx, ty, th;
-       unsigned int ew;
+       unsigned int ew = 0;
        Colormap cmap;
        Visual *vis;
        XftDraw *d;
diff --git a/sent.c b/sent.c
index 5c72597..51f6eed 100644
--- a/sent.c
+++ b/sent.c
@@ -162,23 +162,25 @@ filter(int fd, const char *cmd)
 Image *ffopen(char *filename)
 {
        unsigned char hdr[16];
-       char *bin;
+       char *bin = NULL;
        regex_t regex;
        Image *img;
        size_t i;
        int tmpfd, fd;
 
-       for (bin = NULL, i = 0; i < LEN(filters); i++) {
+       for (i = 0; i < LEN(filters); i++) {
                if (regcomp(&regex, filters[i].regex,
                            REG_NOSUB | REG_EXTENDED | REG_ICASE))
                        continue;
                if (!regexec(&regex, filename, 0, NULL, 0)) {
-                       if (!(bin = filters[i].bin))
-                               return NULL;
+                       bin = filters[i].bin
                        break;
                }
        }
 
+       if (!bin)
+               return NULL;
+
        if ((fd = open(filename, O_RDONLY)) < 0) {
                eprintf("Unable to open file %s:", filename);
                return NULL;
@@ -355,7 +357,7 @@ void ffdraw(Image *img)
 void getfontsize(Slide *s, unsigned int *width, unsigned int *height)
 {
        int i, j;
-       unsigned int curw, imax;
+       unsigned int curw, new_max;
        float lfac = linespacing * (s->linecount - 1) + 1;
 
        /* fit height */
@@ -369,13 +371,12 @@ void getfontsize(Slide *s, unsigned int *width, unsigned 
int *height)
        *width = 0;
        for (i = 0; i < s->linecount; i++) {
                curw = drw_fontset_getwidth(d, s->lines[i]);
-               if (curw >= *width)
-                       imax = i;
+               newmax = (curw >= *width);
                while (j > 0 && curw > xw.uw) {
                        drw_setfontset(d, fonts[--j]);
                        curw = drw_fontset_getwidth(d, s->lines[i]);
                }
-               if (imax == i)
+               if (newmax)
                        *width = curw;
        }
        *height = fonts[j]->h * lfac;