The code didn't compile (must have been sleepy when committing). "return"
authorTor Lillqvist <tml@iki.fi>
Mon, 31 Jul 2000 15:11:02 +0000 (15:11 +0000)
committerTor Lillqvist <tml@src.gnome.org>
Mon, 31 Jul 2000 15:11:02 +0000 (15:11 +0000)
2000-07-31  Tor Lillqvist  <tml@iki.fi>

* giowin32.c (buffer_read): The code didn't compile (must
have been sleepy when committing). "return" instead of "break"

(g_io_win32_fd_add_watch): Cannot check if the file descriptor is
readable by calling ReadFile to read zero bytes. ReadFile blocks
on NT even if trying to read nothing at all. So, don't check if
file descriptor is readable; assume this function isn't called
otherwise.

ChangeLog
ChangeLog.pre-2-0
ChangeLog.pre-2-10
ChangeLog.pre-2-12
ChangeLog.pre-2-2
ChangeLog.pre-2-4
ChangeLog.pre-2-6
ChangeLog.pre-2-8
giowin32.c
glib/giowin32.c

index c2afe466bbaaaf585235f765a9bff4525bcf5149..34c24d2402f8c701c740a7eb9d283cec9587e816 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,14 @@
+2000-07-31  Tor Lillqvist  <tml@iki.fi>
+
+       * giowin32.c (buffer_read): The code didn't compile (must
+       have been sleepy when committing). "return" instead of "break"
+
+       (g_io_win32_fd_add_watch): Cannot check if the file descriptor is
+       readable by calling ReadFile to read zero bytes. ReadFile blocks
+       on NT even if trying to read nothing at all. So, don't check if
+       file descriptor is readable; assume this function isn't called
+       otherwise.
+
 Sun Jul 30 10:44:16 2000  Tim Janik  <timj@gtk.org>
 
        * gmain.c (g_get_current_time): fix tor's recent changes which
index c2afe466bbaaaf585235f765a9bff4525bcf5149..34c24d2402f8c701c740a7eb9d283cec9587e816 100644 (file)
@@ -1,3 +1,14 @@
+2000-07-31  Tor Lillqvist  <tml@iki.fi>
+
+       * giowin32.c (buffer_read): The code didn't compile (must
+       have been sleepy when committing). "return" instead of "break"
+
+       (g_io_win32_fd_add_watch): Cannot check if the file descriptor is
+       readable by calling ReadFile to read zero bytes. ReadFile blocks
+       on NT even if trying to read nothing at all. So, don't check if
+       file descriptor is readable; assume this function isn't called
+       otherwise.
+
 Sun Jul 30 10:44:16 2000  Tim Janik  <timj@gtk.org>
 
        * gmain.c (g_get_current_time): fix tor's recent changes which
index c2afe466bbaaaf585235f765a9bff4525bcf5149..34c24d2402f8c701c740a7eb9d283cec9587e816 100644 (file)
@@ -1,3 +1,14 @@
+2000-07-31  Tor Lillqvist  <tml@iki.fi>
+
+       * giowin32.c (buffer_read): The code didn't compile (must
+       have been sleepy when committing). "return" instead of "break"
+
+       (g_io_win32_fd_add_watch): Cannot check if the file descriptor is
+       readable by calling ReadFile to read zero bytes. ReadFile blocks
+       on NT even if trying to read nothing at all. So, don't check if
+       file descriptor is readable; assume this function isn't called
+       otherwise.
+
 Sun Jul 30 10:44:16 2000  Tim Janik  <timj@gtk.org>
 
        * gmain.c (g_get_current_time): fix tor's recent changes which
index c2afe466bbaaaf585235f765a9bff4525bcf5149..34c24d2402f8c701c740a7eb9d283cec9587e816 100644 (file)
@@ -1,3 +1,14 @@
+2000-07-31  Tor Lillqvist  <tml@iki.fi>
+
+       * giowin32.c (buffer_read): The code didn't compile (must
+       have been sleepy when committing). "return" instead of "break"
+
+       (g_io_win32_fd_add_watch): Cannot check if the file descriptor is
+       readable by calling ReadFile to read zero bytes. ReadFile blocks
+       on NT even if trying to read nothing at all. So, don't check if
+       file descriptor is readable; assume this function isn't called
+       otherwise.
+
 Sun Jul 30 10:44:16 2000  Tim Janik  <timj@gtk.org>
 
        * gmain.c (g_get_current_time): fix tor's recent changes which
index c2afe466bbaaaf585235f765a9bff4525bcf5149..34c24d2402f8c701c740a7eb9d283cec9587e816 100644 (file)
@@ -1,3 +1,14 @@
+2000-07-31  Tor Lillqvist  <tml@iki.fi>
+
+       * giowin32.c (buffer_read): The code didn't compile (must
+       have been sleepy when committing). "return" instead of "break"
+
+       (g_io_win32_fd_add_watch): Cannot check if the file descriptor is
+       readable by calling ReadFile to read zero bytes. ReadFile blocks
+       on NT even if trying to read nothing at all. So, don't check if
+       file descriptor is readable; assume this function isn't called
+       otherwise.
+
 Sun Jul 30 10:44:16 2000  Tim Janik  <timj@gtk.org>
 
        * gmain.c (g_get_current_time): fix tor's recent changes which
index c2afe466bbaaaf585235f765a9bff4525bcf5149..34c24d2402f8c701c740a7eb9d283cec9587e816 100644 (file)
@@ -1,3 +1,14 @@
+2000-07-31  Tor Lillqvist  <tml@iki.fi>
+
+       * giowin32.c (buffer_read): The code didn't compile (must
+       have been sleepy when committing). "return" instead of "break"
+
+       (g_io_win32_fd_add_watch): Cannot check if the file descriptor is
+       readable by calling ReadFile to read zero bytes. ReadFile blocks
+       on NT even if trying to read nothing at all. So, don't check if
+       file descriptor is readable; assume this function isn't called
+       otherwise.
+
 Sun Jul 30 10:44:16 2000  Tim Janik  <timj@gtk.org>
 
        * gmain.c (g_get_current_time): fix tor's recent changes which
index c2afe466bbaaaf585235f765a9bff4525bcf5149..34c24d2402f8c701c740a7eb9d283cec9587e816 100644 (file)
@@ -1,3 +1,14 @@
+2000-07-31  Tor Lillqvist  <tml@iki.fi>
+
+       * giowin32.c (buffer_read): The code didn't compile (must
+       have been sleepy when committing). "return" instead of "break"
+
+       (g_io_win32_fd_add_watch): Cannot check if the file descriptor is
+       readable by calling ReadFile to read zero bytes. ReadFile blocks
+       on NT even if trying to read nothing at all. So, don't check if
+       file descriptor is readable; assume this function isn't called
+       otherwise.
+
 Sun Jul 30 10:44:16 2000  Tim Janik  <timj@gtk.org>
 
        * gmain.c (g_get_current_time): fix tor's recent changes which
index c2afe466bbaaaf585235f765a9bff4525bcf5149..34c24d2402f8c701c740a7eb9d283cec9587e816 100644 (file)
@@ -1,3 +1,14 @@
+2000-07-31  Tor Lillqvist  <tml@iki.fi>
+
+       * giowin32.c (buffer_read): The code didn't compile (must
+       have been sleepy when committing). "return" instead of "break"
+
+       (g_io_win32_fd_add_watch): Cannot check if the file descriptor is
+       readable by calling ReadFile to read zero bytes. ReadFile blocks
+       on NT even if trying to read nothing at all. So, don't check if
+       file descriptor is readable; assume this function isn't called
+       otherwise.
+
 Sun Jul 30 10:44:16 2000  Tim Janik  <timj@gtk.org>
 
        * gmain.c (g_get_current_time): fix tor's recent changes which
index 3d6443143733a17162b6a72c9c1fd467dcf0c84c..ea00711b072093fcd6fbb3c9e0ad76a0f27ee841 100644 (file)
@@ -273,7 +273,10 @@ buffer_read (GIOWin32Channel *channel,
       WaitForSingleObject (channel->data_avail_event, INFINITE);
       LOCK (channel->mutex);
       if (channel->rdp == channel->wrp && !channel->running)
-       break;
+       {
+         UNLOCK (channel->mutex);
+         return 0;
+       }
     }
   
   if (channel->rdp < channel->wrp)
@@ -635,10 +638,7 @@ g_io_win32_fd_add_watch (GIOChannel    *channel,
     g_print ("g_io_win32_fd_add_watch: fd:%d handle:%#x\n",
             win32_channel->fd, watch->pollfd.fd);
   
-  /* Is it readable? (Would be strange to watch it otherwise, but... */
-  if (ReadFile ((HANDLE) _get_osfhandle (win32_channel->fd),
-               dummy, 0, &nbytes, NULL))
-    create_reader_thread (win32_channel, fd_reader);
+  create_reader_thread (win32_channel, fd_reader);
 
   g_main_add_poll (&watch->pollfd, priority);
   
index 3d6443143733a17162b6a72c9c1fd467dcf0c84c..ea00711b072093fcd6fbb3c9e0ad76a0f27ee841 100644 (file)
@@ -273,7 +273,10 @@ buffer_read (GIOWin32Channel *channel,
       WaitForSingleObject (channel->data_avail_event, INFINITE);
       LOCK (channel->mutex);
       if (channel->rdp == channel->wrp && !channel->running)
-       break;
+       {
+         UNLOCK (channel->mutex);
+         return 0;
+       }
     }
   
   if (channel->rdp < channel->wrp)
@@ -635,10 +638,7 @@ g_io_win32_fd_add_watch (GIOChannel    *channel,
     g_print ("g_io_win32_fd_add_watch: fd:%d handle:%#x\n",
             win32_channel->fd, watch->pollfd.fd);
   
-  /* Is it readable? (Would be strange to watch it otherwise, but... */
-  if (ReadFile ((HANDLE) _get_osfhandle (win32_channel->fd),
-               dummy, 0, &nbytes, NULL))
-    create_reader_thread (win32_channel, fd_reader);
+  create_reader_thread (win32_channel, fd_reader);
 
   g_main_add_poll (&watch->pollfd, priority);