bug-gnu-emacs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: accept-process-output incompatible change in CVS Emacs: patch includ


From: Kim F. Storm
Subject: Re: accept-process-output incompatible change in CVS Emacs: patch included
Date: Sat, 05 May 2007 00:33:39 +0200
User-agent: Gnus/5.11 (Gnus v5.11) Emacs/22.0.98 (gnu/linux)

Per Cederqvist <address@hidden> writes:

> This change in the upcoming Emacs 22 breaks some code:
>
>> 2006-03-22  Kim F. Storm  <address@hidden>
>> 
>>      * process.c (Faccept_process_output): Fix to comply with lisp
>>      reference.  Change arg "timeout" to "seconds" and allow both
>>      integer and float value.  Change arg "timeout-msec" to "millisec"
>>      and interpret" as milliseconds rather than microseconds.  Fix doc
>>      string accordingly.

Please see:

http://lists.gnu.org/archive/html/emacs-devel/2006-03/msg00937.html

>
> I enclose a patch that makes accept-process-output
> backwards-compatible with Emacs 21.
>
> One application I'm running now runs very slowly, since its carefully
> tuned timeoutes are multiplied by 1000.

Well, other code believing the lisp ref, and was broken by the old
behaviour.

So just reverting the change is no good at this time.

What application is that?  

>  If, for some reason, my patch is unacceptable, please
> mention this as an incompatible change in the NEWS file.

This is a good point.

-- 
Kim F. Storm  http://www.cua.dk





reply via email to

[Prev in Thread] Current Thread [Next in Thread]