Use the dedicated player callback. Don't wait for user input.

Line-based buffering was enabled in 19ea8feb.

(Beware the locale decimal separator.)

Not sure what it meant, probably a short for status. But since status
is printed asynchronously nowadays, it serves no purpuse.

The player lock is already serialising all the code paths.

We already have the player state callback.

Make sure the state of the player when the interface state is printed
before any other state update.

(no functional changes)

Zhao Zhili authored Oct 04, 2020 and Steve Lhomme committed Oct 16, 2020

1. FF_THREAD_FRAME is in the FFmpeg code base since 2011.

2. codec/avcodec/video.c and codec/avcodec/directx_va.c use
FF_THREAD_FRAME directly without check.

3. FF_THREAD_FRAME is a thread type, it's not a multithreads decoding
support flag.

Signed-off-by: Steve Lhomme <robux4@ycbcr.xyz>

Zhao Zhili authored Oct 01, 2020 and Steve Lhomme committed Oct 16, 2020

Signed-off-by: Steve Lhomme <robux4@ycbcr.xyz>

Zhao Zhili authored Oct 04, 2020 and Steve Lhomme committed Oct 16, 2020

It's useless to set struct fields to NULL just before free the struct.

Signed-off-by: Steve Lhomme <robux4@ycbcr.xyz>