[Ffmpeg-devel] Naming conventions
Tue Mar 6 15:18:39 CET 2007
On Tue, Mar 06, 2007 at 01:58:24PM -0000, M?ns Rullg?rd wrote:
> Michael Niedermayer said:
> > 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!
> Stop being facetious. Please.
huh? what is facetious? me saying that the naming convention should clearly
say what type and action is or me pointing at cases where your
suggestion is ambiguous?
or are you trying to say that you have no real arguments?
or are you trying to say that you will decide all names based on unwritten
rules and everyone who doesnt understand you rules is facetious?
also its funny that on nut-devel you where nitpicking that the spec wasnt
clear enough and now you attack my for trying to clarify a really ambiguous
single line suggestion of a nameing convention
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
I am the wisest man alive, for I know one thing, and that is that I know
nothing. -- Socrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 189 bytes
Desc: not available
More information about the ffmpeg-devel