[FFmpeg-devel] [PATCH][H264] Record extra fields for VA API support (take 3)

[Michael Niedermayer]

On Mon, Feb 02, 2009 at 03:03:03PM +0100, Gwenole Beauchesne wrote:
> Hi,
>
> On Sat, 31 Jan 2009, Michael Niedermayer wrote:
>
>> what about setting these flags like use_weight, that is when the
>> actual weights are non default?
>
> e.g. as in attachment?
[...]
> @@ -2996,11 +2996,11 @@ static int pred_weight_table(H264Context *h){
>      chroma_def = 1<<h->chroma_log2_weight_denom;
>  
>      for(list=0; list<2; list++){
> +        h->luma_weight_flag[list] = 0;
> +        h->chroma_weight_flag[list] = 0;
>          for(i=0; i<h->ref_count[list]; i++){
> -            int luma_weight_flag, chroma_weight_flag;
> -
> -            luma_weight_flag= get_bits1(&s->gb);
> -            if(luma_weight_flag){
> +            h->luma_weight_flag[list]= get_bits1(&s->gb);
> +            if(h->luma_weight_flag[list]){
>                  h->luma_weight[list][i]= get_se_golomb(&s->gb);
>                  h->luma_offset[list][i]= get_se_golomb(&s->gb);
>                  if(   h->luma_weight[list][i] != luma_def

no

as in:
                if(   h->luma_weight[list][i] != luma_def
                   || h->luma_offset[list][i] != 0){
                    h->use_weight= 1;
+                   h->luma_weight_flag[list]= 1;
                }


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

Frequently ignored awnser#1 FFmpeg bugs should be sent to our bugtracker. User
questions about the command line tools should be sent to the ffmpeg-user ML.
And questions about how to use libav* should be sent to the libav-user ML.
-------------- 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/20090202/6ee156e0/attachment.pgp>