[FFmpeg-cvslog] r9547 - in trunk/libavcodec: cavs.h h264.c mpegvideo.h snow.c

[Michael Niedermayer]

Hi

On Mon, Jul 09, 2007 at 02:38:46PM -0700, Roman Shaposhnik wrote:
> Hi
> 
> On Mon, 2007-07-09 at 22:05 +0100, M?ns Rullg?rd wrote:
> > >> So what do we do with this function?  Make it "extern inline"?  Move
> > >> it to a header as "static inline"?
> > >
> > > hmm id say Move it to a header as "static inline"
> > 
> > Something like this then?
> 
>   Yes, but keeping ff_h263_chroma_roundtab[] separate from the
> function itself might be more trouble than it is worth. Something
> like:
> 
> static inline int ff_h263_round_chroma(int x){
>     static const unsigned char ff_h263_chroma_roundtab[16]...

iam not opposed to that ...

[...]
-- 
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-cvslog/attachments/20070710/aa4010a9/attachment.pgp>