2011/5/9 Aℓex Converse <aconve...@google.com>:
> On Mon, May 9, 2011 at 7:02 AM, Diego Biurrun <di...@biurrun.de> wrote:
>> On Mon, May 09, 2011 at 03:34:03PM +0200, Diego Biurrun wrote:
>>> On Thu, May 05, 2011 at 09:41:27AM -0700, Aℓex Converse wrote:
>>> > On Thu, May 5, 2011 at 2:39 AM, Diego Biurrun <di...@biurrun.de> wrote:
>>> > > Please trim your quotes, thank you.
>>> >
>>> > Maybe if you sent a patch on top of the patch you hijacked there would
>>> > be less to trim.
>>>
>>> For your perusal I attached the output of
>>>
>>>   git diff -w s302m_alex:libavcodec/s302m.c s302m:libavcodec/s302m.c
>>
>> Scratch that broken one, here comes the real deal.
>>
>
>>      const uint8_t *buf = avpkt->data;
>> -    int frame_size, buf_size = avpkt->size;
>> +    int buf_size       = avpkt->size, frame_size;
>>
>>     frame_size = s302m_parse_frame_header(avctx, buf, buf_size);
>
> I really don't think this is any more readable. In fact I think it's
> arguably less readable.
>

Anyway I really don't car. Looks good enough to me. Someone please
commit this so I can be done with this nightmare.
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to