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

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to