[MPlayer-dev-eng] [PATCH] Fix capture width/height selection in tvi_dshow

[Vladimir Voroshilov]

2013/5/4 Reimar Döffinger <Reimar.Doeffinger at gmx.de>:
>
>
> On 04.05.2013, at 08:17, Vladimir Voroshilov <voroshil at gmail.com> wrote:
>
>> Hi again, All
>>
>>> [...]
>>>
>>> I think the behaviour would be more consistent if this was a function
>>> and also used from SET_WIDTH and SET_HEIGHT, in the way that
>>> SET_WIDTH would call it with (new width, old height)
>>>
>>> [...]
>>>
>>> You can probably even put all of this code into that function, and
>>> thus avoid having things like the Min/Max size validation duplicated
>>> between those different SET_ functions.
>>> Or alternatively, if the SET_WIDTH and SET_HEIGHT are actually
>>> not used at all any more after this change (I didn't double-check
>>> how all this works) then just removing them might be an option, too.
>>> Still, I am a fan of using lots of functions, they tend to make
>>> the code much easier to understand.
>>
>> This is second version of patch.
>> As suggested by Reimar i've implemented changing width/height via
>> separated routine.
>> Moreover i've also extracted checking width/height into separate
>> routines - this slighty imporoved routine readability.
>
> I didn't review it carefully, but I think it looks fairly nice, great work!

Great! Fill free to apply then, if nobody says against (i did not
restore entire build environment at my pc, so still don't have
properly configured svn here).


-- 
Vladimir Voroshilov     mailto:voroshil at gmail.com
Jabber: voroshil at gmail.com
ICQ: 95587719