[MPlayer-dev-eng] [PATCH] ve_lavc cosmetics
Oded Shimon
ods15 at ods15.dyndns.org
Sun Feb 12 17:29:50 CET 2006
On Sun, Feb 12, 2006 at 03:52:23PM +0000, Robert Swain wrote:
> Hello,
>
> A while ago when I was trying to do MPEG-4 profiles in ve_lavc I noticed
> how ugly libmpcodecs/ve_lavc.c was in terms of mixing styles, mixing
> tabs and spaces for indentation, irregular indentation and so on. Today
> I got bored so I decided to go through it and clean it up. So, this is
> what I have done:
>
> - All indentation is now four spaces per 'level'.
> - Bad indentation has been fixed.
> - Many long lines in the main code are now < 80 columns or thereabouts.
> - There were a number of styles for if(){} paranthesis position and
> indentation, these have been made uniform.
> - This is just personal taste, I think, but there was a mix of a=b, a= b
> and a = b. I prefer the latter so I have made this consistent
> throughout.
> - Around (patched) line 556 ['e = sscanf (lavc_param_aspect, "%f",
> &ratio);'] there were {} but they are unnecessary as there was only one
> line in the else statement so I removed these. I hope this doesn't have
> any ABI connotations that I'm unaware of due to being a n00b. :)
>
> Hopefully all this will be OK but if I've done anything idiotic or you
> guys and gals really don't like any of it, please say.
>
> Warning, the attached patch is difficult to read because of its nature
> as a cosmetic diff.
Your patch contains cosmetics. rejected.
Oh.
I'm for this patch, I can't go over it, just have to take your word for it
that there are absoloutely no functional changes. Is anyone against this?
I'd like to commit.
- ods15
More information about the MPlayer-dev-eng
mailing list