Signed-off-by: Anton Khirnov <anton@khirnov.net>

Signed-off-by: Mans Rullgard <mans@mansr.com>

It's supposed to be called only from (de)muxers.

Trivial replacements with sed are done in this commit:
sed 's/av_new_stream(\([^)]*\), 0)/avformat_new_stream(\1, NULL)/'

AVFormatParameters are converted into corresponding private options in
av_open_input_file/stream() compat wrappers, so accessing them from
demuxers is redundant.

Show the invalid string in the error message.

While at it also prefer "Could not" over "Couldn't", plain forms are
preferred over contractions (simplify readability, especially for non
English-savvy people).
Signed-off-by: Anton Khirnov <anton@khirnov.net>

The variable is used for containing the parsed value of framerate,
using a lexically consistent name eases readability/understanding.
Signed-off-by: Anton Khirnov <anton@khirnov.net>

It's simpler and less error-prone.

Fixes some memleaks along the way.

Signed-off-by: Anton Khirnov <anton@khirnov.net>