Mercurial > libavcodec.hg
annotate mpegvideo_xvmc.c @ 8887:c4f2086d74f8 libavcodec
cosmetics: comment wording/grammar
author | diego |
---|---|
date | Sat, 14 Feb 2009 21:52:44 +0000 |
parents | f873e2b1ac7f |
children | 5b7c6ebb4a36 |
rev | line source |
---|---|
8831 | 1 /* |
2 * XVideo Motion Compensation | |
3 * Copyright (c) 2003 Ivan Kalvachev | |
4 * | |
5 * This file is part of FFmpeg. | |
6 * | |
7 * FFmpeg is free software; you can redistribute it and/or | |
8 * modify it under the terms of the GNU Lesser General Public | |
9 * License as published by the Free Software Foundation; either | |
10 * version 2.1 of the License, or (at your option) any later version. | |
11 * | |
12 * FFmpeg is distributed in the hope that it will be useful, | |
13 * but WITHOUT ANY WARRANTY; without even the implied warranty of | |
14 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU | |
15 * Lesser General Public License for more details. | |
16 * | |
17 * You should have received a copy of the GNU Lesser General Public | |
18 * License along with FFmpeg; if not, write to the Free Software | |
19 * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA | |
20 */ | |
21 | |
22 #include <limits.h> | |
23 | |
24 #include "avcodec.h" | |
25 #include "dsputil.h" | |
26 #include "mpegvideo.h" | |
27 | |
28 #undef NDEBUG | |
29 #include <assert.h> | |
30 | |
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 | 33 |
34 //set s->block | |
8855 | 35 void ff_xvmc_init_block(MpegEncContext *s) |
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
36 { |
8871 | 37 struct xvmc_render_state *render; |
8831 | 38 render = (struct xvmc_render_state*)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
|
39 assert(render); |
8860 | 40 if (!render || render->magic != AV_XVMC_RENDER_MAGIC) { |
8831 | 41 assert(0); |
8867
0694627caaea
whitespace cosmetics: consistently format all comments
diego
parents:
8866
diff
changeset
|
42 return; // make sure that this is a render packet |
8831 | 43 } |
8860 | 44 s->block = (DCTELEM *)(render->data_blocks + render->next_free_data_block_num * 64); |
8831 | 45 } |
46 | |
8855 | 47 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
|
48 { |
8871 | 49 int i, j; |
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
50 const int mb_block_count = 4 + (1 << s->chroma_format); |
8831 | 51 |
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
52 j = 0; |
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
53 cbp <<= 12-mb_block_count; |
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
54 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
|
55 if (cbp & (1 << 11)) |
8856 | 56 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
|
57 else |
8834 | 58 s->pblocks[i] = NULL; |
8831 | 59 cbp+=cbp; |
60 } | |
61 } | |
62 | |
8867
0694627caaea
whitespace cosmetics: consistently format all comments
diego
parents:
8866
diff
changeset
|
63 // These functions should be called on every new field and/or frame. |
0694627caaea
whitespace cosmetics: consistently format all comments
diego
parents:
8866
diff
changeset
|
64 // They should be safe if they are called a few times for the same field! |
8855 | 65 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
|
66 { |
8871 | 67 struct xvmc_render_state *render, *last, *next; |
8831 | 68 |
8844
9ea7195b2ed4
Replace all (x == NULL) or (x != NULL) in assert and if conditions by !x and x.
diego
parents:
8842
diff
changeset
|
69 assert(avctx); |
8831 | 70 |
71 render = (struct xvmc_render_state*)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
|
72 assert(render); |
8860 | 73 if (!render || render->magic != AV_XVMC_RENDER_MAGIC) |
8870
c5112df7f8b8
cosmetics: grammar/spelling/wording fixes in comments
diego
parents:
8869
diff
changeset
|
74 return -1; // make sure that this is a render packet |
8831 | 75 |
76 render->picture_structure = s->picture_structure; | |
8860 | 77 render->flags = s->first_field ? 0 : XVMC_SECOND_FIELD; |
8831 | 78 |
8885 | 79 if (render->filled_mv_blocks_num) { |
8884 | 80 av_log(avctx, AV_LOG_ERROR, |
8885 | 81 "Rendering surface contains %i unprocessed blocks\n", |
82 render->filled_mv_blocks_num); | |
8884 | 83 assert(0); |
84 } | |
8831 | 85 |
86 render->p_future_surface = NULL; | |
8837 | 87 render->p_past_surface = NULL; |
8831 | 88 |
8871 | 89 switch(s->pict_type) { |
8831 | 90 case FF_I_TYPE: |
8867
0694627caaea
whitespace cosmetics: consistently format all comments
diego
parents:
8866
diff
changeset
|
91 return 0; // no prediction from other frames |
8831 | 92 case FF_B_TYPE: |
93 next = (struct xvmc_render_state*)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
|
94 assert(next); |
9ea7195b2ed4
Replace all (x == NULL) or (x != NULL) in assert and if conditions by !x and x.
diego
parents:
8842
diff
changeset
|
95 if (!next) |
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
96 return -1; |
8845
aee8c48ff4b0
Give public #defines that are used by MPlayer an AV_ prefix.
diego
parents:
8844
diff
changeset
|
97 if (next->magic != AV_XVMC_RENDER_MAGIC) |
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
98 return -1; |
8831 | 99 render->p_future_surface = next->p_surface; |
8867
0694627caaea
whitespace cosmetics: consistently format all comments
diego
parents:
8866
diff
changeset
|
100 // no return here, going to set forward prediction |
8831 | 101 case FF_P_TYPE: |
102 last = (struct xvmc_render_state*)s->last_picture.data[2]; | |
8872 | 103 if (!last) |
8867
0694627caaea
whitespace cosmetics: consistently format all comments
diego
parents:
8866
diff
changeset
|
104 last = render; // predict second field from the first |
8845
aee8c48ff4b0
Give public #defines that are used by MPlayer an AV_ prefix.
diego
parents:
8844
diff
changeset
|
105 if (last->magic != AV_XVMC_RENDER_MAGIC) |
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
106 return -1; |
8831 | 107 render->p_past_surface = last->p_surface; |
108 return 0; | |
109 } | |
110 | |
111 return -1; | |
112 } | |
113 | |
8855 | 114 void ff_xvmc_field_end(MpegEncContext *s) |
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
115 { |
8871 | 116 struct xvmc_render_state *render; |
8831 | 117 render = (struct xvmc_render_state*)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
|
118 assert(render); |
8831 | 119 |
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
120 if (render->filled_mv_blocks_num > 0) |
8877 | 121 ff_draw_horiz_band(s, 0, 0); |
8831 | 122 } |
123 | |
8855 | 124 void ff_xvmc_decode_mb(MpegEncContext *s) |
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
125 { |
8871 | 126 XvMCMacroBlock *mv_block; |
127 struct xvmc_render_state *render; | |
128 int i, cbp, blocks_per_mb; | |
8831 | 129 |
8834 | 130 const int mb_xy = s->mb_y * s->mb_stride + s->mb_x; |
8831 | 131 |
132 | |
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
133 if (s->encoding) { |
8831 | 134 av_log(s->avctx, AV_LOG_ERROR, "XVMC doesn't support encoding!!!\n"); |
135 return; | |
136 } | |
137 | |
8867
0694627caaea
whitespace cosmetics: consistently format all comments
diego
parents:
8866
diff
changeset
|
138 // from MPV_decode_mb(), update DC predictors for P macroblocks |
8831 | 139 if (!s->mb_intra) { |
140 s->last_dc[0] = | |
141 s->last_dc[1] = | |
142 s->last_dc[2] = 128 << s->intra_dc_precision; | |
143 } | |
144 | |
8867
0694627caaea
whitespace cosmetics: consistently format all comments
diego
parents:
8866
diff
changeset
|
145 // MC doesn't skip blocks |
8831 | 146 s->mb_skipped = 0; |
147 | |
148 | |
8866
09d0ff5cdc81
whitespace cosmetics: Correctly indent all comments.
diego
parents:
8865
diff
changeset
|
149 // Do I need to export quant when I could not perform postprocessing? |
09d0ff5cdc81
whitespace cosmetics: Correctly indent all comments.
diego
parents:
8865
diff
changeset
|
150 // Anyway, it doesn't hurt. |
8831 | 151 s->current_picture.qscale_table[mb_xy] = s->qscale; |
152 | |
8870
c5112df7f8b8
cosmetics: grammar/spelling/wording fixes in comments
diego
parents:
8869
diff
changeset
|
153 // start of XVMC-specific code |
8831 | 154 render = (struct xvmc_render_state*)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
|
155 assert(render); |
8871 | 156 assert(render->magic == AV_XVMC_RENDER_MAGIC); |
8831 | 157 assert(render->mv_blocks); |
158 | |
8867
0694627caaea
whitespace cosmetics: consistently format all comments
diego
parents:
8866
diff
changeset
|
159 // take the next free macroblock |
8831 | 160 mv_block = &render->mv_blocks[render->start_mv_blocks_num + |
8834 | 161 render->filled_mv_blocks_num ]; |
8831 | 162 |
8837 | 163 mv_block->x = s->mb_x; |
164 mv_block->y = s->mb_y; | |
8867
0694627caaea
whitespace cosmetics: consistently format all comments
diego
parents:
8866
diff
changeset
|
165 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
|
166 if (s->mb_intra) { |
8867
0694627caaea
whitespace cosmetics: consistently format all comments
diego
parents:
8866
diff
changeset
|
167 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
|
168 } else { |
8831 | 169 mv_block->macroblock_type = XVMC_MB_TYPE_PATTERN; |
170 | |
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
171 if (s->mv_dir & MV_DIR_FORWARD) { |
8836
4c175e9d492e
whitespace cosmetics: another round of formatting consistency fixes
diego
parents:
8835
diff
changeset
|
172 mv_block->macroblock_type |= XVMC_MB_TYPE_MOTION_FORWARD; |
8873
02c59b74ba71
Make one comment slightly clearer and more readable.
diego
parents:
8872
diff
changeset
|
173 // PMV[n][dir][xy] = mv[dir][n][xy] |
8831 | 174 mv_block->PMV[0][0][0] = s->mv[0][0][0]; |
175 mv_block->PMV[0][0][1] = s->mv[0][0][1]; | |
176 mv_block->PMV[1][0][0] = s->mv[0][1][0]; | |
177 mv_block->PMV[1][0][1] = s->mv[0][1][1]; | |
178 } | |
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
179 if (s->mv_dir & MV_DIR_BACKWARD) { |
8836
4c175e9d492e
whitespace cosmetics: another round of formatting consistency fixes
diego
parents:
8835
diff
changeset
|
180 mv_block->macroblock_type |= XVMC_MB_TYPE_MOTION_BACKWARD; |
8831 | 181 mv_block->PMV[0][1][0] = s->mv[1][0][0]; |
182 mv_block->PMV[0][1][1] = s->mv[1][0][1]; | |
183 mv_block->PMV[1][1][0] = s->mv[1][1][0]; | |
184 mv_block->PMV[1][1][1] = s->mv[1][1][1]; | |
185 } | |
186 | |
8877 | 187 switch(s->mv_type) { |
8831 | 188 case MV_TYPE_16X16: |
189 mv_block->motion_type = XVMC_PREDICTION_FRAME; | |
190 break; | |
191 case MV_TYPE_16X8: | |
192 mv_block->motion_type = XVMC_PREDICTION_16x8; | |
193 break; | |
194 case MV_TYPE_FIELD: | |
195 mv_block->motion_type = XVMC_PREDICTION_FIELD; | |
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
196 if (s->picture_structure == PICT_FRAME) { |
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
197 mv_block->PMV[0][0][1] <<= 1; |
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
198 mv_block->PMV[1][0][1] <<= 1; |
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
199 mv_block->PMV[0][1][1] <<= 1; |
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
200 mv_block->PMV[1][1][1] <<= 1; |
8831 | 201 } |
202 break; | |
203 case MV_TYPE_DMV: | |
204 mv_block->motion_type = XVMC_PREDICTION_DUAL_PRIME; | |
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
205 if (s->picture_structure == PICT_FRAME) { |
8831 | 206 |
8876 | 207 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
|
208 mv_block->PMV[0][0][1] = s->mv[0][0][1] << 1; |
8831 | 209 |
8876 | 210 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
|
211 mv_block->PMV[0][1][1] = s->mv[0][0][1] << 1; |
8831 | 212 |
8876 | 213 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
|
214 mv_block->PMV[1][0][1] = s->mv[0][2][1] << 1; // dmv01 |
8831 | 215 |
8876 | 216 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
|
217 mv_block->PMV[1][1][1] = s->mv[0][3][1] << 1; // dmv11 |
8831 | 218 |
8861
da23b389e856
cosmetics: Consistently format all if/else statements in K&R style
diego
parents:
8860
diff
changeset
|
219 } else { |
8876 | 220 mv_block->PMV[0][1][0] = s->mv[0][2][0]; // dmv00 |
221 mv_block->PMV[0][1][1] = s->mv[0][2][1]; // dmv01 | |
8831 | 222 } |
223 break; | |
224 default: | |
225 assert(0); | |
226 } | |
227 | |
228 mv_block->motion_vertical_field_select = 0; | |
229 | |
8867
0694627caaea
whitespace cosmetics: consistently format all comments
diego
parents:
8866
diff
changeset
|
230 // set correct field references |
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
231 if (s->mv_type == MV_TYPE_FIELD || s->mv_type == MV_TYPE_16X8) { |
8862 | 232 mv_block->motion_vertical_field_select |= s->field_select[0][0]; |
8871 | 233 mv_block->motion_vertical_field_select |= s->field_select[1][0] << 1; |
234 mv_block->motion_vertical_field_select |= s->field_select[0][1] << 2; | |
235 mv_block->motion_vertical_field_select |= s->field_select[1][1] << 3; | |
8831 | 236 } |
8867
0694627caaea
whitespace cosmetics: consistently format all comments
diego
parents:
8866
diff
changeset
|
237 } // !intra |
8870
c5112df7f8b8
cosmetics: grammar/spelling/wording fixes in comments
diego
parents:
8869
diff
changeset
|
238 // time to handle data blocks |
8831 | 239 mv_block->index = render->next_free_data_block_num; |
240 | |
241 blocks_per_mb = 6; | |
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
242 if (s->chroma_format >= 2) { |
8856 | 243 blocks_per_mb = 4 + (1 << s->chroma_format); |
8831 | 244 } |
245 | |
8867
0694627caaea
whitespace cosmetics: consistently format all comments
diego
parents:
8866
diff
changeset
|
246 // calculate cbp |
8831 | 247 cbp = 0; |
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
248 for (i = 0; i < blocks_per_mb; i++) { |
8836
4c175e9d492e
whitespace cosmetics: another round of formatting consistency fixes
diego
parents:
8835
diff
changeset
|
249 cbp += cbp; |
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
250 if (s->block_last_index[i] >= 0) |
8831 | 251 cbp++; |
252 } | |
253 | |
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
254 if (s->flags & CODEC_FLAG_GRAY) { |
8876 | 255 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
|
256 for (i = 4; i < blocks_per_mb; i++) { |
8886 | 257 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
|
258 if (!render->unsigned_intra) |
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
259 s->pblocks[i][0] = 1 << 10; |
8831 | 260 } |
8861
da23b389e856
cosmetics: Consistently format all if/else statements in K&R style
diego
parents:
8860
diff
changeset
|
261 } else { |
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
262 cbp &= 0xf << (blocks_per_mb - 4); |
8876 | 263 blocks_per_mb = 4; // luminance blocks only |
8831 | 264 } |
265 } | |
266 mv_block->coded_block_pattern = cbp; | |
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
267 if (cbp == 0) |
8831 | 268 mv_block->macroblock_type &= ~XVMC_MB_TYPE_PATTERN; |
269 | |
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
270 for (i = 0; i < blocks_per_mb; i++) { |
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
271 if (s->block_last_index[i] >= 0) { |
8831 | 272 // 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
|
273 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
|
274 s->pblocks[i][0] -= 1 << 10; |
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
275 if (!render->idct) { |
8831 | 276 s->dsp.idct(s->pblocks[i]); |
8870
c5112df7f8b8
cosmetics: grammar/spelling/wording fixes in comments
diego
parents:
8869
diff
changeset
|
277 /* 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
|
278 * in the range [-255;255]. TODO: Clipping if such hardware is |
c5112df7f8b8
cosmetics: grammar/spelling/wording fixes in comments
diego
parents:
8869
diff
changeset
|
279 * ever found. As of now it would only be an unnecessary |
c5112df7f8b8
cosmetics: grammar/spelling/wording fixes in comments
diego
parents:
8869
diff
changeset
|
280 * slowdown. */ |
8831 | 281 } |
8867
0694627caaea
whitespace cosmetics: consistently format all comments
diego
parents:
8866
diff
changeset
|
282 // 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
|
283 if (s->avctx->xvmc_acceleration == 1) { |
8856 | 284 memcpy(&render->data_blocks[render->next_free_data_block_num*64], |
8880 | 285 s->pblocks[i], sizeof(short)*64); |
8831 | 286 } |
287 render->next_free_data_block_num++; | |
288 } | |
289 } | |
290 render->filled_mv_blocks_num++; | |
291 | |
8837 | 292 assert(render->filled_mv_blocks_num <= render->total_number_of_mv_blocks); |
8831 | 293 assert(render->next_free_data_block_num <= render->total_number_of_data_blocks); |
8887 | 294 /* The above conditions should not be able to fail as long as this function |
295 * is used and the following 'if ()' automatically calls a callback to free | |
296 * blocks. */ | |
8831 | 297 |
298 | |
8835
8c11ebbca9b3
whitespace cosmetics: consistent (more or less) K&R style
diego
parents:
8834
diff
changeset
|
299 if (render->filled_mv_blocks_num >= render->total_number_of_mv_blocks) |
8871 | 300 ff_draw_horiz_band(s, 0, 0); |
8831 | 301 } |