[MPlayer-dev-eng] [PATCH] getch2() support for OS/2
KO Myung-Hun
komh at chollian.net
Wed Feb 20 15:27:12 CET 2008
Hi/2.
Reimar Döffinger wrote:
> Hello,
> On Wed, Feb 20, 2008 at 09:20:35PM +0900, KO Myung-Hun wrote:
>
>>> I do not like this renaming at all, this is still code that is full of
>>> Windows-API stuff. Sure it is used for OS/2 as well, but is is still the
>>> Windows API it uses, so the previous name still seems more correct.
>>>
>>>
>> I agree with you. But it seems that Alban and Diego want it.
>>
>
> Well, with the reasoning in that mail it actually makes sense.
> I would slightly prefer if whoever applies this first renames
> the mp_input_win32_slave_cmd_func and then applies the rest, but it's
> not too important.
>
It's important to me as patch committer. Because I should follow the
advice of maintainers and maintainers should coincide in opinion so that
the patch can be accepted. BTW if they have different opinions from one
another, it is too hard that I decide to follow which opinion.
So do you have a objection to the patch ?
--
KO Myung-Hun
Using Mozilla SeaMonkey 1.1.8
Under OS/2 Warp 4 for Korean with FixPak #15
On AMD ThunderBird 750 MHz with 512 MB RAM
Korean OS/2 User Community : http://www.ecomstation.co.kr
More information about the MPlayer-dev-eng
mailing list