Mercurial > libavcodec.hg
annotate simple_idct.h @ 8130:c45366b01126 libavcodec
ARM: fix j_rev_dct_ARM
This is a bugfix for ARMv4 assembly implementation of 'j_rev_dct'
function.
The problem was in the incorrect partially empty row detection. Even
if the first two coefficients in the row were nonzero, it handled this
just like the case with only the first nonzero coefficient.
Now this function produces exactly the same output as the stripped
down reference C version of 'j_rev_dct' (with the nested checks like
'if (d6) { if (d2) { ...' always evaluated as true, avoiding shortcut
branches).
author | mru |
---|---|
date | Wed, 12 Nov 2008 20:23:36 +0000 |
parents | c4a4495715dd |
children | e9d9d946f213 |
rev | line source |
---|---|
175
bd77d3cbb233
new IDCT code by Michael Niedermayer (michaelni@gmx.at) - #define SIMPLE_IDCT to enable
arpi_esp
parents:
diff
changeset
|
1 /* |
429 | 2 * Simple IDCT |
3 * | |
4 * Copyright (c) 2001 Michael Niedermayer <michaelni@gmx.at> | |
5 * | |
3947
c8c591fe26f8
Change license headers to say 'FFmpeg' instead of 'this program/this library'
diego
parents:
3036
diff
changeset
|
6 * This file is part of FFmpeg. |
c8c591fe26f8
Change license headers to say 'FFmpeg' instead of 'this program/this library'
diego
parents:
3036
diff
changeset
|
7 * |
c8c591fe26f8
Change license headers to say 'FFmpeg' instead of 'this program/this library'
diego
parents:
3036
diff
changeset
|
8 * FFmpeg is free software; you can redistribute it and/or |
429 | 9 * modify it under the terms of the GNU Lesser General Public |
10 * License as published by the Free Software Foundation; either | |
3947
c8c591fe26f8
Change license headers to say 'FFmpeg' instead of 'this program/this library'
diego
parents:
3036
diff
changeset
|
11 * version 2.1 of the License, or (at your option) any later version. |
429 | 12 * |
3947
c8c591fe26f8
Change license headers to say 'FFmpeg' instead of 'this program/this library'
diego
parents:
3036
diff
changeset
|
13 * FFmpeg is distributed in the hope that it will be useful, |
429 | 14 * but WITHOUT ANY WARRANTY; without even the implied warranty of |
15 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU | |
16 * Lesser General Public License for more details. | |
17 * | |
18 * You should have received a copy of the GNU Lesser General Public | |
3947
c8c591fe26f8
Change license headers to say 'FFmpeg' instead of 'this program/this library'
diego
parents:
3036
diff
changeset
|
19 * License along with FFmpeg; if not, write to the Free Software |
3036
0b546eab515d
Update licensing information: The FSF changed postal address.
diego
parents:
2967
diff
changeset
|
20 * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA |
429 | 21 */ |
175
bd77d3cbb233
new IDCT code by Michael Niedermayer (michaelni@gmx.at) - #define SIMPLE_IDCT to enable
arpi_esp
parents:
diff
changeset
|
22 |
1106 | 23 /** |
24 * @file simple_idct.h | |
25 * simple idct header. | |
26 */ | |
2967 | 27 |
7760 | 28 #ifndef AVCODEC_SIMPLE_IDCT_H |
29 #define AVCODEC_SIMPLE_IDCT_H | |
5163 | 30 |
5162 | 31 #include <stdint.h> |
32 #include "dsputil.h" | |
33 | |
6001 | 34 void ff_simple_idct_put(uint8_t *dest, int line_size, DCTELEM *block); |
35 void ff_simple_idct_add(uint8_t *dest, int line_size, DCTELEM *block); | |
1008 | 36 void ff_simple_idct_mmx(int16_t *block); |
1064 | 37 void ff_simple_idct_add_mmx(uint8_t *dest, int line_size, int16_t *block); |
38 void ff_simple_idct_put_mmx(uint8_t *dest, int line_size, int16_t *block); | |
6001 | 39 void ff_simple_idct(DCTELEM *block); |
719 | 40 |
6001 | 41 void ff_simple_idct248_put(uint8_t *dest, int line_size, DCTELEM *block); |
936 | 42 |
6001 | 43 void ff_simple_idct84_add(uint8_t *dest, int line_size, DCTELEM *block); |
44 void ff_simple_idct48_add(uint8_t *dest, int line_size, DCTELEM *block); | |
6007 | 45 void ff_simple_idct44_add(uint8_t *dest, int line_size, DCTELEM *block); |
5163 | 46 |
7760 | 47 #endif /* AVCODEC_SIMPLE_IDCT_H */ |