Skip to content

Commit

Permalink
extmod/modselect: Remove undocumented support for flags arg to poll.
Browse files Browse the repository at this point in the history
The signature of this method was poller.poll(timeout=-1, flags=0, /) but
the flags argument was not documented and is not CPython compatible.  So
it's removed in this commit.

(The optional flags remains for the ipoll() method, which is documented.)

Signed-off-by: Damien George <damien@micropython.org>
  • Loading branch information
dpgeorge committed Aug 7, 2023
1 parent ef71028 commit 3f417e8
Showing 1 changed file with 2 additions and 6 deletions.
8 changes: 2 additions & 6 deletions extmod/modselect.c
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@

#endif

// Flags for poll()
// Flags for ipoll()
#define FLAG_ONESHOT (1)

// A single pollable object.
Expand Down Expand Up @@ -524,15 +524,11 @@ STATIC mp_obj_t poll_poll(size_t n_args, const mp_obj_t *args) {
if (poll_obj_get_revents(poll_obj) != 0) {
mp_obj_t tuple[2] = {poll_obj->obj, MP_OBJ_NEW_SMALL_INT(poll_obj_get_revents(poll_obj))};
ret_list->items[n_ready++] = mp_obj_new_tuple(2, tuple);
if (self->flags & FLAG_ONESHOT) {
// Don't poll next time, until new event mask will be set explicitly
poll_obj_set_events(poll_obj, 0);
}
}
}
return MP_OBJ_FROM_PTR(ret_list);
}
MP_DEFINE_CONST_FUN_OBJ_VAR_BETWEEN(poll_poll_obj, 1, 3, poll_poll);
MP_DEFINE_CONST_FUN_OBJ_VAR_BETWEEN(poll_poll_obj, 1, 2, poll_poll);

STATIC mp_obj_t poll_ipoll(size_t n_args, const mp_obj_t *args) {
mp_obj_poll_t *self = MP_OBJ_TO_PTR(args[0]);
Expand Down

0 comments on commit 3f417e8

Please sign in to comment.