Review (part 3)
pipewirerecord.cpp
m_lastRecordThreadFinished
it's not really "last"
why are we using low level egl rather than Qt's
paramaters need eventually to be exposed. We don't necessarily want to expose too much to the user - but we should still have them in the lib.
what happens if we take longer to record than the framerate? Ideally we shoudl drop frames. I can't see any code doing this, but maybe it's automagic by pipewire?
CustomAVFrame can alloc be the ctor?
av_freep(&m_avFrame->data[0]);
is this safe?
auto avStream = avformat_new_stream(m_avFormatContext, nullptr);
leaks?