Re: [FFmpeg-devel] libavcodec/hapdec : add support for hapqa decoding
2017-11-13 20:06 GMT+01:00 Martin Vignali : > > > 2017-10-21 21:37 GMT+02:00 Carl Eugen Hoyos : > >> 2017-10-21 21:32 GMT+02:00 Martin Vignali : >> > 2017-10-21 21:23 GMT+02:00 Carl Eugen Hoyos : >> > >> >> 2017-10-21 19:35 GMT+02:00 Martin Vignali : >> >> > Hello, >> >> > >> >> > I split the previous patch for HAPQAlpha decoding >> >> > in several part, to be easier to read >> >> > >> >> > 0002-libavcodec-hapdec-reorganize-code-before-adding-mult : >> >> > prepare for multi texture support but only decode the first texture >> >> > >> >> > 0003-libavcodec-hapdec-indent-after-previous-commit : >> >> > Cosmetic : indent >> >> > >> >> > 0004-libavcodec-hapdec-add-support-for-hapqa-decoding : >> >> > Decode HAQA, and support 2 texture decoding >> >> > > > Ping > Ping ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] libavcodec/hapdec : add support for hapqa decoding
2017-10-21 21:37 GMT+02:00 Carl Eugen Hoyos : > 2017-10-21 21:32 GMT+02:00 Martin Vignali : > > 2017-10-21 21:23 GMT+02:00 Carl Eugen Hoyos : > > > >> 2017-10-21 19:35 GMT+02:00 Martin Vignali : > >> > Hello, > >> > > >> > I split the previous patch for HAPQAlpha decoding > >> > in several part, to be easier to read > >> > > >> > 0002-libavcodec-hapdec-reorganize-code-before-adding-mult : > >> > prepare for multi texture support but only decode the first texture > >> > > >> > 0003-libavcodec-hapdec-indent-after-previous-commit : > >> > Cosmetic : indent > >> > > >> > 0004-libavcodec-hapdec-add-support-for-hapqa-decoding : > >> > Decode HAQA, and support 2 texture decoding > >> Ping ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] libavcodec/hapdec : add support for hapqa decoding
2017-10-21 21:32 GMT+02:00 Martin Vignali : > 2017-10-21 21:23 GMT+02:00 Carl Eugen Hoyos : > >> 2017-10-21 19:35 GMT+02:00 Martin Vignali : >> > Hello, >> > >> > I split the previous patch for HAPQAlpha decoding >> > in several part, to be easier to read >> > >> > 0002-libavcodec-hapdec-reorganize-code-before-adding-mult : >> > prepare for multi texture support but only decode the first texture >> > >> > 0003-libavcodec-hapdec-indent-after-previous-commit : >> > Cosmetic : indent >> > >> > 0004-libavcodec-hapdec-add-support-for-hapqa-decoding : >> > Decode HAQA, and support 2 texture decoding >> >> Is this the same patch as the one you explained to me >> would show both unexpected behaviour for users and >> make usage of the format needlessly difficult for >> users? >> >> If yes, I don't think it is a good idea to resend it. >> >> >> > Are you sure, you don't talk about the discussion for HapAlphaOnly decoding > /encoding ? > if yes, the patch here are for another codec (HAQAlpha, who is RGBA with 2 > textures (one for RGB and one for Alpha)) Sorry! Thank you for the clarification, Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] libavcodec/hapdec : add support for hapqa decoding
2017-10-21 21:23 GMT+02:00 Carl Eugen Hoyos : > 2017-10-21 19:35 GMT+02:00 Martin Vignali : > > Hello, > > > > I split the previous patch for HAPQAlpha decoding > > in several part, to be easier to read > > > > 0002-libavcodec-hapdec-reorganize-code-before-adding-mult : > > prepare for multi texture support but only decode the first texture > > > > 0003-libavcodec-hapdec-indent-after-previous-commit : > > Cosmetic : indent > > > > 0004-libavcodec-hapdec-add-support-for-hapqa-decoding : > > Decode HAQA, and support 2 texture decoding > > Is this the same patch as the one you explained to me > would show both unexpected behaviour for users and > make usage of the format needlessly difficult for > users? > > If yes, I don't think it is a good idea to resend it. > > > Are you sure, you don't talk about the discussion for HapAlphaOnly decoding /encoding ? if yes, the patch here are for another codec (HAQAlpha, who is RGBA with 2 textures (one for RGB and one for Alpha)) I think for this one, everyone will be agree, that we need to use RGBA :-) Martin ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] libavcodec/hapdec : add support for hapqa decoding
2017-10-21 19:35 GMT+02:00 Martin Vignali : > Hello, > > I split the previous patch for HAPQAlpha decoding > in several part, to be easier to read > > 0002-libavcodec-hapdec-reorganize-code-before-adding-mult : > prepare for multi texture support but only decode the first texture > > 0003-libavcodec-hapdec-indent-after-previous-commit : > Cosmetic : indent > > 0004-libavcodec-hapdec-add-support-for-hapqa-decoding : > Decode HAQA, and support 2 texture decoding Is this the same patch as the one you explained to me would show both unexpected behaviour for users and make usage of the format needlessly difficult for users? If yes, I don't think it is a good idea to resend it. Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel