`api2-decode-encode-video.cpp` example and correct stream tear-down sequence about avcpp HOT 3 CLOSED

This is what I do to manually count the frames w/ resynchronization on the input stream when it is possible:
master...mmomtchev:avcpp:example-pts-dts

from avcpp.

There are two separate issues that prevent the clean teardown

First one:

There are two possible sources for the timebase here - either from the current frame - which works for all frames except the last one where the encoder is called with an empty frame - either from m_stream when m_stream.isValid(). However the encoder is not constructed with the stream - only the decoder is. The encoder must use its own timebase field here - and all three options must be mutually exclusive - currently a valid stream will replace the timebase of the frame.

Second one:

This should be:
if (ret < 0 && ret != AVERROR(EAGAIN) && ret != AVERROR_EOF)
just like in the function above and below it

from avcpp.

Thanks for reporting and fix!

from avcpp.