On 30/04/12 01:39, Diego Biurrun wrote:
> On Sun, Apr 29, 2012 at 01:51:47PM -0700, Luca Barbato wrote:
>> Apply coding style to the structs.
>
> Well, sort of, you basically just moved some '='.
>
>> --- a/libavcodec/avcodec.h
>> +++ b/libavcodec/avcodec.h
>> @@ -465,49 +465,49 @@ enum Motion_Est_ID {
>> enum AVDiscard{
>
> space before {
>
> more of the same below
>
>> /* We leave some space between them for extensions (drop some
>> * keyframes for intra-only or drop just some bidir frames). */
>> - AVDISCARD_NONE =-16, ///< discard nothing
>> + AVDISCARD_NONE =-16, ///< discard nothing
>
> space after =
>
>> enum AVColorPrimaries{
>> - AVCOL_PRI_NB , ///< Not part of ABI
>> + AVCOL_PRI_NB , ///< Not part of ABI
>> };
>
> Spaces befor ',' are ugly.
>
> more of the same below
I'll go and do another round on this file soon if you don't beat me in
that =)
lu
--
Luca Barbato
Gentoo/linux
http://dev.gentoo.org/~lu_zero
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
