[MPlayer-dev-eng] Re: [PATCH] add support for big-endian PCM output
Grant Edwards
grante at visi.com
Thu Mar 22 22:17:49 CET 2007
On 2007-03-22, Dominik 'Rathann' Mierzejewski <dominik at rangers.eu.org> wrote:
> On Thursday, 22 March 2007 at 21:26, Grant Edwards wrote:
>> On 2007-03-22, Dominik 'Rathann' Mierzejewski <dominik at rangers.eu.org> wrote:
>>> On Wednesday, 21 March 2007 at 19:42, Giacomo Comes wrote:
>>>> On Wed, Mar 21, 2007 at 05:11:29PM +0000, Grant Edwards wrote:
>>>>> I needed to be able to encode to 16-bit uncompressed big-endian PCM.
>>>>> [...]
>>>>> Although it works I have a feeling that this wasn't really the
>>>>> "right" way to do it. If anybody would care to comment or
>>>>> suggest a better way to get big-endian PCM output please feel
>>>>> free to do so.
>>>>
>>>> I did create some time ago a patch that allow to use the option -format
>>>> togheter with -ao pcm:nowaveheader.
>>>> [...]
>>>>
>>>> Could you use my (simpler) patch for your purpose?
>>>
>>> Cosmetics. ;)
>>
>> Sorry, I don't understand.
>
> He's mixing cosmetic changes (adding a semicolon) with the
> rest of the patch (which is functional). IOW, I'm nitpicking.
So the (unasked) question you're answering is why his patch
wasn't accepted?
--
Grant Edwards grante Yow! I demand IMPUNITY!
at
visi.com
More information about the MPlayer-dev-eng
mailing list