annotate mpegvideo_xvmc.c @ 8953:5e70bdb97d0c libavcodec

Thoroughly check all fields set by the application in xvmc struct.
author iive
date Mon, 16 Feb 2009 22:46:30 +0000
parents 8d3722a5973a
children a8c7a467a287
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
1 /*
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
2 * XVideo Motion Compensation
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
3 * Copyright (c) 2003 Ivan Kalvachev
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
4 *
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
5 * This file is part of FFmpeg.
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
6 *
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
7 * FFmpeg is free software; you can redistribute it and/or
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
8 * modify it under the terms of the GNU Lesser General Public
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
9 * License as published by the Free Software Foundation; either
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
10 * version 2.1 of the License, or (at your option) any later version.
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
11 *
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
12 * FFmpeg is distributed in the hope that it will be useful,
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
13 * but WITHOUT ANY WARRANTY; without even the implied warranty of
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
14 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
15 * Lesser General Public License for more details.
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
16 *
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
17 * You should have received a copy of the GNU Lesser General Public
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
18 * License along with FFmpeg; if not, write to the Free Software
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
19 * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
20 */
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
21
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
22 #include <limits.h>
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
23
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
24 #include "avcodec.h"
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
25 #include "dsputil.h"
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
26 #include "mpegvideo.h"
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
27
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
28 #undef NDEBUG
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
29 #include <assert.h>
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
30
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
31 #include "xvmc.h"
8881
f8042554d4c8 Add xvmc_internal.h that contains all internal xvmc function declarations.
diego
parents: 8880
diff changeset
32 #include "xvmc_internal.h"
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
33
8902
aa49d3e97b70 Doxygen comment/explanation for ff_xvmc_init_block().
diego
parents: 8897
diff changeset
34 /**
aa49d3e97b70 Doxygen comment/explanation for ff_xvmc_init_block().
diego
parents: 8897
diff changeset
35 * Initializes the block field of the MpegEncContext pointer passed as
aa49d3e97b70 Doxygen comment/explanation for ff_xvmc_init_block().
diego
parents: 8897
diff changeset
36 * parameter after making sure that the data is not corrupted.
8930
8987e082c09f spelling/grammar/wording fixes for Doxygen comments
diego
parents: 8927
diff changeset
37 * In order to implement something like direct rendering instead of decoding
8987e082c09f spelling/grammar/wording fixes for Doxygen comments
diego
parents: 8927
diff changeset
38 * coefficients in s->blocks and then copying them, copy them directly
8926
b8da7a21372c Document all functions in mpegvideo_xvmc.
iive
parents: 8925
diff changeset
39 * into the data_blocks array provided by xvmc.
8902
aa49d3e97b70 Doxygen comment/explanation for ff_xvmc_init_block().
diego
parents: 8897
diff changeset
40 */
8855
6ce8ebfc9d86 use ff_xvmc for function names instead of XVMC_
iive
parents: 8845
diff changeset
41 void ff_xvmc_init_block(MpegEncContext *s)
8835
8c11ebbca9b3 whitespace cosmetics: consistent (more or less) K&R style
diego
parents: 8834
diff changeset
42 {
8925
23051e3e9d25 Rename xvmc_pixfmt_render structure to xvmc_pix_fmt.
diego
parents: 8924
diff changeset
43 struct xvmc_pix_fmt *render = (struct xvmc_pix_fmt*)s->current_picture.data[2];
8934
a613cd6b6071 Rename AV_XVMC_RENDER_MAGIC constant to AV_XVMC_ID to reflect a similar
diego
parents: 8932
diff changeset
44 assert(render && render->xvmc_id == AV_XVMC_ID);
8916
bd643af669df Check all critical xvmc struct fields in ff_xvmc_field_start()
iive
parents: 8915
diff changeset
45
8860
0617c9f594d0 Remove some useless parentheses.
diego
parents: 8858
diff changeset
46 s->block = (DCTELEM *)(render->data_blocks + render->next_free_data_block_num * 64);
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
47 }
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
48
8926
b8da7a21372c Document all functions in mpegvideo_xvmc.
iive
parents: 8925
diff changeset
49 /**
b8da7a21372c Document all functions in mpegvideo_xvmc.
iive
parents: 8925
diff changeset
50 * Fill individual block pointers, so there are no gaps in the data_block array
b8da7a21372c Document all functions in mpegvideo_xvmc.
iive
parents: 8925
diff changeset
51 * in case not all blocks in MB are coded.
b8da7a21372c Document all functions in mpegvideo_xvmc.
iive
parents: 8925
diff changeset
52 */
8855
6ce8ebfc9d86 use ff_xvmc for function names instead of XVMC_
iive
parents: 8845
diff changeset
53 void ff_xvmc_pack_pblocks(MpegEncContext *s, int cbp)
8835
8c11ebbca9b3 whitespace cosmetics: consistent (more or less) K&R style
diego
parents: 8834
diff changeset
54 {
8889
b4838c71d52a Merge some declarations and initializations.
diego
parents: 8888
diff changeset
55 int i, j = 0;
8835
8c11ebbca9b3 whitespace cosmetics: consistent (more or less) K&R style
diego
parents: 8834
diff changeset
56 const int mb_block_count = 4 + (1 << s->chroma_format);
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
57
8835
8c11ebbca9b3 whitespace cosmetics: consistent (more or less) K&R style
diego
parents: 8834
diff changeset
58 cbp <<= 12-mb_block_count;
8c11ebbca9b3 whitespace cosmetics: consistent (more or less) K&R style
diego
parents: 8834
diff changeset
59 for (i = 0; i < mb_block_count; i++) {
8861
da23b389e856 cosmetics: Consistently format all if/else statements in K&R style
diego
parents: 8860
diff changeset
60 if (cbp & (1 << 11))
8856
e5cf06287032 remove some useless ()
iive
parents: 8855
diff changeset
61 s->pblocks[i] = (short *)(&s->block[j++]);
8861
da23b389e856 cosmetics: Consistently format all if/else statements in K&R style
diego
parents: 8860
diff changeset
62 else
8834
1f4d7469a663 whitespace cosmetics: Fix indentation depth.
diego
parents: 8833
diff changeset
63 s->pblocks[i] = NULL;
8947
8d3722a5973a whitespace cosmetics: Place spaces around += for better readability.
diego
parents: 8944
diff changeset
64 cbp += cbp;
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
65 }
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
66 }
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
67
8893
1f3e03da966a Doxygenize ff_xvmc_field_start() comment.
diego
parents: 8892
diff changeset
68 /**
8926
b8da7a21372c Document all functions in mpegvideo_xvmc.
iive
parents: 8925
diff changeset
69 * Find and store the surfaces that are used as reference frames.
8893
1f3e03da966a Doxygenize ff_xvmc_field_start() comment.
diego
parents: 8892
diff changeset
70 * This function should be called for every new field and/or frame.
1f3e03da966a Doxygenize ff_xvmc_field_start() comment.
diego
parents: 8892
diff changeset
71 * It should be safe to call the function a few times for the same field.
1f3e03da966a Doxygenize ff_xvmc_field_start() comment.
diego
parents: 8892
diff changeset
72 */
8942
9dcb220e968c whitespace cosmetics
diego
parents: 8941
diff changeset
73 int ff_xvmc_field_start(MpegEncContext *s, AVCodecContext *avctx)
8835
8c11ebbca9b3 whitespace cosmetics: consistent (more or less) K&R style
diego
parents: 8834
diff changeset
74 {
8925
23051e3e9d25 Rename xvmc_pixfmt_render structure to xvmc_pix_fmt.
diego
parents: 8924
diff changeset
75 struct xvmc_pix_fmt *last, *next, *render = (struct xvmc_pix_fmt*)s->current_picture.data[2];
8916
bd643af669df Check all critical xvmc struct fields in ff_xvmc_field_start()
iive
parents: 8915
diff changeset
76 const int mb_block_count = 4 + (1 << s->chroma_format);
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
77
8844
9ea7195b2ed4 Replace all (x == NULL) or (x != NULL) in assert and if conditions by !x and x.
diego
parents: 8842
diff changeset
78 assert(avctx);
8934
a613cd6b6071 Rename AV_XVMC_RENDER_MAGIC constant to AV_XVMC_ID to reflect a similar
diego
parents: 8932
diff changeset
79 if (!render || render->xvmc_id != AV_XVMC_ID ||
8923
7117ea11df1e K&R whitespace cosmetics
diego
parents: 8922
diff changeset
80 !render->data_blocks || !render->mv_blocks) {
8916
bd643af669df Check all critical xvmc struct fields in ff_xvmc_field_start()
iive
parents: 8915
diff changeset
81 av_log(avctx, AV_LOG_ERROR,
bd643af669df Check all critical xvmc struct fields in ff_xvmc_field_start()
iive
parents: 8915
diff changeset
82 "Render token doesn't look as expected.\n");
8870
c5112df7f8b8 cosmetics: grammar/spelling/wording fixes in comments
diego
parents: 8869
diff changeset
83 return -1; // make sure that this is a render packet
8916
bd643af669df Check all critical xvmc struct fields in ff_xvmc_field_start()
iive
parents: 8915
diff changeset
84 }
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
85
8885
d3f806fb5f71 cosmetics: K&R style
diego
parents: 8884
diff changeset
86 if (render->filled_mv_blocks_num) {
8884
a054304f57d4 Give nice message for failed assert.
iive
parents: 8881
diff changeset
87 av_log(avctx, AV_LOG_ERROR,
8916
bd643af669df Check all critical xvmc struct fields in ff_xvmc_field_start()
iive
parents: 8915
diff changeset
88 "Rendering surface contains %i unprocessed blocks.\n",
8885
d3f806fb5f71 cosmetics: K&R style
diego
parents: 8884
diff changeset
89 render->filled_mv_blocks_num);
8888
5b7c6ebb4a36 Return error instead of triggering assert.
iive
parents: 8887
diff changeset
90 return -1;
8884
a054304f57d4 Give nice message for failed assert.
iive
parents: 8881
diff changeset
91 }
8941
42a048ecbba3 Give struct members more sensible names:
diego
parents: 8934
diff changeset
92 if (render->allocated_mv_blocks < 1 ||
8953
5e70bdb97d0c Thoroughly check all fields set by the application in xvmc struct.
iive
parents: 8947
diff changeset
93 render->allocated_data_blocks < render->allocated_mv_blocks*mb_block_count ||
5e70bdb97d0c Thoroughly check all fields set by the application in xvmc struct.
iive
parents: 8947
diff changeset
94 render->start_mv_blocks_num >= render->allocated_mv_blocks ||
5e70bdb97d0c Thoroughly check all fields set by the application in xvmc struct.
iive
parents: 8947
diff changeset
95 render->next_free_data_block_num >
5e70bdb97d0c Thoroughly check all fields set by the application in xvmc struct.
iive
parents: 8947
diff changeset
96 render->allocated_data_blocks -
5e70bdb97d0c Thoroughly check all fields set by the application in xvmc struct.
iive
parents: 8947
diff changeset
97 mb_block_count*(render->allocated_mv_blocks-render->start_mv_blocks_num)) {
8916
bd643af669df Check all critical xvmc struct fields in ff_xvmc_field_start()
iive
parents: 8915
diff changeset
98 av_log(avctx, AV_LOG_ERROR,
bd643af669df Check all critical xvmc struct fields in ff_xvmc_field_start()
iive
parents: 8915
diff changeset
99 "Rendering surface doesn't provide enough block structures to work with.\n");
bd643af669df Check all critical xvmc struct fields in ff_xvmc_field_start()
iive
parents: 8915
diff changeset
100 return -1;
bd643af669df Check all critical xvmc struct fields in ff_xvmc_field_start()
iive
parents: 8915
diff changeset
101 }
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
102
8917
422a9dbebb53 Almost cosmetics.
iive
parents: 8916
diff changeset
103 render->picture_structure = s->picture_structure;
422a9dbebb53 Almost cosmetics.
iive
parents: 8916
diff changeset
104 render->flags = s->first_field ? 0 : XVMC_SECOND_FIELD;
8918
b27254b4f633 Cosmetics. Vertical align.
iive
parents: 8917
diff changeset
105 render->p_future_surface = NULL;
b27254b4f633 Cosmetics. Vertical align.
iive
parents: 8917
diff changeset
106 render->p_past_surface = NULL;
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
107
8871
199e9809eb5a some more K&R whitespace cosmetics
diego
parents: 8870
diff changeset
108 switch(s->pict_type) {
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
109 case FF_I_TYPE:
8867
0694627caaea whitespace cosmetics: consistently format all comments
diego
parents: 8866
diff changeset
110 return 0; // no prediction from other frames
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
111 case FF_B_TYPE:
8925
23051e3e9d25 Rename xvmc_pixfmt_render structure to xvmc_pix_fmt.
diego
parents: 8924
diff changeset
112 next = (struct xvmc_pix_fmt*)s->next_picture.data[2];
8844
9ea7195b2ed4 Replace all (x == NULL) or (x != NULL) in assert and if conditions by !x and x.
diego
parents: 8842
diff changeset
113 if (!next)
8835
8c11ebbca9b3 whitespace cosmetics: consistent (more or less) K&R style
diego
parents: 8834
diff changeset
114 return -1;
8934
a613cd6b6071 Rename AV_XVMC_RENDER_MAGIC constant to AV_XVMC_ID to reflect a similar
diego
parents: 8932
diff changeset
115 if (next->xvmc_id != AV_XVMC_ID)
8835
8c11ebbca9b3 whitespace cosmetics: consistent (more or less) K&R style
diego
parents: 8834
diff changeset
116 return -1;
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
117 render->p_future_surface = next->p_surface;
8867
0694627caaea whitespace cosmetics: consistently format all comments
diego
parents: 8866
diff changeset
118 // no return here, going to set forward prediction
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
119 case FF_P_TYPE:
8925
23051e3e9d25 Rename xvmc_pixfmt_render structure to xvmc_pix_fmt.
diego
parents: 8924
diff changeset
120 last = (struct xvmc_pix_fmt*)s->last_picture.data[2];
8872
706274a30a80 Remove one more commented-out line.
diego
parents: 8871
diff changeset
121 if (!last)
8867
0694627caaea whitespace cosmetics: consistently format all comments
diego
parents: 8866
diff changeset
122 last = render; // predict second field from the first
8934
a613cd6b6071 Rename AV_XVMC_RENDER_MAGIC constant to AV_XVMC_ID to reflect a similar
diego
parents: 8932
diff changeset
123 if (last->xvmc_id != AV_XVMC_ID)
8835
8c11ebbca9b3 whitespace cosmetics: consistent (more or less) K&R style
diego
parents: 8834
diff changeset
124 return -1;
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
125 render->p_past_surface = last->p_surface;
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
126 return 0;
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
127 }
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
128
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
129 return -1;
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
130 }
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
131
8897
fa198e44b199 Add Doxygen comment similar to ff_xvmc_field_start() to ff_xvmc_field_end().
diego
parents: 8896
diff changeset
132 /**
8926
b8da7a21372c Document all functions in mpegvideo_xvmc.
iive
parents: 8925
diff changeset
133 * Complete frame/field rendering by passing any remaining blocks.
8930
8987e082c09f spelling/grammar/wording fixes for Doxygen comments
diego
parents: 8927
diff changeset
134 * Normally ff_draw_horiz_band() is called for each slice, however,
8987e082c09f spelling/grammar/wording fixes for Doxygen comments
diego
parents: 8927
diff changeset
135 * some leftover blocks, for example from error_resilience(), may remain.
8897
fa198e44b199 Add Doxygen comment similar to ff_xvmc_field_start() to ff_xvmc_field_end().
diego
parents: 8896
diff changeset
136 * It should be safe to call the function a few times for the same field.
fa198e44b199 Add Doxygen comment similar to ff_xvmc_field_start() to ff_xvmc_field_end().
diego
parents: 8896
diff changeset
137 */
8855
6ce8ebfc9d86 use ff_xvmc for function names instead of XVMC_
iive
parents: 8845
diff changeset
138 void ff_xvmc_field_end(MpegEncContext *s)
8835
8c11ebbca9b3 whitespace cosmetics: consistent (more or less) K&R style
diego
parents: 8834
diff changeset
139 {
8925
23051e3e9d25 Rename xvmc_pixfmt_render structure to xvmc_pix_fmt.
diego
parents: 8924
diff changeset
140 struct xvmc_pix_fmt *render = (struct xvmc_pix_fmt*)s->current_picture.data[2];
8844
9ea7195b2ed4 Replace all (x == NULL) or (x != NULL) in assert and if conditions by !x and x.
diego
parents: 8842
diff changeset
141 assert(render);
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
142
8835
8c11ebbca9b3 whitespace cosmetics: consistent (more or less) K&R style
diego
parents: 8834
diff changeset
143 if (render->filled_mv_blocks_num > 0)
8877
4044a647cb2a another (last?) round of K&R whitespace cosmetics
diego
parents: 8876
diff changeset
144 ff_draw_horiz_band(s, 0, 0);
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
145 }
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
146
8926
b8da7a21372c Document all functions in mpegvideo_xvmc.
iive
parents: 8925
diff changeset
147 /**
b8da7a21372c Document all functions in mpegvideo_xvmc.
iive
parents: 8925
diff changeset
148 * Synthesize the data needed by XvMC to render one macroblock of data.
8932
f647961ab2ac cosmetics: typo fix
diego
parents: 8930
diff changeset
149 * Fill all relevant fields, if necessary do IDCT.
8926
b8da7a21372c Document all functions in mpegvideo_xvmc.
iive
parents: 8925
diff changeset
150 */
8855
6ce8ebfc9d86 use ff_xvmc for function names instead of XVMC_
iive
parents: 8845
diff changeset
151 void ff_xvmc_decode_mb(MpegEncContext *s)
8835
8c11ebbca9b3 whitespace cosmetics: consistent (more or less) K&R style
diego
parents: 8834
diff changeset
152 {
8871
199e9809eb5a some more K&R whitespace cosmetics
diego
parents: 8870
diff changeset
153 XvMCMacroBlock *mv_block;
8925
23051e3e9d25 Rename xvmc_pixfmt_render structure to xvmc_pix_fmt.
diego
parents: 8924
diff changeset
154 struct xvmc_pix_fmt *render;
8871
199e9809eb5a some more K&R whitespace cosmetics
diego
parents: 8870
diff changeset
155 int i, cbp, blocks_per_mb;
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
156
8834
1f4d7469a663 whitespace cosmetics: Fix indentation depth.
diego
parents: 8833
diff changeset
157 const int mb_xy = s->mb_y * s->mb_stride + s->mb_x;
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
158
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
159
8835
8c11ebbca9b3 whitespace cosmetics: consistent (more or less) K&R style
diego
parents: 8834
diff changeset
160 if (s->encoding) {
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
161 av_log(s->avctx, AV_LOG_ERROR, "XVMC doesn't support encoding!!!\n");
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
162 return;
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
163 }
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
164
8867
0694627caaea whitespace cosmetics: consistently format all comments
diego
parents: 8866
diff changeset
165 // from MPV_decode_mb(), update DC predictors for P macroblocks
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
166 if (!s->mb_intra) {
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
167 s->last_dc[0] =
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
168 s->last_dc[1] =
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
169 s->last_dc[2] = 128 << s->intra_dc_precision;
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
170 }
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
171
8867
0694627caaea whitespace cosmetics: consistently format all comments
diego
parents: 8866
diff changeset
172 // MC doesn't skip blocks
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
173 s->mb_skipped = 0;
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
174
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
175
8866
09d0ff5cdc81 whitespace cosmetics: Correctly indent all comments.
diego
parents: 8865
diff changeset
176 // Do I need to export quant when I could not perform postprocessing?
09d0ff5cdc81 whitespace cosmetics: Correctly indent all comments.
diego
parents: 8865
diff changeset
177 // Anyway, it doesn't hurt.
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
178 s->current_picture.qscale_table[mb_xy] = s->qscale;
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
179
8870
c5112df7f8b8 cosmetics: grammar/spelling/wording fixes in comments
diego
parents: 8869
diff changeset
180 // start of XVMC-specific code
8925
23051e3e9d25 Rename xvmc_pixfmt_render structure to xvmc_pix_fmt.
diego
parents: 8924
diff changeset
181 render = (struct xvmc_pix_fmt*)s->current_picture.data[2];
8844
9ea7195b2ed4 Replace all (x == NULL) or (x != NULL) in assert and if conditions by !x and x.
diego
parents: 8842
diff changeset
182 assert(render);
8934
a613cd6b6071 Rename AV_XVMC_RENDER_MAGIC constant to AV_XVMC_ID to reflect a similar
diego
parents: 8932
diff changeset
183 assert(render->xvmc_id == AV_XVMC_ID);
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
184 assert(render->mv_blocks);
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
185
8867
0694627caaea whitespace cosmetics: consistently format all comments
diego
parents: 8866
diff changeset
186 // take the next free macroblock
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
187 mv_block = &render->mv_blocks[render->start_mv_blocks_num +
8896
96f244848498 whitespace cosmetics: remove stray space
diego
parents: 8893
diff changeset
188 render->filled_mv_blocks_num];
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
189
8837
867767165fa8 whitespace cosmetics: vertical alignment
diego
parents: 8836
diff changeset
190 mv_block->x = s->mb_x;
867767165fa8 whitespace cosmetics: vertical alignment
diego
parents: 8836
diff changeset
191 mv_block->y = s->mb_y;
8867
0694627caaea whitespace cosmetics: consistently format all comments
diego
parents: 8866
diff changeset
192 mv_block->dct_type = s->interlaced_dct; // XVMC_DCT_TYPE_FRAME/FIELD;
8861
da23b389e856 cosmetics: Consistently format all if/else statements in K&R style
diego
parents: 8860
diff changeset
193 if (s->mb_intra) {
8867
0694627caaea whitespace cosmetics: consistently format all comments
diego
parents: 8866
diff changeset
194 mv_block->macroblock_type = XVMC_MB_TYPE_INTRA; // no MC, all done
8861
da23b389e856 cosmetics: Consistently format all if/else statements in K&R style
diego
parents: 8860
diff changeset
195 } else {
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
196 mv_block->macroblock_type = XVMC_MB_TYPE_PATTERN;
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
197
8835
8c11ebbca9b3 whitespace cosmetics: consistent (more or less) K&R style
diego
parents: 8834
diff changeset
198 if (s->mv_dir & MV_DIR_FORWARD) {
8836
4c175e9d492e whitespace cosmetics: another round of formatting consistency fixes
diego
parents: 8835
diff changeset
199 mv_block->macroblock_type |= XVMC_MB_TYPE_MOTION_FORWARD;
8873
02c59b74ba71 Make one comment slightly clearer and more readable.
diego
parents: 8872
diff changeset
200 // PMV[n][dir][xy] = mv[dir][n][xy]
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
201 mv_block->PMV[0][0][0] = s->mv[0][0][0];
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
202 mv_block->PMV[0][0][1] = s->mv[0][0][1];
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
203 mv_block->PMV[1][0][0] = s->mv[0][1][0];
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
204 mv_block->PMV[1][0][1] = s->mv[0][1][1];
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
205 }
8835
8c11ebbca9b3 whitespace cosmetics: consistent (more or less) K&R style
diego
parents: 8834
diff changeset
206 if (s->mv_dir & MV_DIR_BACKWARD) {
8836
4c175e9d492e whitespace cosmetics: another round of formatting consistency fixes
diego
parents: 8835
diff changeset
207 mv_block->macroblock_type |= XVMC_MB_TYPE_MOTION_BACKWARD;
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
208 mv_block->PMV[0][1][0] = s->mv[1][0][0];
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
209 mv_block->PMV[0][1][1] = s->mv[1][0][1];
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
210 mv_block->PMV[1][1][0] = s->mv[1][1][0];
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
211 mv_block->PMV[1][1][1] = s->mv[1][1][1];
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
212 }
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
213
8877
4044a647cb2a another (last?) round of K&R whitespace cosmetics
diego
parents: 8876
diff changeset
214 switch(s->mv_type) {
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
215 case MV_TYPE_16X16:
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
216 mv_block->motion_type = XVMC_PREDICTION_FRAME;
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
217 break;
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
218 case MV_TYPE_16X8:
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
219 mv_block->motion_type = XVMC_PREDICTION_16x8;
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
220 break;
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
221 case MV_TYPE_FIELD:
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
222 mv_block->motion_type = XVMC_PREDICTION_FIELD;
8835
8c11ebbca9b3 whitespace cosmetics: consistent (more or less) K&R style
diego
parents: 8834
diff changeset
223 if (s->picture_structure == PICT_FRAME) {
8c11ebbca9b3 whitespace cosmetics: consistent (more or less) K&R style
diego
parents: 8834
diff changeset
224 mv_block->PMV[0][0][1] <<= 1;
8c11ebbca9b3 whitespace cosmetics: consistent (more or less) K&R style
diego
parents: 8834
diff changeset
225 mv_block->PMV[1][0][1] <<= 1;
8c11ebbca9b3 whitespace cosmetics: consistent (more or less) K&R style
diego
parents: 8834
diff changeset
226 mv_block->PMV[0][1][1] <<= 1;
8c11ebbca9b3 whitespace cosmetics: consistent (more or less) K&R style
diego
parents: 8834
diff changeset
227 mv_block->PMV[1][1][1] <<= 1;
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
228 }
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
229 break;
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
230 case MV_TYPE_DMV:
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
231 mv_block->motion_type = XVMC_PREDICTION_DUAL_PRIME;
8835
8c11ebbca9b3 whitespace cosmetics: consistent (more or less) K&R style
diego
parents: 8834
diff changeset
232 if (s->picture_structure == PICT_FRAME) {
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
233
8876
ae637f4c4e55 whitespace cosmetics: Align some comments.
diego
parents: 8875
diff changeset
234 mv_block->PMV[0][0][0] = s->mv[0][0][0]; // top from top
8875
266bd7d64985 whitespace cosmetics: Put some spaces around operators for better readability.
diego
parents: 8874
diff changeset
235 mv_block->PMV[0][0][1] = s->mv[0][0][1] << 1;
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
236
8876
ae637f4c4e55 whitespace cosmetics: Align some comments.
diego
parents: 8875
diff changeset
237 mv_block->PMV[0][1][0] = s->mv[0][0][0]; // bottom from bottom
8875
266bd7d64985 whitespace cosmetics: Put some spaces around operators for better readability.
diego
parents: 8874
diff changeset
238 mv_block->PMV[0][1][1] = s->mv[0][0][1] << 1;
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
239
8876
ae637f4c4e55 whitespace cosmetics: Align some comments.
diego
parents: 8875
diff changeset
240 mv_block->PMV[1][0][0] = s->mv[0][2][0]; // dmv00, top from bottom
8875
266bd7d64985 whitespace cosmetics: Put some spaces around operators for better readability.
diego
parents: 8874
diff changeset
241 mv_block->PMV[1][0][1] = s->mv[0][2][1] << 1; // dmv01
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
242
8876
ae637f4c4e55 whitespace cosmetics: Align some comments.
diego
parents: 8875
diff changeset
243 mv_block->PMV[1][1][0] = s->mv[0][3][0]; // dmv10, bottom from top
8875
266bd7d64985 whitespace cosmetics: Put some spaces around operators for better readability.
diego
parents: 8874
diff changeset
244 mv_block->PMV[1][1][1] = s->mv[0][3][1] << 1; // dmv11
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
245
8861
da23b389e856 cosmetics: Consistently format all if/else statements in K&R style
diego
parents: 8860
diff changeset
246 } else {
8876
ae637f4c4e55 whitespace cosmetics: Align some comments.
diego
parents: 8875
diff changeset
247 mv_block->PMV[0][1][0] = s->mv[0][2][0]; // dmv00
ae637f4c4e55 whitespace cosmetics: Align some comments.
diego
parents: 8875
diff changeset
248 mv_block->PMV[0][1][1] = s->mv[0][2][1]; // dmv01
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
249 }
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
250 break;
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
251 default:
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
252 assert(0);
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
253 }
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
254
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
255 mv_block->motion_vertical_field_select = 0;
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
256
8867
0694627caaea whitespace cosmetics: consistently format all comments
diego
parents: 8866
diff changeset
257 // set correct field references
8835
8c11ebbca9b3 whitespace cosmetics: consistent (more or less) K&R style
diego
parents: 8834
diff changeset
258 if (s->mv_type == MV_TYPE_FIELD || s->mv_type == MV_TYPE_16X8) {
8862
5aa3cfee2291 get rid of the branch prediction
iive
parents: 8861
diff changeset
259 mv_block->motion_vertical_field_select |= s->field_select[0][0];
8871
199e9809eb5a some more K&R whitespace cosmetics
diego
parents: 8870
diff changeset
260 mv_block->motion_vertical_field_select |= s->field_select[1][0] << 1;
199e9809eb5a some more K&R whitespace cosmetics
diego
parents: 8870
diff changeset
261 mv_block->motion_vertical_field_select |= s->field_select[0][1] << 2;
199e9809eb5a some more K&R whitespace cosmetics
diego
parents: 8870
diff changeset
262 mv_block->motion_vertical_field_select |= s->field_select[1][1] << 3;
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
263 }
8867
0694627caaea whitespace cosmetics: consistently format all comments
diego
parents: 8866
diff changeset
264 } // !intra
8870
c5112df7f8b8 cosmetics: grammar/spelling/wording fixes in comments
diego
parents: 8869
diff changeset
265 // time to handle data blocks
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
266 mv_block->index = render->next_free_data_block_num;
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
267
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
268 blocks_per_mb = 6;
8835
8c11ebbca9b3 whitespace cosmetics: consistent (more or less) K&R style
diego
parents: 8834
diff changeset
269 if (s->chroma_format >= 2) {
8856
e5cf06287032 remove some useless ()
iive
parents: 8855
diff changeset
270 blocks_per_mb = 4 + (1 << s->chroma_format);
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
271 }
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
272
8867
0694627caaea whitespace cosmetics: consistently format all comments
diego
parents: 8866
diff changeset
273 // calculate cbp
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
274 cbp = 0;
8835
8c11ebbca9b3 whitespace cosmetics: consistent (more or less) K&R style
diego
parents: 8834
diff changeset
275 for (i = 0; i < blocks_per_mb; i++) {
8836
4c175e9d492e whitespace cosmetics: another round of formatting consistency fixes
diego
parents: 8835
diff changeset
276 cbp += cbp;
8835
8c11ebbca9b3 whitespace cosmetics: consistent (more or less) K&R style
diego
parents: 8834
diff changeset
277 if (s->block_last_index[i] >= 0)
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
278 cbp++;
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
279 }
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
280
8835
8c11ebbca9b3 whitespace cosmetics: consistent (more or less) K&R style
diego
parents: 8834
diff changeset
281 if (s->flags & CODEC_FLAG_GRAY) {
8876
ae637f4c4e55 whitespace cosmetics: Align some comments.
diego
parents: 8875
diff changeset
282 if (s->mb_intra) { // intra frames are always full chroma blocks
8835
8c11ebbca9b3 whitespace cosmetics: consistent (more or less) K&R style
diego
parents: 8834
diff changeset
283 for (i = 4; i < blocks_per_mb; i++) {
8886
f873e2b1ac7f cosmetics: Change '8*8' to '64'.
diego
parents: 8885
diff changeset
284 memset(s->pblocks[i], 0, sizeof(short)*64); // so we need to clear them
8835
8c11ebbca9b3 whitespace cosmetics: consistent (more or less) K&R style
diego
parents: 8834
diff changeset
285 if (!render->unsigned_intra)
8c11ebbca9b3 whitespace cosmetics: consistent (more or less) K&R style
diego
parents: 8834
diff changeset
286 s->pblocks[i][0] = 1 << 10;
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
287 }
8861
da23b389e856 cosmetics: Consistently format all if/else statements in K&R style
diego
parents: 8860
diff changeset
288 } else {
8835
8c11ebbca9b3 whitespace cosmetics: consistent (more or less) K&R style
diego
parents: 8834
diff changeset
289 cbp &= 0xf << (blocks_per_mb - 4);
8876
ae637f4c4e55 whitespace cosmetics: Align some comments.
diego
parents: 8875
diff changeset
290 blocks_per_mb = 4; // luminance blocks only
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
291 }
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
292 }
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
293 mv_block->coded_block_pattern = cbp;
8835
8c11ebbca9b3 whitespace cosmetics: consistent (more or less) K&R style
diego
parents: 8834
diff changeset
294 if (cbp == 0)
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
295 mv_block->macroblock_type &= ~XVMC_MB_TYPE_PATTERN;
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
296
8835
8c11ebbca9b3 whitespace cosmetics: consistent (more or less) K&R style
diego
parents: 8834
diff changeset
297 for (i = 0; i < blocks_per_mb; i++) {
8c11ebbca9b3 whitespace cosmetics: consistent (more or less) K&R style
diego
parents: 8834
diff changeset
298 if (s->block_last_index[i] >= 0) {
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
299 // I do not have unsigned_intra MOCO to test, hope it is OK.
8868
2534e44a9472 Restore one set of parentheses to avoid the warning:
diego
parents: 8867
diff changeset
300 if (s->mb_intra && (render->idct || (!render->idct && !render->unsigned_intra)))
8835
8c11ebbca9b3 whitespace cosmetics: consistent (more or less) K&R style
diego
parents: 8834
diff changeset
301 s->pblocks[i][0] -= 1 << 10;
8c11ebbca9b3 whitespace cosmetics: consistent (more or less) K&R style
diego
parents: 8834
diff changeset
302 if (!render->idct) {
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
303 s->dsp.idct(s->pblocks[i]);
8870
c5112df7f8b8 cosmetics: grammar/spelling/wording fixes in comments
diego
parents: 8869
diff changeset
304 /* It is unclear if MC hardware requires pixel diff values to be
c5112df7f8b8 cosmetics: grammar/spelling/wording fixes in comments
diego
parents: 8869
diff changeset
305 * in the range [-255;255]. TODO: Clipping if such hardware is
c5112df7f8b8 cosmetics: grammar/spelling/wording fixes in comments
diego
parents: 8869
diff changeset
306 * ever found. As of now it would only be an unnecessary
c5112df7f8b8 cosmetics: grammar/spelling/wording fixes in comments
diego
parents: 8869
diff changeset
307 * slowdown. */
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
308 }
8867
0694627caaea whitespace cosmetics: consistently format all comments
diego
parents: 8866
diff changeset
309 // copy blocks only if the codec doesn't support pblocks reordering
8835
8c11ebbca9b3 whitespace cosmetics: consistent (more or less) K&R style
diego
parents: 8834
diff changeset
310 if (s->avctx->xvmc_acceleration == 1) {
8856
e5cf06287032 remove some useless ()
iive
parents: 8855
diff changeset
311 memcpy(&render->data_blocks[render->next_free_data_block_num*64],
8880
d8f9926e684d Cosmetic 64 instead 8*8
iive
parents: 8879
diff changeset
312 s->pblocks[i], sizeof(short)*64);
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
313 }
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
314 render->next_free_data_block_num++;
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
315 }
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
316 }
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
317 render->filled_mv_blocks_num++;
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
318
8944
1b00a4a76ade Replace two asserts by checks and error messages.
diego
parents: 8942
diff changeset
319
1b00a4a76ade Replace two asserts by checks and error messages.
diego
parents: 8942
diff changeset
320 if (render->filled_mv_blocks_num > render->allocated_mv_blocks)
1b00a4a76ade Replace two asserts by checks and error messages.
diego
parents: 8942
diff changeset
321 av_log(s->avctx, AV_LOG_ERROR,
1b00a4a76ade Replace two asserts by checks and error messages.
diego
parents: 8942
diff changeset
322 "Not enough space to store mv blocks allocated.\n");
1b00a4a76ade Replace two asserts by checks and error messages.
diego
parents: 8942
diff changeset
323
1b00a4a76ade Replace two asserts by checks and error messages.
diego
parents: 8942
diff changeset
324 if (render->next_free_data_block_num > render->allocated_data_blocks)
1b00a4a76ade Replace two asserts by checks and error messages.
diego
parents: 8942
diff changeset
325 av_log(s->avctx, AV_LOG_ERROR,
1b00a4a76ade Replace two asserts by checks and error messages.
diego
parents: 8942
diff changeset
326 "Offset to next data block exceeds number of allocated data blocks.\n");
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
327
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
328
8941
42a048ecbba3 Give struct members more sensible names:
diego
parents: 8934
diff changeset
329 if (render->filled_mv_blocks_num == render->allocated_mv_blocks)
8871
199e9809eb5a some more K&R whitespace cosmetics
diego
parents: 8870
diff changeset
330 ff_draw_horiz_band(s, 0, 0);
8831
27616b326d57 Rename xvmc.c --> mpegvideo_xvmc.c as suggested by Ivan.
diego
parents:
diff changeset
331 }