[FFmpeg-devel] [PATCH] Get rid of p
Sun Jun 8 22:17:14 CEST 2008
On Sun, Jun 08, 2008 at 10:09:52PM +0200, Michael Niedermayer wrote:
> On Sun, Jun 08, 2008 at 08:57:21PM +0100, Ramiro Polla wrote:
> > > we can discuss the renaming then seperately ...
> > > iam not sure if its good or not for the functions in your patch ...
> > I know my argumentation skills are close to NULL, but I'll try anyways...
> > Once you know what the function does, having small variable names might
> > be better, but if you're reading it for the first time to try and find
> > out what it does, having single-letter or non-descriptive names makes it
> > harder to understand. Or if you're debugging and stumble upon those
> > variables, it takes longer to find out what they actually are.
> > I use FFmpeg's source code as a basis to learn lots of things. I prefer
> > learning from well-written C code than textual descriptions like
> > textbooks or tutorials, and I'm probably not the only one.
> > Another reason is that searching for any of occurrence of some variable
> > becomes hard, since its name is part of many other variable names and
> > mostly anything.
> get cscope
and a simple search for [^a-z]s[^a-z] looks pretty darn accurate for finding
all and only them
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Thouse who are best at talking, realize last or never when they are wrong.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 189 bytes
Desc: Digital signature
More information about the ffmpeg-devel