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

[Måns Rullgård]

Michael Niedermayer <michaelni at gmx.at> writes:

> 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 ...

The patch would look like this.

-- 
M?ns Rullg?rd
mans at mansr.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: h263_round_chroma2.patch
Type: text/x-patch
Size: 2387 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20070710/444d9ad6/attachment.bin>