[FFmpeg-devel] [RFC] use get_unary in multiple places

Michael Niedermayer michaelni
Fri Aug 10 15:36:23 CEST 2007


Hi

On Fri, Aug 10, 2007 at 02:23:32PM +0200, Alex Beregszaszi wrote:
> On Thu, 2007-08-09 at 12:11 +0200, Michael Niedermayer wrote:
> > Hi
> > 
> > On Thu, Aug 09, 2007 at 10:31:05AM +0200, Alex Beregszaszi wrote:
> > > Hi,
> > > 
> > > > PS: dont commit any further code to files you dont maintain without
> > > > first getting approval from the respective maintainer
> > > 
> > > And why do you commit to code what you don't maintain?
> > > 
> > > I don't think it has sense that people have commit rights, but cannot do
> > > commits.
> > 
> > people can do commits
> > * in files they maintain
> > * trivial, correct and needed changes in files they dont maintain
> > * changes which where approved by the maintainer
> > * changes for which patches where posted but no reaction from the maintainer
> > 
> > besides this code should be tested before commit, should be clean, ...
> > 
> > but you know these rules very well yourself ...
> > 
> > 
> > your changes broke compilation
> > moved messy code to bitstream.h (#if 1, random indention, ...)
> > where not approved by the maintainer
> > no patch was posted
> > moved code to the wrong file, it should have been golomb.h not bitstream.h
> > or maybe a new file should have been created so that conditional compilation
> > became possible ...
> 
> Conditional compilation? Did you noticed that I just made a function
> common which was used in several places? Did you?

yes, yes and yes


> 
> This is not an optional function for those codecs, but a mandatory one.
> And is present in at least 5 forms in 5 codecs.

golomb.c is used by 13 codecs and is compiled conditionally

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

No great genius has ever existed without some touch of madness. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070810/b0cd63c9/attachment.pgp>



More information about the ffmpeg-devel mailing list