>> + // for managing AVFilterPad::request_frame and >> AVFilterPad::filter_samples >> nit: AVFilterPad.request_frame and AVFilterPad.filter_samples > > > done, although it looks strange to me. Is :: scope operator not part of C > language? No, the scope resolution operator is C++ only.