[FFmpeg-devel] [PATCH] Get rid of p

Ramiro Polla ramiro
Fri Jun 6 18:09:27 CEST 2008


By the message subject you probably asked yourself "what 'p'?".

Exactly, I ask myself the same question when reading the source.

I find one-letter variable names hard to grep for and hard to 
understand. Are patches to get rid of them or use more descriptive names 

Particular cases that come to my mind are:
  - 's', used throughout FFmpeg to mean context. I suggest to change 
them to ctx;
  - 'p', used in video encoders for something that I still have no idea 
what is doing there.

Ramiro Polla
-------------- next part --------------
A non-text attachment was scrubbed...
Name: getridofp.diff
Type: text/x-patch
Size: 4132 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080606/876ec1cb/attachment.bin>

More information about the ffmpeg-devel mailing list