[Ffmpeg-devel] BlackFin lowlevel pixel operations PATCH

Marc Hoffman mmh
Tue Apr 3 14:40:32 CEST 2007


Diego Biurrun writes:
 > On Tue, Apr 03, 2007 at 07:46:14AM -0400, Marc Hoffman wrote:
 > > 
 > > How about we add this to the environment?
 > 
 > I don't have an opinion on this patch in general, but please use more
 > descriptive names.
 > 
Diego,

The beauty of the short names is tha they are in competition with the simple 'print'

(gdb) p avariable

(gdb) pblk pix 8 8 width

this is kind of a place where you don't want to have a lot of extra
typing.  At least thats my humble opinion.  So let me see you want to see the names like

pblk --> print_block_8bit
pblks -> print_block_16bit
clrblk > clear_block_8bit

How about this kind of a compromise

pblk --> pb_print_block_8bit
pblks -> pbs_print_block_16bit
clrblk > cb_clear_block_8bit


Marc

-------------- next part --------------
A non-text attachment was scrubbed...
Name: gdbi.p0
Type: text/x-patch
Size: 2832 bytes
Desc: gdb debug
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070403/4aa626a4/attachment.bin>



More information about the ffmpeg-devel mailing list