|
 |
dd8478d |
From jwboyer@linux.vnet.ibm.com Tue Jun 28 08:47:09 2011
|
|
|
dd8478d |
Date: Tue, 28 Jun 2011 08:47:09 -0400
|
|
|
dd8478d |
From: Josh Boyer <jwboyer@linux.vnet.ibm.com>
|
|
|
dd8478d |
To: Jon Loeliger <jdl@jdl.com>
|
|
|
dd8478d |
Cc: linuxppc-dev@lists.ozlabs.org
|
|
|
dd8478d |
Subject: [PATCH] dtc: Remove unused check variable
|
|
|
dd8478d |
Message-ID: <20110628124709.GC10237@zod.rchland.ibm.com>
|
|
|
dd8478d |
MIME-Version: 1.0
|
|
|
dd8478d |
Content-Type: text/plain; charset=us-ascii
|
|
|
dd8478d |
Content-Disposition: inline
|
|
|
dd8478d |
User-Agent: Mutt/1.5.21 (2010-09-15)
|
|
|
dd8478d |
Status: RO
|
|
|
dd8478d |
Content-Length: 1315
|
|
|
dd8478d |
Lines: 49
|
|
|
dd8478d |
|
|
|
dd8478d |
Commit 376ab6f2 removed the old style check functionality from DTC,
|
|
|
dd8478d |
however the check option and variable were not removed. This leads to
|
|
|
dd8478d |
build failures when -Werror=unused-but-set-variable is specified:
|
|
|
dd8478d |
|
|
|
dd8478d |
dtc.c: In function 'main':
|
|
|
dd8478d |
dtc.c:102:17: error: variable 'check' set but not used [-Werror=unused-but-set-variable]
|
|
|
dd8478d |
cc1: all warnings being treated as errors
|
|
|
dd8478d |
make: *** [dtc.o] Error 1
|
|
|
dd8478d |
make: *** Waiting for unfinished jobs....
|
|
|
dd8478d |
|
|
|
dd8478d |
Remove the check variable.
|
|
|
dd8478d |
|
|
|
dd8478d |
Signed-off-by: Josh Boyer <jwboyer@linux.vnet.ibm.com>
|
|
|
dd8478d |
|
|
|
dd8478d |
---
|
|
|
dd8478d |
|
|
|
dd8478d |
t a/dtc.c b/dtc.c
|
|
|
dd8478d |
index cbc0193..15d2fc2 100644
|
|
|
dd8478d |
--- a/dtc.c
|
|
|
dd8478d |
+++ b/dtc.c
|
|
|
dd8478d |
@@ -99,7 +99,7 @@ int main(int argc, char *argv[])
|
|
|
dd8478d |
const char *inform = "dts";
|
|
|
dd8478d |
const char *outform = "dts";
|
|
|
dd8478d |
const char *outname = "-";
|
|
|
dd8478d |
- int force = 0, check = 0, sort = 0;
|
|
|
dd8478d |
+ int force = 0, sort = 0;
|
|
|
dd8478d |
const char *arg;
|
|
|
dd8478d |
int opt;
|
|
|
dd8478d |
FILE *outf = NULL;
|
|
|
dd8478d |
@@ -111,7 +111,7 @@ int main(int argc, char *argv[])
|
|
|
dd8478d |
minsize = 0;
|
|
|
dd8478d |
padsize = 0;
|
|
|
dd8478d |
|
|
|
dd8478d |
- while ((opt = getopt(argc, argv, "hI:O:o:V:R:S:p:fcqb:vH:s")) != EOF) {
|
|
|
dd8478d |
+ while ((opt = getopt(argc, argv, "hI:O:o:V:R:S:p:fqb:vH:s")) != EOF) {
|
|
|
dd8478d |
switch (opt) {
|
|
|
dd8478d |
case 'I':
|
|
|
dd8478d |
inform = optarg;
|
|
|
dd8478d |
@@ -137,9 +137,6 @@ int main(int argc, char *argv[])
|
|
|
dd8478d |
case 'f':
|
|
|
dd8478d |
force = 1;
|
|
|
dd8478d |
break;
|
|
|
dd8478d |
- case 'c':
|
|
|
dd8478d |
- check = 1;
|
|
|
dd8478d |
- break;
|
|
|
dd8478d |
case 'q':
|
|
|
dd8478d |
quiet++;
|
|
|
dd8478d |
break;
|
|
|
dd8478d |
|
|
|
dd8478d |
|