summaryrefslogtreecommitdiff
path: root/libavcodec/jrevdct.c
Commit message (Collapse)AuthorAge
* Remove unnecessary libavutil/(avutil|common|internal).h inclusionsAndreas Rheinhardt2022-02-24
| | | | | | | | | | Some of these were made possible by moving several common macros to libavutil/macros.h. While just at it, also improve the other headers a bit. Reviewed-by: Martin Storsjö <martin@martin.st> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
* lavc/jrevdct: Avoid an aliasing violation.Carl Eugen Hoyos2018-11-27
| | | | | | Fixes fate on different PowerPC systems with some compilers. Analyzed-by: Lauri Kasanen
* jrev/xvid: hardcode use of C put/add_pixels_clamped.Ronald S. Bultje2017-04-06
| | | | | | | | | | | | | | | | | | | This removes the last use of the ff_put/add_pixels_clamped global function pointers, and as such they are removed. This patch has a negative effect on performance on MIPS, since there's a SIMD-optimized put/add_pixels_clamped, but no xvid or jrev. From a code maintenance point of view, that is probably acceptable. Because the global function pointers are removed, this fixes the following tsan warnings when running e.g. fate-dnxhr-parse: WARNING: ThreadSanitizer: data race (pid=29917) Write of size 8 at 0x0000025b12d8 by thread T2 (mutexes: write M1543): #0 ff_idctdsp_init src/libavcodec/idctdsp.c:313 (ffmpeg+0x00000044b68e) [..] Previous write of size 8 at 0x0000025b12d8 by thread T1 (mutexes: write M1541): #0 ff_idctdsp_init src/libavcodec/idctdsp.c:313 (ffmpeg+0x00000044b68e)
* Merge commit '2ec9fa5ec60dcd10e1cb10d8b4e4437e634ea428'James Almer2017-03-21
|\ | | | | | | | | | | | | * commit '2ec9fa5ec60dcd10e1cb10d8b4e4437e634ea428': idct: Change type of array stride parameters to ptrdiff_t Merged-by: James Almer <jamrial@gmail.com>
| * idct: Change type of array stride parameters to ptrdiff_tDiego Biurrun2016-09-29
| | | | | | | | ptrdiff_t is the correct type for array strides and similar.
| * idct8x8: Fix undefined negative shiftsKaterina Barone-Adesi2016-03-05
| | | | | | | | | | | | | | | | | | | | | | The original code left-shifts negative values, which is undefined in the C99 specification (the one used during normal Libav compilation). This change multiplies by (1 << shift), which is functionally equivalent, but has defined behavior. With this change, fate-idct8x8 compiled with --fsanitize=undefined works. Bug-Id: 686
* | Merge commit '95c0cec03acec0a80cc1c7db48f3b2355d9e767b'Michael Niedermayer2014-09-03
|\| | | | | | | | | | | | | | | | | | | | | | | | | * commit '95c0cec03acec0a80cc1c7db48f3b2355d9e767b': idctdsp: Add global function pointers for {add|put}_pixels_clamped functions Conflicts: libavcodec/arm/idctdsp_init_arm.c libavcodec/dct.h libavcodec/idctdsp.c libavcodec/jrevdct.c Merged-by: Michael Niedermayer <michaelni@gmx.at>
| * idctdsp: Add global function pointers for {add|put}_pixels_clamped functionsDiego Biurrun2014-09-02
| | | | | | | | | | | | These function pointers already existed in the ARM code. Adding them globally allows calls to the function pointers to access arch-optimized versions of the functions transparently.
* | avcodec/fate-idct8x8: Defined behavior: eliminate negative left-shifts.Katerina Barone-Adesi2014-05-20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | The original code left-shifts negative values, which is undefined in the C99 specification (the one used during normal FFmpeg compilation). This change multiplies by (1 << shift), which is functionally equivalent, but has defined behavior. With this change, fate-idct8x8 compiled with --fsanitize=undefined works. Bug-Id: 686 Signed-off-by: Michael Niedermayer <michaelni@gmx.at> The assembly generated by gcc is unchanged by this commit
* | Merge remote-tracking branch 'qatar/master'Michael Niedermayer2013-02-09
|\| | | | | | | | | | | | | | | | | | | * qatar/master: dsputil: Move fdct function declarations to dct.h Conflicts: libavcodec/dsputil.h Merged-by: Michael Niedermayer <michaelni@gmx.at>
| * dsputil: Move fdct function declarations to dct.hDiego Biurrun2013-02-09
| |
* | Merge commit '88bd7fdc821aaa0cbcf44cf075c62aaa42121e3f'Michael Niedermayer2013-01-23
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * commit '88bd7fdc821aaa0cbcf44cf075c62aaa42121e3f': Drop DCTELEM typedef Conflicts: libavcodec/alpha/dsputil_alpha.h libavcodec/alpha/motion_est_alpha.c libavcodec/arm/dsputil_init_armv6.c libavcodec/bfin/dsputil_bfin.h libavcodec/bfin/pixels_bfin.S libavcodec/cavs.c libavcodec/cavsdec.c libavcodec/dct-test.c libavcodec/dnxhdenc.c libavcodec/dsputil.c libavcodec/dsputil.h libavcodec/dsputil_template.c libavcodec/eamad.c libavcodec/h264_cavlc.c libavcodec/h264idct_template.c libavcodec/mpeg12.c libavcodec/mpegvideo.c libavcodec/mpegvideo.h libavcodec/mpegvideo_enc.c libavcodec/ppc/dsputil_altivec.c libavcodec/proresdsp.c Merged-by: Michael Niedermayer <michaelni@gmx.at>
| * Drop DCTELEM typedefDiego Biurrun2013-01-22
| | | | | | | | | | | | It does not help as an abstraction and adds dsputil dependencies. Signed-off-by: Ronald S. Bultje <rsbultje@gmail.com>
* | lowres2 support.Michael Niedermayer2012-04-22
|/ | | | | | | | | The new lowres support is limited to decoders where lowres decoding is possible in high quality. I was not able to measure any speed difference, but if one is found the 2-3 lines that might affect speed can be made compile time conditional Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
* Remove lowres video decodingMans Rullgard2012-04-21
| | | | | | | This feature is complex, of questionable utility, and slows down normal decoding. Signed-off-by: Mans Rullgard <mans@mansr.com>
* libavcodec: Add ff_ prefix to j_rev_dct*Martin Storsjö2012-02-15
| | | | Signed-off-by: Martin Storsjö <martin@martin.st>
* Remove redundant filename self-references inside files.Diego Biurrun2011-11-08
| | | | Filenames are brittle across renames and add no useful information.
* Remove explicit filename from Doxygen @file commands.Diego Biurrun2010-04-20
| | | | | | | | Passing an explicit filename to this command is only necessary if the documentation in the @file block refers to a file different from the one the block resides in. Originally committed as revision 22921 to svn://svn.ffmpeg.org/ffmpeg/trunk
* Remove unused assignment from j_rev_dct() found by CSA.Michael Niedermayer2009-04-17
| | | | Originally committed as revision 18558 to svn://svn.ffmpeg.org/ffmpeg/trunk
* Remvoe useless assignment found by CSA.Michael Niedermayer2009-04-17
| | | | Originally committed as revision 18551 to svn://svn.ffmpeg.org/ffmpeg/trunk
* Use full internal pathname in doxygen @file directives.Diego Biurrun2009-02-01
| | | | | | | Otherwise doxygen complains about ambiguous filenames when files exist under the same name in different subdirectories. Originally committed as revision 16912 to svn://svn.ffmpeg.org/ffmpeg/trunk
* Use full path for #includes from another directory.Diego Biurrun2008-05-09
| | | | Originally committed as revision 13098 to svn://svn.ffmpeg.org/ffmpeg/trunk
* Add copyright notice from the Independent JPEG Group instead of referringDiego Biurrun2006-09-03
| | | | | | to a README we don't include. Originally committed as revision 6157 to svn://svn.ffmpeg.org/ffmpeg/trunk
* COSMETICS: tabs --> spaces, some prettyprintingDiego Biurrun2005-12-22
| | | | Originally committed as revision 4764 to svn://svn.ffmpeg.org/ffmpeg/trunk
* COSMETICS: Remove all trailing whitespace.Diego Biurrun2005-12-17
| | | | Originally committed as revision 4749 to svn://svn.ffmpeg.org/ffmpeg/trunk
* porting optimizations from 4x4 dct to 8x8Michael Niedermayer2004-09-26
| | | | Originally committed as revision 3515 to svn://svn.ffmpeg.org/ffmpeg/trunk
* optimizing 4x4 idctMichael Niedermayer2004-09-26
| | | | Originally committed as revision 3514 to svn://svn.ffmpeg.org/ffmpeg/trunk
* 1/8 resolution decodingMichael Niedermayer2004-09-26
| | | | Originally committed as revision 3511 to svn://svn.ffmpeg.org/ffmpeg/trunk
* 1/4 resolution decodingMichael Niedermayer2004-09-26
| | | | Originally committed as revision 3509 to svn://svn.ffmpeg.org/ffmpeg/trunk
* 1/2 resolution decodingMichael Niedermayer2004-09-25
| | | | Originally committed as revision 3507 to svn://svn.ffmpeg.org/ffmpeg/trunk
* per file doxyMichael Niedermayer2003-03-06
| | | | Originally committed as revision 1634 to svn://svn.ffmpeg.org/ffmpeg/trunk
* * UINTX -> uintx_t INTX -> intx_tZdenek Kabelac2003-02-11
| | | | Originally committed as revision 1578 to svn://svn.ffmpeg.org/ffmpeg/trunk
* * started to cleanup name clashes for onetime compilationZdenek Kabelac2002-05-27
| | | | Originally committed as revision 617 to svn://svn.ffmpeg.org/ffmpeg/trunk
* permuted coefs in normal IDCT to avoid having different cases thereFabrice Bellard2001-08-07
| | | | Originally committed as revision 47 to svn://svn.ffmpeg.org/ffmpeg/trunk
* Initial revisionFabrice Bellard2001-07-22
Originally committed as revision 5 to svn://svn.ffmpeg.org/ffmpeg/trunk