[vlc-devel] Reworking OpenSLES audio output

Hugo Beauzee-Luyssen beauze.h at gmail.com
Fri May 6 13:25:08 CEST 2011


On Fri, May 6, 2011 at 1:09 PM, Hugo Beauzee-Luyssen <beauze.h at gmail.com> wrote:
> On Fri, May 6, 2011 at 1:01 PM, Hugo Beauzee-Luyssen <beauze.h at gmail.com> wrote:
>> On Wed, May 4, 2011 at 1:32 PM, Hugo Beauzee-Luyssen <beauze.h at gmail.com> wrote:
>>> On Tue, May 3, 2011 at 10:15 PM, Dominique Martinet
>>> <asmadeus at codewreck.org> wrote:
>>>> Hi,
>>>>
>>>> Well, you didn't run a diff so you probably didn't notice, but most of
>>>> the diff is lines with just spaces added around brakes and
>>>> parenthesis, indentation padding, etc... Doesn't mean I was right with
>>>> my conventions, just that it's hard to tell the real changes quickly.
>>>> If it ever gets commited, it might be better to split it into two
>>>> patches (and find out if there is something about that here:
>>>> http://wiki.videolan.org/Code_Conventions )
>>>>
>>>
>>> Here is the current diff, as promised.
>>>
>>
>> Here is a new version, that works way better.
>> I guess there's still a lot to fix, but it seems like a better start.
>>
>> Regards,
>>
>>
>> --
>> Hugo Beauzée-Luyssen
>>
>
> Woops, some warnings removed.
>
> --
> Hugo Beauzée-Luyssen
>

Woops #2, sorry.
p_buffer wasn't properly checked.

-- 
Hugo Beauzée-Luyssen
-------------- next part --------------
A non-text attachment was scrubbed...
Name: opensles.c
Type: text/x-csrc
Size: 13811 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20110506/e9009a30/attachment.c>


More information about the vlc-devel mailing list