[FFmpeg-devel] [PATCH 1/4] Implement avfilter_ref_video_buffer().

Michael Niedermayer michaelni
Sun Nov 21 17:57:34 CET 2010


On Sat, Nov 20, 2010 at 12:59:30PM +0100, Stefano Sabatini wrote:
> On date Saturday 2010-11-20 03:39:44 +0100, Michael Niedermayer encoded:
> > On Sat, Nov 13, 2010 at 03:02:50PM +0100, Stefano Sabatini wrote:
> > > On date Friday 2010-11-12 20:19:08 +0100, Michael Niedermayer encoded:
> > > > On Fri, Nov 12, 2010 at 03:49:38PM +0100, Stefano Sabatini wrote:
> > > [...]
> > > > > Other proposals:
> > > > > 
> > > > 
> > > > > avfilter_get_video_buffer_ref_from_arrays
> > > > 
> > > > just longer than my suggestion otherwise its the same
> > > 
> > > Your suggestion:
> > > avfilter_arrays_to_video_buffer_ref()
> > > 
> > > I slightly dislike it as it lacks a verb.
> > 
> > If you associate a function with doing something then yes that makes sense
> > doxy then should also be writen to indicate so which leads to 3rd person ...
> > 
> > the use of to or 2 is quite common to indicate "convert to" though
> > 
> > 
> > >  
> > > > > avfilter_ref_video_buffer_arrays
> > > > 
> > > > ambigous
> > > 
> > > "Reference video buffer arrays", that is get a reference for the given
> > > video buffer arrays (data+linesizes), I don't see that so much
> > > ambiguous and a name of a function has not to be necessarily
> > > *completely* explicative.
> > 
> > avfilter_ref_video_buffer_arrays, does not contain a verb
> > either. More precissely ref is not a english word at all.
> 
> "ref" clearly stands for "reference".
> 
> > > > > avfilter_ref_buffer_from_video_buffer_arrays
> > > > > avfilter_get_video_buffer_ref_from_video_buffer_arrays
> > > > > avfilter_get_buffer_ref_from_video_buffer_arrays
> > > > 
> > > > we have no video anything array, as the thing just represents a single frame
> > > 
> > > AVFilterVideoBufferRef references just an image anyway, so if this is
> > > amniguous the same is for the name of that struct.
> > 
> > Thats not untrue
> > 
> > 
> > > 
> > > Alternatively we could use the term "frame" in place of image (which
> > > is more consistent with the names already used in the rest of the
> > > lavfi API, start/end_frame).
> > 
> > To quote h264
> > 3.104 picture: A collective term for a field or a frame.
> > 
> > So while you might think this is bikeshed, its not the terms you switch between
> > have subtiley different meaning
> > 
> > 
> > > 
> > > > > avfilter_ref_buffer_from_image_arrays
> > > > > avfilter_get_video_buffer_ref_from_image_arrays
> > > > > avfilter_get_buffer_ref_from_image_arrays
> > > > 
> > > > image arrays implicates to me that there are several images
> > > 
> > > That's unfortunately a quirk of the English language, image arrays ~
> > > images arrays.
> > > 
> > > Resuming:
> > > avfilter_get_video_buffer_ref_from_frame
> > 
> > > avfilter_get_video_buffer_ref_from_arrays
> > 
> > from your list this is the only that is not totally nonsense
> > still it is just longer than what i suggested with zero additional information
> > added
> > avfilter_arrays_to_video_buffer_ref
> > vs.
> > avfilter_get_video_buffer_ref_from_arrays
> 
> So we have the candidates:
> avfilter_arrays_to_video_buffer_ref
> avfilter_get_video_buffer_ref_from_arrays
> 
> Other proposals:
> avfilter_ref_video_buffer_from_arrays => reference video buffer from arrays 
> avfilter_ref_video_buffer_arrays      => reference video buffer arrays
> 
> otherwise I'll stick with one of the candidates.

pick what you want, iam too busy for this

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Old school: Use the lowest level language in which you can solve the problem
            conveniently.
New school: Use the highest level language in which the latest supercomputer
            can solve the problem without the user falling asleep waiting.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20101121/5b3f39e7/attachment.pgp>



More information about the ffmpeg-devel mailing list