[FFmpeg-devel] [PATCH]Use libopenjpeg by default if it was enabled

Michael Niedermayer michaelni at gmx.at
Tue Jun 9 20:07:55 CEST 2015

On Mon, Jun 08, 2015 at 10:01:18PM +0200, Carl Eugen Hoyos wrote:
> Hi!
> Attached patch fixes ticket #3619 for me.
> Please comment, Carl Eugen

>  allcodecs.c |    4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 96a81d0e28289471a9bfb353cecc1c8da289f8cd  patchj2kdefault.diff
> diff --git a/libavcodec/allcodecs.c b/libavcodec/allcodecs.c
> index ce97746..6f40b51 100644
> --- a/libavcodec/allcodecs.c
> +++ b/libavcodec/allcodecs.c
> @@ -104,6 +104,9 @@ void avcodec_register_all(void)
>      REGISTER_HWACCEL(WMV3_VAAPI,        wmv3_vaapi);
>      REGISTER_HWACCEL(WMV3_VDPAU,        wmv3_vdpau);
> +    /* libopenjpeg should be used by default if enabled */
> +    REGISTER_ENCDEC (LIBOPENJPEG,       libopenjpeg);

i tried to apply this and make it work but

it breaks fate
also i get random crashes with this with fuzzed files

i dont know if this issue is specific to old versions of libopenjpeg
ill push the patch that marks 1.3 as experimantal as that one
definitly doesnt work properly. Ill also fix the fate issue this
patch leads to (as i already fixed it locally) but beyond that i do
not know Its not really fun for me to try to get libopenjpeg working
at least not when ive to deal with distros shiping ancient non
functional versions of it.
Improving our native decoder is at least to me much more fun to work


Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The greatest way to live with honor in this world is to be what we pretend
to be. -- Socrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150609/5efee231/attachment.asc>

More information about the ffmpeg-devel mailing list