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