[Ffmpeg-devel] Naming conventions
Michael Niedermayer
michaelni
Tue Mar 6 14:43:09 CET 2007
Hi
On Tue, Mar 06, 2007 at 02:32:49PM +0100, Michael Niedermayer wrote:
> Hi
>
> On Tue, Mar 06, 2007 at 02:18:17PM +0100, Baptiste Coudurier wrote:
> > Michael Niedermayer wrote:
> > > Hi
> > >
> > > On Tue, Mar 06, 2007 at 01:37:00PM +0100, Baptiste Coudurier wrote:
> > > [...]
> > >>> and what about
> > >>>
> > >>> avpicture_get_size would that be av_picture_get_size or av_picture_size_get ?
> > >> av_picture_get_size, av_picture_fill, av_picture_pad ...
> > >
> > > why get_size? and not size_get? you mix type_action with action_type in a
> > > single name
> >
> > I don't mix them, type: av_picture, action: get_size
>
> why not type: av_picture_size action: get?
>
> the question maters for a naming convention ...
>
> also what would av_new_stream() be called then?
> av_format_context_new_stream() ? this is ugly
>
> and av_index_search_timestamp() would end with
> av_stream_index_timestamp_search() or av_stream_index_search_timestamp() or
> av_stream_search_index_timestamp()
>
> all depending on where the type ends not to mention the silly long
> name
> and the action is logically here on the index not on AVStream but
> AVStream is the type you must pass to the function!
>
> the whole naming convention talk reminds me of M$ hungarian notation
> = "useless and silly"
my _current_ suggestion for a nameing convention would be
1. av_ prefix on all new public global symbols
2. ff_ prefix on all new private global symbols
3. remaining name is the decission of the maintainer of the specific part
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Good people do not need laws to tell them to act responsibly, while bad
people will find a way around the laws. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070306/849f2a11/attachment.pgp>
More information about the ffmpeg-devel
mailing list