Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 29 Jan 2020 17:01:02 +0000 (UTC)
From:      Dimitry Andric <dim@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-projects@freebsd.org
Subject:   svn commit: r357260 - projects/clang1000-import/usr.bin/procstat
Message-ID:  <202001291701.00TH12Kn033954@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: dim
Date: Wed Jan 29 17:01:02 2020
New Revision: 357260
URL: https://svnweb.freebsd.org/changeset/base/357260

Log:
  Fix the following -Werror warning from clang 10.0.0 in procstat:
  
  usr.bin/procstat/procstat_sigs.c:79:3: error: misleading indentation; statement is not part of the previous 'else' [-Werror,-Wmisleading-indentation]
                  xo_close_container(name);
                  ^
  usr.bin/procstat/procstat_sigs.c:77:4: note: previous statement is here
          } else
            ^
  
  The intent was to group the xo_close_container() call to the previous
  snprintf() call.
  
  MFC after:	3 days

Modified:
  projects/clang1000-import/usr.bin/procstat/procstat_sigs.c

Modified: projects/clang1000-import/usr.bin/procstat/procstat_sigs.c
==============================================================================
--- projects/clang1000-import/usr.bin/procstat/procstat_sigs.c	Wed Jan 29 16:57:55 2020	(r357259)
+++ projects/clang1000-import/usr.bin/procstat/procstat_sigs.c	Wed Jan 29 17:01:02 2020	(r357260)
@@ -74,9 +74,10 @@ procstat_close_signame(int sig)
 		for (i = 0; name[i] != 0; i++)
 			name[i] = toupper(name[i]);
 		xo_close_container(name);
-	} else
+	} else {
 		snprintf(name, 12, "%d", sig);
 		xo_close_container(name);
+	}
 }
 
 static void



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202001291701.00TH12Kn033954>