On Tue, Feb 28, 2012 at 5:43 PM, Diego Biurrun <[email protected]> wrote:
> On Tue, Feb 28, 2012 at 12:20:32AM +0200, Felipe Contreras wrote:
>> On Mon, Feb 27, 2012 at 11:29 PM, Diego Biurrun <[email protected]> wrote:
>> > On Mon, Feb 27, 2012 at 10:46:40PM +0200, Felipe Contreras wrote:
>> >> Notice the library is called libtidsp, and the name libtidspbridge is
>> >> abusrdly long, inconvenient, and inconsistent with the actual library
>> >> name, but that's what was requested on the mailing list.
>> >>
>> >> --- /dev/null
>> >> +++ b/libavcodec/tidspbridge_mpeg4.c
>> >> @@ -0,0 +1,173 @@
>> >> +static int start_frame(AVCodecContext *avctx, const uint8_t *buffer,
>> >> + uint32_t size)
>> >
>> > static int start_frame(AVCodecContext *avctx, const uint8_t *buffer,
>> > uint32_t size)
>>
>> I don't see any of that in the documentation.
>>
>> Can you update the section 'Editor configuration' with the vim
>> cinoptions you are talking about, or at least some examples of this?
>
> set cindent
> set cinoptions=(0
>
> seems to do the trick. Let me know if you have a better suggestion.
Yeap, that should do it, also :0, t0, and probably l1, if:
case foo: {
break;
}
Is preferred over:
case foo: {
break;
}
But it's still not mentioned in 1.3.1 Code formatting conventions.
Cheers.
--
Felipe Contreras
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
