Changeset 2435 for neercsTweet

Timestamp:

Jun 15, 2008, 8:40:22 PM (13 years ago)

Author:

Pascal Terjan

Message:

• No need to close the fd just before calling destroy_screen

• Don't skip one element when we are looping through the list and remove one

File:

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

neercs/trunk/src/screens.c

@@ -337 +337 @@
 
             if(nr == 0 || errno != EWOULDBLOCK) {
-                close(screen_list->screen[i]->fd);
-                screen_list->screen[i]->fd = -1;
                 destroy_screen(screen_list->screen[i]);
                 remove_screen(screen_list, i, 0);
@@ -348 +346 @@
                 }
                 if(i < (screen_list->pty)) (screen_list->pty)--;
+                /* Don't skip the element which is now at position i */
+                i--;
                 refresh = 1;
             }