.

.

Remove spurious space.

(tail_bytes) [from_start]: For regular files, seek
relative to the initial input file pointer position, not
necessarily from the beginning of the file.
[!from_start]: Don't back up past the initial position of the
input file pointer.
(tail_lines): Call file_lines only if FD refers to a regular file
with its file pointer positioned at beginning of file.  Otherwise,
call pipe_lines.  This is a kludge.  Once there's a decent test
suite, fix this properly.
Before, (echo 1; echo 2) > k; sh -c 'read x; tail' < k
would output both lines of the input file even though the first had
already been read.  Reported by John Roll (john@panic.harvard.edu).

(have_read_stdin): New global variable.
(md5_file, md5_check): Set it.
(main): Use it.

[OPENOPTS]: Depend explicitly on BINARY.
(md5_file): Take a new parameter, MD5_RESULT, and no longer
generate output.
(md5_check): Invoke md5_file instead of calling fopen directly.
When giving a diagnostic for a line with invalid format, also report
the line number.
(main): Generate output after a successful md5_file call.
[in many places]: Upon detection of an error, rather than exiting
immediately, issue a diagnostic, note that an error occurred and
exit later.

(main): When too few arguments are given, don't just
issue the usage message; also report why.

[_LIBC || STDC_HEADERS] (TOLOWER): Define to tolower.

(md5_check): New function -- extracted from main.

reformat comment

.

(md5_file): New function -- extracted from main.
(main): Call the new function instead of doing all that in an if stmt.
Always use "%s" format rather than raw filename as format argument
in printf-style functions like error.  Otherwise, filenames containing
e.g. `%s' lose.
Remove all traces of `old format'.  There is only one format now.
The compatible one.

(md5_stream): Update prototype.
[__P]: Define macro.

(md5_stream): Check for read failure and return
indication of success rather than second argument.

Use EXIT_FAILURE and EXIT_SUCCESS.
(main): Report an error if md5_stream fails.  Otherwise, running
`md5sum dir-on-mounted-filesystem' always reported the checksum
for an empty file.  Now it gets the `is a directory' error.
Greg McGary reported that the released version got stuck in an
infinite loop with such arguments.

(split_3): Accept only old format.
Properly handle file names with leading and trailing white space.

more comment tweaks

Correct typo in Copyright.
Reformat paragraph so line with new FSF address fits in 80 columns.

(usage): Move it to precede all other functions.
Fiddle comments.

New version from Drepper.

.

.

.

.

Add #else block after #ifdef VMS.
(main): Rename local variable.
Add FIXME: allow newlines in filenames.
Detect and report failed fclose calls on stdout and checkfile_stream.

(build_spec_list): Declare CLOSING_DELIM_IDX and CLOSING_BRACKET_IDX to
be of type size_t, not int.
(main): Declare NR and CHARS_READ to be of type long, not int.

(unqote): Comment out unreachable break stmts.
(append_range): Declare FIRST and LAST parameters to be unsigned int,
not size_t.
(find_closing_delim): Clean up interface, separating boolean success
indicator and index.
(find_bracketed_repeat): Likewise.
(build_spec_list): Adapt to use new interfaces.

(main): Comment out unreachable break stmt.

(dump): Convert to use smaller -- but seemingly equivalent -- patch.

Reorder functions so main is last, so no forward dcls are needed.

(tail): Explicitly cast to long the N_UNITS argumennt in
call to tail_lines.  Otherwise, NetBSD lost because N_UNITS (of type
off_t, a 64-bit type on NetBSD) wasn't converted to the target type,
long.  Reported by Dan Hagerty (hag@gnu.ai.it.edu).

(main): Add UCHAR casts.

.

(main): When using obsolescent +pos -pos syntax, allow
omission of field spec only when character offset is specified.
Otherwise, -. +. would be accepted.

(main): Allow field spec, M, to be omitted in +M.N or -M.N
position specification.  Omitting M is equivalent to specifying `0'
for M.  From Rick Sladkey (jrs@world.std.com).

.

(dump): Correct loop-termination criterion.
Before, running `printf 1234| ./od --width=4 --read-bytes=4'
printed output for 8 bytes -- the last four were garbage.
This happened only when the dump limit, N, was specified (with
--read-bytes=N) and N was a multiple of bytes_per_block
(usually 16, but 4 in this example).  From Andreas Schwab.

Remove trailing white space.

(od): Remove space beteen -w and optional argument.

.

.

(split_3): Rewrite to parse Plumb/Lankester format as well.
(main): Write (de facto) standard Plumb/Lankester format.
New option: --quiet.
Check option no longer takes an argument.
When checking, exit status reflects success.

Remove unused dcl of isnumber.

(parse_group): Use xstrtoul instead of isnumber; the latter would
silently overflow, accepting a group id larger than INT_MAX.
(isnumber): Remove now-unused static function.
(change_file_group) [MAXUID]: Give a more descriptive message
when numeric group id is larger than MAXUID.

merge with 1.12a

.

Don't conditionalize use of "inline".  Autoconf takes care of defining
it away for compilers that don't grok it.

.

.

Undefine min and/or max before defining to avoid redefinition warning
on systems.

(read_input): Rename paramater MAX to MAX_N_BYTES.  To avoid potential
conflict with max macro from some system's header files and bad pre-ANSI
compilers.

[uint32]: When testing for 32-bit arithmetic type,
also check `unsigned long', as a concession to systems with
int's smaller than 32 bits.

(usage): Fix typo.

(main): Include program name in --version output.

[join]: Redefine.  Otherwise, systems (Unicos for one) with
headers that have a different prototype for `join' get an error.

.

[word]: Redefine.  Otherwise, systems (Unicos for one) with
headers that define `word' to be a type get syntax errors because
of the variable by the same name.

(interrupt_handler): Declare to have a single integer
parameter.  Otherwise, some compilers failed with a type mismatch
error in sa_handler assignment.

(main): Remove spurious colon in getopt spec string.
From Ken Pizzini <kenp@spry.com>.

(md5_file): Initialize two elements of LEN portably,
rather than with ANSI aggregate initialization.  Reported by
Edzer Pebesma <Edzer.Pebesma@rivm.nl>.

merge with 1.12

.

.

(process_buffer): Copy values from struct into
individual local variables before performing computation on them,
then copy results back into returned struct.  Solely for better
optimization by compilers that can't keep struct members in
registers.  From Ulrich Drepper.

.

.

.

(split_3): Unprotoize.
(hex_digits): Unprotoize.
(main): Interpret `no file args' as request to read from stdin.
(usage): Reflect that change.

Half-assed patch from Karl -- probably need to change `file name' to `filename'.

.

Remove unnecessary uses of `defined' in #if* tests.
Don't use #elif.  Some older compilers don't grok it.
(split_3): New function to parse out sum, flag, and filename
when reading check file.
(hex_digits): Remove length parameter since string parameter is
now nul-terminated.
(main): Don't allocate separate arrays for filename and sum
when checking.  Get pointers into line buffer with split_3 instead
of using sscanf.

Don't include values.h.
[__P]: Undefine it.

[UNIX || unix]: Also test __UNIX__, __unix__,
and _POSIX_VERSION.  Reported by Arne H. Juul.

.

merge with 1.11.5b

patch from drepper

Alloca fix for ConvexOS-11.0.

Use void* alloca, not char*.  The latter lost on convexOS.

md5sum docs from Ulrich.

(main): Rename `char binary' to type_flag to avoid confusion with `int binary'.

Don't use #elif.  Some old C compilers don't recognize it.
(hex_digits): New function.
(result): Give ctx parameter the `const' attribute.
(usage): Rewrite to make it clearer that there are three modes.
(main): Process all options before generating any output.
In particular, collect all --string options rather than operating
on them as they are encountered.

Allow --check=-.  Before it tried to open the file named `-'.

Detect bogus input when --check'ing: non-hex, non-[tb] flag.

Map to lower case all hex digits read from check file.

.

.

[NDEBUG]: Define it to disable assertions.
(is_char_class_member): Set a variable in each branch of switch stmt
and return that value after the switch (rather than returning directly
from every branch).
(unquote): More int->size_t changes.
(build_spec_list): More int->size_t changes.
Be very careful about comparison now that variables are unsigned:
Use i + 2 < len rather than i < len - 2.  The latter failed for len < 2.

.

(xfclose): Don't try to flush stdin, only stdout.
Otherwise, at least Ultrix-4.3's fflush would return EOF.
Reported by Jim Blandy (jimb@cyclic.com).

.

[struct Spec_list] (indefinite_repeat_element): New member.
Use size_t rather than int or unsigned long where reasonable.
(get_spec_status): Make interface cleaner.
(get_s1_spec_status): New function.
(get_s2_spec_status): New function.
(validate): Use new functions.

Indent some things.
(long_options): Add NULL entry.  From Uli.
Touch up some comments.
(process_buffer): Bracket definition of OP with do...while(0) so stmts can be
semicolon terminated.

(non_neg_strtol): Don't compare signed and unsigned.

[!__GNUC__]: Define INLINE to /* empty */.

.

.

Remove now-obsolete comment and leading space.

Remove now-obsolete comment and leading space.

.

Use `invalid', not `illegal' in error message.

Remove trailing white space.

[SWAP]: Parenthesize uses of macro argument.
Don't use autoconf macros that require running test programs.

From Ulrich Drepper.

(look_up_char_class, append_char_class, append_equiv_class): Use const
attribute when appropriate.

(Convert): Use 2037 as threshold, not 1999.
Otherwise years after 1999 are treated as invalid.
From Andreas Schwab.