[MPlayer-dev-eng] [PATCH] split filter_n_bytes function a bit
Uoti Urpala
uoti.urpala at pp1.inet.fi
Sat Nov 3 23:24:00 CET 2007
On Sat, 2007-11-03 at 15:16 +0100, Reimar Döffinger wrote:
> Hello,
> I for some reason didn't like that it does two mostly unrelated things:
> decoding to fill the in buffer with data and passing it through the
> filter chain.
> Attached patch patch would split it. Objections?
There are phases in the functionality but so there are in most
functions. IMO splitting the flow of the function into dozen-line
subfunctions is harmful.
Additionally your patch breaks the EOF/error logic.
More information about the MPlayer-dev-eng
mailing list