Changeset 2434 for neercs/trunk/srcTweet

Timestamp:

Jun 15, 2008, 8:17:16 PM (14 years ago)

Author:

Pascal Terjan

Message:

• It's not very useful to check a pointer after dereferencing it
• It's not very useful to change the value of a parameter before leaving
• 0 is a valid fd

File:

• neercs/trunk/src/screens.c (modified) (2 diffs)

neercs/trunk/src/screens.c

@@ -90 +90 @@
 int destroy_screen(struct screen *s)
 {
-    if(s->fd>0)
+    if(s->fd>=0)
         close(s->fd);
     if(s->buf)
@@ -100 +100 @@
         cucul_free_canvas(s->cv);
     s->cv = NULL;
-    if(s)
-        free(s);
-    s = NULL;
+    free(s);
     return 1;
 }