Re: [PATCH] cal: drop trailing blank line

[Bruce Dubbs <bruce.dubbs@gmail.com>]

Mike Frysinger wrote:
> There's no history to be found behind this, and the comment in the
> code indicates no one else knows, and it makes me angry :).
>
> Signed-off-by: Mike Frysinger <vapier@gentoo.org>
> ---
>   misc-utils/cal.c | 3 ---
>   1 file changed, 3 deletions(-)
>
> diff --git a/misc-utils/cal.c b/misc-utils/cal.c
> index 3e20530..479c967 100644
> --- a/misc-utils/cal.c
> +++ b/misc-utils/cal.c
> @@ -767,9 +767,6 @@ static void yearly(const struct cal_control *ctl)
>   		my_putstring("\n\n");
>   	}
>   	monthly(ctl);
> -
> -	/* Is empty line at the end year output really needed? */
> -	my_putstring("\n");
>   }
>
>   /*

Personally I like the blank line. It sets off the cal output from the 
shell prompt making it easier to read.  It is basically adding the 
equivalent of the css property padding-bottom="1em".

I note that if doing a 'cal -y' that there is also a blank line between 
the year and February.  That should stay also.

   -- Bruce Dubbs
      linuxfromscratch.org