[PATCH] fix English in top.c

walter harms wharms at bfs.de
Fri Feb 10 16:41:14 UTC 2006


hi denis ,

"/proc" should be ok since perror will add strerror(errno).
an other question: does it make sense to use a #define ?

re,
  walter


Denis Vlasenko wrote:
> Hello.
> 
> Patch changes "/proc not be mounted?" into "is /proc mounted?"
> 
> --
> vda
> 
> 
> ------------------------------------------------------------------------
> 
> diff -urpN busybox.org/procps/top.c busybox.fix/procps/top.c
> --- busybox.org/procps/top.c	Thu Feb  9 11:40:07 2006
> +++ busybox.fix/procps/top.c	Fri Feb 10 10:30:10 2006
> @@ -132,7 +132,7 @@ static unsigned long Hertz;
>  
>  #define FILE_TO_BUF(filename, fd) do{                           \
>      if (fd == -1 && (fd = open(filename, O_RDONLY)) == -1) {    \
> -	bb_perror_msg_and_die("/proc not be mounted?");            \
> +	bb_perror_msg_and_die("is /proc mounted?");            \
>      }                                                      \
>      lseek(fd, 0L, SEEK_SET);                                    \
>      if ((local_n = read(fd, buf, sizeof buf - 1)) < 0) {        \
> 
> 
> ------------------------------------------------------------------------
> 
> _______________________________________________
> busybox mailing list
> busybox at busybox.net
> http://busybox.net/cgi-bin/mailman/listinfo/busybox



More information about the busybox mailing list