[FFmpeg-devel] [PATCH] RoQ video encoder

[Eric Lasota]

Michael Niedermayer wrote:
> it may or may not end up the same as your sorting code, i dont know ...
I just did some verification, and the rate distortion on all 
non-selected possibilities is always worse than the last entry to be 
accepted.
> but a single pass over the possibilities, with no sorting, no lists, ...
> beats your code by a quite large amount in terms of simplicity
I removed the sorting from the plist builder, including the two sort 
functions, it now consumes 130 fewer lines of code.  I sent Vitor the 
replacement.  The number of lines specific to this method, including the 
existing code, is around 75.

Keeping the progressive reducer in is a good idea as the quality factor 
needed to maintain a bitrate gets extremely erratic, and it may be 
neccessary to cap frame sizes, although I am admittedly a newbie when it 
comes to rate control.