Re: [FFmpeg-devel] libavcodec/hapdec : add support for hapqa decoding

[Martin Vignali]

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

[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
___
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

[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
>>
>> 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

[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))
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

[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.

Carl Eugen
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel