[cadcdev-svn-commits] [SCM] branch master updated. v2.0.0-365-gf9bbd47

[Lawrence S. <ljs...@us...>]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "A pseudo Operating System for the Dreamcast.".

The branch, master has been updated
       via  f9bbd470a9313d707447144a62d399df56df37ad (commit)
      from  956a83c75c40b17197474162beae60d055cf6cbb (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit f9bbd470a9313d707447144a62d399df56df37ad
Author: Lawrence Sebald <ljs...@us...>
Date:   Tue Feb 11 23:56:16 2020 -0500

    Fix a few more fall-through cases that GCC didn't complain about.
    
    Because GCC actually looks at comments before complaining about these,
    apparently.

-----------------------------------------------------------------------

Summary of changes:
 addons/libppp/lcp.c  | 4 ++++
 kernel/net/net_tcp.c | 3 +++
 2 files changed, 7 insertions(+)

diff --git a/addons/libppp/lcp.c b/addons/libppp/lcp.c
index 220c70f..986d1e6 100644
--- a/addons/libppp/lcp.c
+++ b/addons/libppp/lcp.c
@@ -218,6 +218,7 @@ static int lcp_handle_configure_req(ppp_protocol_t *self, const lcp_pkt_t *pkt,
         case PPP_STATE_OPENED:
             /* XXXX: This layer down. */
             /* Fall through... */
+            __fallthrough;
 
         case PPP_STATE_STOPPED:
             lcp_send_client_cfg(self, 0);
@@ -486,6 +487,7 @@ static int lcp_handle_configure_ack(ppp_protocol_t *self, const lcp_pkt_t *pkt,
             /* Uh oh... Something's gone wrong. */
             /* XXXX: This layer down. */
             /* Fall through... */
+            __fallthrough;
 
         case PPP_STATE_ACK_RECEIVED:
             /* Well, this isn't good... Resend the configure request and back
@@ -542,6 +544,7 @@ static int lcp_handle_configure_nak(ppp_protocol_t *self, const lcp_pkt_t *pkt,
         case PPP_STATE_OPENED:
             /* XXXX: This layer down. */
             /* Fall through... */
+            __fallthrough;
 
         case PPP_STATE_REQUEST_SENT:
         case PPP_STATE_ACK_RECEIVED:
@@ -698,6 +701,7 @@ static int lcp_handle_configure_rej(ppp_protocol_t *self, const lcp_pkt_t *pkt,
         case PPP_STATE_OPENED:
             /* XXXX: This layer down. */
             /* Fall through... */
+            __fallthrough;
 
         case PPP_STATE_REQUEST_SENT:
         case PPP_STATE_ACK_RECEIVED:
diff --git a/kernel/net/net_tcp.c b/kernel/net/net_tcp.c
index 0dfd346..f16305b 100644
--- a/kernel/net/net_tcp.c
+++ b/kernel/net/net_tcp.c
@@ -388,6 +388,7 @@ retry:
             }
 
             /* Fall through... */
+            __fallthrough;
 
         case TCP_STATE_SYN_RECEIVED:
             /* Don't have to worry about queued packets, since we don't allow
@@ -2009,6 +2010,7 @@ static short net_tcp_poll(net_socket_t *hnd, short events) {
             if(sock->data.sndbuf_cur_sz < sock->sndbuf_sz)
                 rv |= POLLWRNORM | POLLWRBAND;
             /* Fall through... */
+            __fallthrough;
 
         case TCP_STATE_TIME_WAIT:
         case TCP_STATE_CLOSING:
@@ -2020,6 +2022,7 @@ static short net_tcp_poll(net_socket_t *hnd, short events) {
             if(sock->data.sndbuf_cur_sz < sock->sndbuf_sz)
                 rv |= POLLWRNORM | POLLWRBAND;
             /* Fall through... */
+            __fallthrough;
 
         case TCP_STATE_FIN_WAIT_1:
         case TCP_STATE_FIN_WAIT_2:


hooks/post-receive
-- 
A pseudo Operating System for the Dreamcast.