tool_operate: fix misplaced initialization of orig_noprogress
authorKamil Dudka <kdudka@redhat.com>
Sat, 21 Jul 2012 23:55:45 +0000 (01:55 +0200)
committerKamil Dudka <kdudka@redhat.com>
Sun, 22 Jul 2012 00:06:22 +0000 (02:06 +0200)
... and orig_isatty which caused --silent to be entirely ignored in case
the standard output was redirected to a file!

src/tool_operate.c

index 7c0b8b5fec0357f64b092ae35c6b4aac580a28ca..f4abc3b75952b2f2eb4d4d38dd003776903a4dd0 100644 (file)
@@ -140,8 +140,8 @@ int operate(struct Configurable *config, int argc, argv_item_t argv[])
   int res = 0;
   int i;
 
-  bool orig_noprogress = config->noprogress;
-  bool orig_isatty = config->isatty;
+  bool orig_noprogress;
+  bool orig_isatty;
 
   errorbuffer[0] = '\0';
   /* default headers output stream is stdout */
@@ -396,6 +396,10 @@ int operate(struct Configurable *config, int argc, argv_item_t argv[])
     }
   }
 
+  /* save the values of noprogress and isatty to restore them later on */
+  orig_noprogress = config->noprogress;
+  orig_isatty = config->isatty;
+
   /*
   ** Nested loops start here.
   */