[FFmpeg-devel] [PATCH] Move H.264 DSP functions from dsputil.c to h264dsp.c

Michael Niedermayer michaelni
Sat Jul 28 20:43:23 CEST 2007


Hi

On Sat, Jul 28, 2007 at 08:35:32PM +0200, Panagiotis Issaris wrote:
> Hi
> 
> Michael Niedermayer wrote:
> > Hi
> >
> > On Sat, Jul 28, 2007 at 12:40:08PM +0200, Panagiotis Issaris wrote:
> >   
> >> Hi,
> >>
> >> Panagiotis Issaris wrote:
> >>     
> >>>> Michael Niedermayer wrote:
> >>>>   
> >>>>     
> >>>>         
> >>>>> Hi
> >>>>>
> >>>>> On Fri, Jul 27, 2007 at 06:04:18PM +0200, Panagiotis Issaris wrote:
> >>>>> [...]    
> >>>>>       
> >>>>>           
> >>>>>> Ouch, I just noticed that with this patch I had accidentally moved lots
> >>>>>> of the H.264 decoder DSP code in a file with a different header,
> >>>>>> resulting in incorrect "Copyright by ..." lines.
> >>>>>>
> >>>>>> Is it okay to just merge the "Copyright by ..." lines? Or would it be
> >>>>>> better to split the h264dsp.c file right away in a h264dspenc.c and
> >>>>>> h264dspdec.c as Aurelien suggested?
> >>>>>>     
> >>>>>>       
> >>>>>>         
> >>>>>>             
> >>>>> IMHO split if possible
> >>>>>
> >>>>> [...]
> >>>>>   
> >>>>>     
> >>>>>       
> >>>>>           
> >>>> I'd like to start with renaming h264dsp.c to h264dspenc.c. So that would be:
> >>>> svn mv h264dsp.c h264dspenc.c and the attached patch. Okay to commit?
> >>>>
> >>>>  Makefile  |    2 -
> >>>>  h264dsp.c |   81
> >>>> --------------------------------------------------------------
> >>>>  2 files changed, 1 insertion(+), 82 deletions(-) 
> >>>>     
> >>>>         
> >>> The previous patch would then be followed by this one, which takes care of
> >>> some renames after the previous pure rename patch.
> >>>   
> >>>       
> >> Next, the attached patch moves the H.264 decoder DSP functions out of dsputil into a separate file called h264dspdec.c.
> >>     
> >
> > but these functions are needed by the encoder too ...
> >
> >   
> They aren't yet used in the encoder patch I am working on. Okay to
> commit it as is for now and migrate them to a file containing
> common DSP code called "h264dsp.c" once it is actually used from both
> the encoder and decoder?

no, move them to the correct file please, or leave them where they
are

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

Democracy is the form of government in which you can choose your dictator
-------------- 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/20070728/f9ed10ff/attachment.pgp>



More information about the ffmpeg-devel mailing list