valbok / QtAVPlayer

Free and open-source Qt Media Player library based on FFmpeg, for Linux, Windows, macOS, iOS and Android.
MIT License
282 stars 54 forks source link

cppcheck++ : add missing constructors explicit. #455

Closed cgilles closed 6 months ago

cgilles commented 6 months ago

cppcheck report missing explicit constructors. This PR fix this.

cgilles commented 6 months ago

Hum, after double checking runtime, i seen that explicit constructors usage break video decoding. I don't know why yet. So ignore this PR, i will cancel it

valbok commented 6 months ago

right, btw QAVFrame is a base class for video and audio frames. And it is possible to QAVVideoFrame by a copy using QAVFrame.

QAVVideoFrame <=> QAVFrame