(Faccept_process_output): If `millisec' is non-nil, `seconds' default to 0.

(wait_reading_process_output): Also return non-nil if we read output
from a non-running process.

diff --git a/src/ChangeLog b/src/ChangeLog
index fbebde7a3a7739b4396bc822651b4d46bc445029..b605c2b3f48b602882b43dddbbbef1bd727e3765 100644 (file)
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -1,7 +1,14 @@
+2008-05-29  Stefan Monnier  <monnier@iro.umontreal.ca>
+
+       * process.c (Faccept_process_output): If `millisec' is non-nil,
+       `seconds' default to 0.
+       (wait_reading_process_output): Also return non-nil if we read output
+       from a non-running process.
+
 2008-05-29  Jason Rumney  <jasonr@gnu.org>
 
-        * w32font.c (w32font_open_internal): Prefer truetype fonts unless
-        'raster' specified.
+       * w32font.c (w32font_open_internal): Prefer truetype fonts unless
+       'raster' specified.
 
 2008-05-29  Juanma Barranquero  <lekktu@gmail.com>
 

diff --git a/src/process.c b/src/process.c
index e17357508063b569d380cdc1cae3b8b2c2155dc1..413bd8522b3be442288f6b01fc7e8918e5494fe3 100644 (file)
--- a/src/process.c
+++ b/src/process.c
@@ -3896,6 +3896,7 @@ Non-nil second arg SECONDS and third arg MILLISEC are number of
 seconds and milliseconds to wait; return after that much time whether
 or not there is input.  If SECONDS is a floating point number,
 it specifies a fractional number of seconds to wait.
+The MILLISEC argument is obsolete and should be avoided.
 
 If optional fourth arg JUST-THIS-ONE is non-nil, only accept output
 from PROCESS, suspending reading output from other processes.
@@ -3911,6 +3912,18 @@ Return non-nil if we received any output before the timeout expired.  */)
   else
     just_this_one = Qnil;
 
+  if (!NILP (millisec))
+    { /* Obsolete calling convention using integers rather than floats.  */
+      CHECK_NUMBER (millisec);
+      if (NILP (seconds))
+       seconds = make_float (XINT (millisec) / 1000.0);
+      else
+       {
+         CHECK_NUMBER (seconds);
+         seconds = make_float (XINT (millisec) / 1000.0 + XINT (seconds));
+       }
+    }
+
   if (!NILP (seconds))
     {
       if (INTEGERP (seconds))
@@ -3924,19 +3937,6 @@ Return non-nil if we received any output before the timeout expired.  */)
       else
        wrong_type_argument (Qnumberp, seconds);
 
-      if (INTEGERP (millisec))
-       {
-         int carry;
-         usecs += XINT (millisec) * 1000;
-         carry = usecs / 1000000;
-         secs += carry;
-         if ((usecs -= carry * 1000000) < 0)
-           {
-             secs--;
-             usecs += 1000000;
-           }
-       }
-
       if (secs < 0 || (secs == 0 && usecs == 0))
        secs = -1, usecs = 0;
     }
@@ -4475,7 +4475,10 @@ wait_reading_process_output (time_limit, microsecs, read_kbd, do_display,
                break;
 
               if (0 < nread)
-                total_nread += nread;
+                {
+                 total_nread += nread;
+                 got_some_input = 1;
+               }
 #ifdef EIO
              else if (nread == -1 && EIO == errno)
                 break;