Re: [PATCH] automount: Drop redundant \n in logerr()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, 2016-07-04 at 19:48 +0900, Tomohiro Kusumi wrote:
> Line break isn't necessary from the way logerr (and logmsg) is implemented,
> and these two probably aren't intentional.

Added to the list, same as the previous in terms of pushing it to the repo.

> 
> Signed-off-by: Tomohiro Kusumi <kusumi.tomohiro@xxxxxxxxx>
> ---
>  daemon/module.c | 2 +-
>  lib/nss_parse.y | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/daemon/module.c b/daemon/module.c
> index bed8f7a..db01db8 100644
> --- a/daemon/module.c
> +++ b/daemon/module.c
> @@ -34,7 +34,7 @@ int load_autofs4_module(void)
>  	 */
>  	fp = open_fopen_r("/proc/filesystems");
>  	if (!fp) {
> -		logerr("cannot open /proc/filesystems\n");
> +		logerr("cannot open /proc/filesystems");
>  		return 0;
>  	}
>  
> diff --git a/lib/nss_parse.y b/lib/nss_parse.y
> index 055e9d7..0721ba1 100644
> --- a/lib/nss_parse.y
> +++ b/lib/nss_parse.y
> @@ -169,7 +169,7 @@ int nsswitch_parse(struct list_head *list)
>  
>  	nsswitch = open_fopen_r(NSSWITCH_FILE);
>  	if (!nsswitch) {
> -		logerr("couldn't open %s\n", NSSWITCH_FILE);
> +		logerr("couldn't open %s", NSSWITCH_FILE);
>  		return 1;
>  	}
>  
--
To unsubscribe from this list: send the line "unsubscribe autofs" in



[Index of Archives]     [Linux Filesystem Development]     [Linux Ext4]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux