diff options
author | Pedro Alves <pedro@palves.net> | 2022-05-16 12:20:08 +0100 |
---|---|---|
committer | Pedro Alves <pedro@palves.net> | 2022-05-16 19:58:08 +0100 |
commit | 187075ebbc0cba7b58685c7214028e791b5af844 (patch) | |
tree | 3c8c5081acc905b26c16112a8c225cadf47ed922 /gdbsupport | |
parent | 36a5b3705352f228cdd14a7f9d5e85177fad034c (diff) | |
download | binutils-187075ebbc0cba7b58685c7214028e791b5af844.zip binutils-187075ebbc0cba7b58685c7214028e791b5af844.tar.gz binutils-187075ebbc0cba7b58685c7214028e791b5af844.tar.bz2 |
Reindent gdbsupport/event-loop.cc:handle_file_event
The handle_file_event function has a few unnecessary {} lexical
blocks, presumably because they were originally if blocks, and the
conditions were removed, or something along those lines.
Remove the unnecessary blocks, and reindent.
Change-Id: Iaecbe5c9f4940a80b81dbbc42e51ce506f6aafb2
Diffstat (limited to 'gdbsupport')
-rw-r--r-- | gdbsupport/event-loop.cc | 104 |
1 files changed, 50 insertions, 54 deletions
diff --git a/gdbsupport/event-loop.cc b/gdbsupport/event-loop.cc index 18f57ef..f078c12 100644 --- a/gdbsupport/event-loop.cc +++ b/gdbsupport/event-loop.cc @@ -492,65 +492,61 @@ handle_file_event (file_handler *file_ptr, int ready_mask) { int mask; + /* See if the desired events (mask) match the received events + (ready_mask). */ + +#ifdef HAVE_POLL + if (use_poll) { - { - /* With poll, the ready_mask could have any of three events - set to 1: POLLHUP, POLLERR, POLLNVAL. These events - cannot be used in the requested event mask (events), but - they can be returned in the return mask (revents). We - need to check for those event too, and add them to the - mask which will be passed to the handler. */ + int error_mask; - /* See if the desired events (mask) match the received - events (ready_mask). */ + /* With poll, the ready_mask could have any of three events set + to 1: POLLHUP, POLLERR, POLLNVAL. These events cannot be + used in the requested event mask (events), but they can be + returned in the return mask (revents). We need to check for + those event too, and add them to the mask which will be + passed to the handler. */ -#ifdef HAVE_POLL - if (use_poll) - { - int error_mask; - - /* POLLHUP means EOF, but can be combined with POLLIN to - signal more data to read. */ - error_mask = POLLHUP | POLLERR | POLLNVAL; - mask = ready_mask & (file_ptr->mask | error_mask); - - if ((mask & (POLLERR | POLLNVAL)) != 0) - { - /* Work in progress. We may need to tell somebody - what kind of error we had. */ - if (mask & POLLERR) - warning (_("Error detected on fd %d"), file_ptr->fd); - if (mask & POLLNVAL) - warning (_("Invalid or non-`poll'able fd %d"), - file_ptr->fd); - file_ptr->error = 1; - } - else - file_ptr->error = 0; - } - else -#endif /* HAVE_POLL */ - { - if (ready_mask & GDB_EXCEPTION) - { - warning (_("Exception condition detected on fd %d"), - file_ptr->fd); - file_ptr->error = 1; - } - else - file_ptr->error = 0; - mask = ready_mask & file_ptr->mask; - } + /* POLLHUP means EOF, but can be combined with POLLIN to + signal more data to read. */ + error_mask = POLLHUP | POLLERR | POLLNVAL; + mask = ready_mask & (file_ptr->mask | error_mask); - /* If there was a match, then call the handler. */ - if (mask != 0) - { - event_loop_ui_debug_printf (file_ptr->is_ui, - "invoking fd file handler `%s`", - file_ptr->name.c_str ()); - file_ptr->proc (file_ptr->error, file_ptr->client_data); - } + if ((mask & (POLLERR | POLLNVAL)) != 0) + { + /* Work in progress. We may need to tell somebody + what kind of error we had. */ + if (mask & POLLERR) + warning (_("Error detected on fd %d"), file_ptr->fd); + if (mask & POLLNVAL) + warning (_("Invalid or non-`poll'able fd %d"), + file_ptr->fd); + file_ptr->error = 1; + } + else + file_ptr->error = 0; + } + else +#endif /* HAVE_POLL */ + { + if (ready_mask & GDB_EXCEPTION) + { + warning (_("Exception condition detected on fd %d"), + file_ptr->fd); + file_ptr->error = 1; } + else + file_ptr->error = 0; + mask = ready_mask & file_ptr->mask; + } + + /* If there was a match, then call the handler. */ + if (mask != 0) + { + event_loop_ui_debug_printf (file_ptr->is_ui, + "invoking fd file handler `%s`", + file_ptr->name.c_str ()); + file_ptr->proc (file_ptr->error, file_ptr->client_data); } } |