Diego Biurrun <di...@biurrun.de> writes:

> On Tue, Jul 10, 2012 at 12:48:00PM +0200, Luca Barbato wrote:
>> On 07/10/2012 11:54 AM, Diego Biurrun wrote:
>> > On Tue, Jul 10, 2012 at 05:16:01AM +0200, Luca Barbato wrote:
>> >>
>> >> --- a/libavformat/mxfdec.c
>> >> +++ b/libavformat/mxfdec.c
>> >> @@ -1936,12 +1941,29 @@ static int mxf_read_packet_old(AVFormatContext 
>> >> *s, AVPacket *pkt)
>> >> +                av_log_ask_for_sample(s,
>> >> +                    "KLV for edit unit %i extends into next edit unit "
>> >> +                    "- OPAtom misinterpreted as OP1a?\n",
>> >> +                    mxf->current_edit_unit);
>> > 
>> > Indentation is off.
>> 
>> Should align it to the s ?
>
> Yes.

That's ridiculous here.  It's so far right there's no space left on the
lines.  I think it's fine as is.

-- 
Måns Rullgård
m...@mansr.com
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to