Re: ifconfig: rm not need variable noprint

2016-01-01 Thread Joerg Jung
On Wed, Dec 30, 2015 at 05:24:27PM +0100, Fabian Raetz wrote:
> Hi tech@,
> 
> this patch removes the 'noprint' variable which was added to ifconfig.c in 
> rev 1.216
> and is not in use since rev. 1.220.

Committed, thanks!
 
> Cheers,
> Fabian
> 
> 
> Index: sbin/ifconfig/ifconfig.c
> ===
> --- sbin/ifconfig/ifconfig.c.orig
> +++ sbin/ifconfig/ifconfig.c
> @@ -604,7 +604,6 @@ main(int argc, char *argv[])
>   int Cflag = 0;
>   int gflag = 0;
>   int i;
> - int noprint = 0;
>  
>   /* If no args at all, print all interfaces.  */
>   if (argc < 2) {
> @@ -760,7 +759,7 @@ nextarg:
>   argc--, argv++;
>   }
>  
> - if (argc == 0 && actions == 0 && !noprint) {
> + if (argc == 0 && actions == 0) {
>   printif(ifr.ifr_name, aflag ? ifaliases : 1);
>   exit(0);
>   }
> 



ifconfig: rm not need variable noprint

2015-12-30 Thread Fabian Raetz
Hi tech@,

this patch removes the 'noprint' variable which was added to ifconfig.c in rev 
1.216
and is not in use since rev. 1.220.

Cheers,
Fabian


Index: sbin/ifconfig/ifconfig.c
===
--- sbin/ifconfig/ifconfig.c.orig
+++ sbin/ifconfig/ifconfig.c
@@ -604,7 +604,6 @@ main(int argc, char *argv[])
int Cflag = 0;
int gflag = 0;
int i;
-   int noprint = 0;
 
/* If no args at all, print all interfaces.  */
if (argc < 2) {
@@ -760,7 +759,7 @@ nextarg:
argc--, argv++;
}
 
-   if (argc == 0 && actions == 0 && !noprint) {
+   if (argc == 0 && actions == 0) {
printif(ifr.ifr_name, aflag ? ifaliases : 1);
exit(0);
}