Allow to skip section name at the begin of each line. Possibly simplify
output.

Factorize writer class definition.

Remove pointless default_print_footer() and
default_print_chapter_header() callbacks, also avoid to print an
empty line when noprint_wrappers is selected.

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

This reduces from 3 to 2 messages for the same syntax error in ffprobe,
and from 4 to 3 in filters.

Add "data" and "extradata" attributes to the schema.
Add "tag" element in "streams".
Print extradata before tags to avoid closing the element.

Fix trac ticket #1588.

This is consistent with what av_dump_format() does.

In particular, fix trac ticket #1568.

Fix crash when a stream codec is unknown, in particular fix trac
ticket #1243.

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

Improve overall consistency, allow some factorization.

A duration value is undefined when is 0, take into consideration this
fact when printing optional fields.

This is for interoperability purpose with the fork.

Make writer_print_chapter_header() set the internal values *before* the
private writer callback is called, otherwise the accessed internal values
will be wrong.

Fix regression introduced in fa6cb2fd.

Found-By: Daemon404

Liberally based on the work of Luca Barbato <lu_zero@gentoo.org>, done
for libav/avprobe.

Move the field "multiple_entries" from the JSON writer private context to
WriterContext, and rename it "multiple_sections".

This allows to factorize the code with the pending INI format patch.

Avoid to specify implementation details, better to check the code for
that.

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

The special-case behavior may complicate parsing when the
-show_format_entry option is used programmatically in a script.

The option default=nk=1 achieves the same purpose, if the objective is to
skip printing the single field key.

Fix warning:
ffprobe.c: In function ‘probe_file’:
ffprobe.c:1576:14: warning: unused variable ‘i’ [-Wunused-variable]

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

Also remove the unused print_fmt_opt() in the process.

Help simplifying parsing in certain cases.

The option is useful for simplifying parsing.

Also use the new option in fate, in order to fix the regression
introduced by the previous commit.

Do not skip chapter header/footer printing, which breaks the parsing for
most writers.

The function was introduced in 653d117c, but is not required.