Signed-off-by: Stefano Sabatini <stefasab@gmail.com>

Signed-off-by: Stefano Sabatini <stefasab@gmail.com>

The index for the binary_unit_prefixes array is expected to be positive,
so avoid to use negative indexes for accessing it.

Second attempt at fixing ticket #921.

There is no guarantee that the casted double which is assigned to the
variable will be contained in an int (also if it is almost sure for most
non-alien architectures).

Use long long int to contain such values instead of an int, which is
required to contain at least 64 bits, so it is guaranteed to contain also
int64_t values, which are used by some fields.

In particular, should fix trac ticket #921.

Consistent with ffmpeg and ffplay.

Call av_free_packets() at each demuxing loop iteration, not only when
decoding.

X-ColaCount: +10l

av_destruct_packet() always frees the packet data even when the demuxer
is going to re-use it, thus causing crashes when decoding audio
frames (as implemented in a pending patch).

av_free_packet() is used instead, as it allows each demuxer to set the
right packet data releasing mechanism through the pkt->destruct callback.

This is required as some section names may contain non-ASCII characters
(e.g. '_').

Print a "\n" at the end of each section, also print the section name in
the section print function, print the chapter name only in case the
chapter contains multiple entries.

Increase textual output readability - different sections can be
distinguished more easily.

Clarify/generalize indent logic.

The passed argument is supposed to be the chapter/section name, rather
than the header/footer. Less confusing.

Possibly faster/cleaner.

Suggested-By: Clément Bœsch <ubitux@gmail.com>

Add -show_frames option to ffprobe.

Partially based on the work of Thomas Kuehnel <kuehnelth@googlemail.com>
for SOCIS 2011.

The wicked idea of creating a special "packets_and_frames" container for
structured formats (JSON and XML) comes from Clément.

Simplify printing from the main() context, in case probe_file() is not
called, as required by the pending -show_version option.

Simplify pending changes, as the writer context will be used in the
main() routine.

Avoid printing on stdout when show_usage is used in an error message.

Fix crash.

This is required by a pending patch, also fixes a memleak due to the
writer context not being closed in case of open_input_file() failure.

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>