[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