On Mon, Sep 10, 2007 at 02:45:37PM +0800, Li Zefan wrote:
> The function warn() in rev.c is actually duplicate code, 
> so here we remove it.
> 
> Signed-off-by: Li Zefan <[EMAIL PROTECTED]>


On Tue, Sep 11, 2007 at 01:48:38PM +0800, Li Zefan wrote:
> -     fprintf(stderr, _("Cannot open \"%s\" for read\n"), filename);
> -     perror("");
> -     exit(1);
> +     err(EXIT_FAILURE, "Cannot open %s for read", filename);

 You forgot _() and initialize the global __progname variable. See
 errs.h and for example cal.c or column.c.


 Well, the best way how fix the errs.h junk is delete it and use
 err.h from glibc everywhere in util-linux. For more details:

 http://www.gnu.org/software/libc/manual/html_node/Error-Messages.html

        Karel

-- 
 Karel Zak  <[EMAIL PROTECTED]>
-
To unsubscribe from this list: send the line "unsubscribe util-linux-ng" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to