emacs-bug-tracker
[Top][All Lists]
Advanced
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#36709: closed (Likely bug in scm_port_poll)

From: GNU bug Tracking System
Subject: bug#36709: closed (Likely bug in scm_port_poll)
Date: Mon, 18 Nov 2019 14:18:01 +0000 
Your message dated Mon, 18 Nov 2019 15:17:29 +0100
with message-id <address@hidden>
and subject line Re: bug#36709: Likely bug in scm_port_poll
has caused the debbugs.gnu.org bug report #36709,
regarding Likely bug in scm_port_poll
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden.)


-- 
36709: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=36709
GNU Bug Tracking System
Contact address@hidden with problems
--- Begin Message --- Subject: Likely bug in scm_port_poll Date: Wed, 17 Jul 2019 15:25:15 -0400 User-agent: Gnus/5.13 (Gnus v5.13) Emacs/26.2 (gnu/linux) 
I sent the following message about a month ago, and now am forwarding to
bug-guile so that we do not forget about it.

       Mark


From: Mark H Weaver <address@hidden>
Subject: Likely bug in scm_port_poll
Date: Tue, 18 Jun 2019 00:40:27 -0400
Message-ID: <address@hidden>
Cc: address@hidden
To: Andy Wingo <address@hidden>

Hi Andy,

If you'd be willing to take a quick look at this, I'd be grateful.
Probably a bug, but I feel uneasy that I might be overlooking something.

scm_port_poll contains the following code:

  if (scm_i_string_contains_char (events, 'r'))
    c_events |= POLLIN;
  if (scm_i_string_contains_char (events, '!'))
    c_events |= POLLPRI;
  if (scm_i_string_contains_char (events, 'w'))
    c_events |= POLLIN;

That second POLLIN should be POLLOUT, right?

      Thanks,
        Mark

https://git.savannah.gnu.org/cgit/guile.git/tree/libguile/ports.c?h=stable-2.2&id=420c2632bb1f48e492a035c1d216f209734f45e6#n1423

--- End Message ---
--- Begin Message --- Subject: Re: bug#36709: Likely bug in scm_port_poll Date: Mon, 18 Nov 2019 15:17:29 +0100 User-agent: Gnus/5.13 (Gnus v5.13) Emacs/26.3 (gnu/linux) 
Hi Mark,

Mark H Weaver <address@hidden> skribis:

> If you'd be willing to take a quick look at this, I'd be grateful.
> Probably a bug, but I feel uneasy that I might be overlooking something.
>
> scm_port_poll contains the following code:
>
>   if (scm_i_string_contains_char (events, 'r'))
>     c_events |= POLLIN;
>   if (scm_i_string_contains_char (events, '!'))
>     c_events |= POLLPRI;
>   if (scm_i_string_contains_char (events, 'w'))
>     c_events |= POLLIN;
>
> That second POLLIN should be POLLOUT, right?

I agree.  Clearly it must be a typo, so I’ve pushed the fix as
659526d33b2db1b0924b2769b6a0b824709860cc.

Thanks!

Ludo’.

--- End Message ---
reply via email to
[Prev in Thread] Current Thread [Next in Thread]