diff --git a/ChangeLog b/ChangeLog index 6f6a49a51a..5cb6e77a19 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,11 @@ 2002-04-18 Ulrich Drepper + * iconv/skeleton.c (RESET_INPUT_BUFFER): Replace ifs with #ifs to + avoid compiler warnings. + + * locale/programs/localedef.c (main): Always call + construct_output_path. + * iconv/Makefile (CFLAGS-simple-hash.c): Define to find hashval.h. * locale/programs/locfile.c (to_archive): New variable. To collect diff --git a/iconv/skeleton.c b/iconv/skeleton.c index a42c02807a..9a609c972e 100644 --- a/iconv/skeleton.c +++ b/iconv/skeleton.c @@ -1,5 +1,5 @@ /* Skeleton for a conversion module. - Copyright (C) 1998, 1999, 2000, 2001 Free Software Foundation, Inc. + Copyright (C) 1998, 1999, 2000, 2001, 2002 Free Software Foundation, Inc. This file is part of the GNU C Library. Contributed by Ulrich Drepper , 1998. @@ -193,15 +193,20 @@ static int to_object; character set we can define RESET_INPUT_BUFFER in a very fast way. */ #if !defined RESET_INPUT_BUFFER && !defined SAVE_RESET_STATE # if MIN_NEEDED_FROM == MAX_NEEDED_FROM && MIN_NEEDED_TO == MAX_NEEDED_TO -/* We have to use these `if's here since the compiler cannot know that - (outbuf - outerr) is always divisible by MIN_NEEDED_TO. */ -# define RESET_INPUT_BUFFER \ - if (MIN_NEEDED_FROM % MIN_NEEDED_TO == 0) \ - *inptrp -= (outbuf - outerr) * (MIN_NEEDED_FROM / MIN_NEEDED_TO); \ - else if (MIN_NEEDED_TO % MIN_NEEDED_FROM == 0) \ - *inptrp -= (outbuf - outerr) / (MIN_NEEDED_TO / MIN_NEEDED_FROM); \ - else \ - *inptrp -= ((outbuf - outerr) / MIN_NEEDED_TO) * MIN_NEEDED_FROM +/* We have to use these `#if's here since the compiler cannot know that + (outbuf - outerr) is always divisible by MIN_NEEDED_TO. We have to + use preprocessor arithmetic and no C code because gcc 3.2 complains + about division by zero even in obviously dead code. */ +# if MIN_NEEDED_FROM % MIN_NEEDED_TO == 0 +# define RESET_INPUT_BUFFER \ + *inptrp -= (outbuf - outerr) * (MIN_NEEDED_FROM / MIN_NEEDED_TO) +# elif MIN_NEEDED_TO % MIN_NEEDED_FROM == 0 +# define RESET_INPUT_BUFFER \ + *inptrp -= (outbuf - outerr) / (MIN_NEEDED_TO / MIN_NEEDED_FROM) +# else +# define RESET_INPUT_BUFFER \ + *inptrp -= ((outbuf - outerr) / MIN_NEEDED_TO) * MIN_NEEDED_FROM +# endif # endif #endif diff --git a/locale/programs/localedef.c b/locale/programs/localedef.c index 526f2025e7..ba8572e9e5 100644 --- a/locale/programs/localedef.c +++ b/locale/programs/localedef.c @@ -223,18 +223,10 @@ main (int argc, char *argv[]) /* The parameter describes the output path of the constructed files. If the described files cannot be written return a NULL pointer. */ - if (no_archive) - { - output_path = construct_output_path (argv[remaining]); - if (output_path == NULL) - error (4, errno, _("cannot create directory for output files")); - cannot_write_why = errno; - } - else - { - output_path = NULL; - cannot_write_why = 0; /* Just to shut the compiler up. */ - } + output_path = construct_output_path (argv[remaining]); + if (output_path == NULL && ! no_archive) + error (4, errno, _("cannot create directory for output files")); + cannot_write_why = errno; /* Now that the parameters are processed we have to reset the local ctype locale. (P1003.2 4.35.5.2) */