[screen-devel] [PATCH 2/8] Process: Fix sign-compare errors

[Kieran Bingham]

DoProcess:
oldlen here is defined as a signed integer, with the lenp coming through
as a size_t (unsigned). Convert the local storage to size_t to match, and
allow the compiler to catch any further signed errors later

suFin:
Another len vs local store of len which hasn't been converted to size_t
In this use case, the local l takes the size of the char arrays in
struct inputsu, which are > 0, so this it should be reasonable
to set l as size_t.

Signed-off-by: Kieran Bingham <address@hidden>
---
 src/process.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/process.c b/src/process.c
index 1bc204b..e443023 100644
--- a/src/process.c
+++ b/src/process.c
@@ -757,7 +757,7 @@ void ProcessInput2(char *ibuf, int ilen)
 
 void DoProcess(Window *window, char **bufp, size_t *lenp, struct paster *pa)
 {
-       int oldlen;
+       size_t oldlen;
        Display *d = display;
 
        /* XXX -> PasteStart */
@@ -5226,7 +5226,7 @@ static void suFin(char *buf, size_t len, void *data)
 {
        struct inputsu *i = (struct inputsu *)data;
        char *p;
-       int l;
+       size_t l;
 
        if (!*i->name) {
                p = i->name;
-- 
2.1.4