Source: nn
Version: 6.7.3-14
Severity: normal
Tags: patch

Dear Maintainer,

>From b400f526f90b73bf6eb0127f528a9c806e5d854e Mon Sep 17 00:00:00 2001
>From: Bjarni Ingi Gislason <bjarn...@rhi.hi.is>
>Date: Thu, 4 Mar 2021 02:44:37 +0000
>Subject: [PATCH] nn.c: silence warnings about a possible fall through

  Add __attribute__ ((fallthrough));

  Add "break" after an "nn_exit()" as the compiler can't know,
that the next code is never reached.

Signed-off-by: Bjarni Ingi Gislason <bjarn...@rhi.hi.is>
---
 nn.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/nn.c b/nn.c
index 7c77057..7bc2779 100644
--- a/nn.c
+++ b/nn.c
@@ -303,6 +303,7 @@ read_news(flag_type access_mode, char *mask)
                        break;
                    case 0:
                        gh = group_sequence;
+                       __attribute__ ((fallthrough));
                    default:
                        after_loop = prev;
                        continue;
@@ -1107,6 +1108,7 @@ nnspew:
                    prt_unread((char *) NULL);
            }
            nn_exit(unread_articles ? 0 : 99);
+           break; /* added to silence warning about possible fall through */
            /* NOTREACHED */
 
        case I_AM_TIDY:
-- 
2.30.1



-- System Information:
Debian Release: bullseye/sid
  APT prefers testing
  APT policy: (500, 'testing'), (500, 'stable')
Architecture: amd64 (x86_64)

Kernel: Linux 5.10.9-1 (SMP w/2 CPU threads)
Locale: LANG=is_IS.iso88591, LC_CTYPE=is_IS.iso88591 (charmap=ISO-8859-1), 
LANGUAGE not set
Shell: /bin/sh linked to /bin/dash
Init: sysvinit (via /sbin/init)

-- debconf information excluded

-- 
Bjarni I. Gislason

Reply via email to