Re: [FFmpeg-devel] [PATCH] avformat/flvdec: added support for KUX container
2019-03-19 18:11 GMT+01:00, swarajhota...@gmail.com : > From: Swaraj Hota > > Fixes ticket #4519. > --- > This is my qualification task for GSoC 2019. Patch applied. Thank you, Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
Re: [FFmpeg-devel] [PATCH] avformat/flvdec: added support for KUX container
2019-03-26 12:28 GMT+01:00, Moritz Barsnick : > On Tue, Mar 26, 2019 at 11:22:58 +0100, Carl Eugen Hoyos wrote: >> > While trying to see if the proposed XV demuxer[1] can be simplified, I >> > tripped over the fact that this KUX patch introduces a new demuxer for >> > what looks like "only" a minor variant of FLV. Could the patch not just >> > as well extend the flv probe and the extension list? >> >> What would be the advantage? > > Not exploding the list of supported formats by many minor subvariants. > (It seems KUX is 99% FLV, with a different header tag and an offset.) > I know it's a special example, but imagine "mov,mp4,m4a,3gp,3g2,mj2" > implemented as six separate formats, six separate probes, ... Not sure how easy it would be to separate them via probe. Did you receive my private mail? Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
Re: [FFmpeg-devel] [PATCH] avformat/flvdec: added support for KUX container
On Tue, Mar 26, 2019 at 11:22:58 +0100, Carl Eugen Hoyos wrote: > > While trying to see if the proposed XV demuxer[1] can be simplified, I > > tripped over the fact that this KUX patch introduces a new demuxer for > > what looks like "only" a minor variant of FLV. Could the patch not just > > as well extend the flv probe and the extension list? > > What would be the advantage? Not exploding the list of supported formats by many minor subvariants. (It seems KUX is 99% FLV, with a different header tag and an offset.) I know it's a special example, but imagine "mov,mp4,m4a,3gp,3g2,mj2" implemented as six separate formats, six separate probes, ... Just my $0.02. > Minor is what is usually done for a new demuxer. Okay, I will do so for XV as well. Thanks, Moritz ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
Re: [FFmpeg-devel] [PATCH] avformat/flvdec: added support for KUX container
On more remark: On Tue, Mar 19, 2019 at 22:41:32 +0530, swarajhota...@gmail.com wrote: > +static int kux_probe(AVProbeData *p) I get a warning if this isn't declared "const AVProbeData *p". Moritz ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
Re: [FFmpeg-devel] [PATCH] avformat/flvdec: added support for KUX container
2019-03-26 11:18 GMT+01:00, Moritz Barsnick : > On Tue, Mar 26, 2019 at 00:24:40 +0100, Carl Eugen Hoyos wrote: >> > Fixes ticket #4519. >> I will push this if there are no objections. > > While trying to see if the proposed XV demuxer[1] can be simplified, I > tripped over the fact that this KUX patch introduces a new demuxer for > what looks like "only" a minor variant of FLV. Could the patch not just > as well extend the flv probe and the extension list? What would be the advantage? [...] > And I personally would have only bumped micro, not minor, but that's > just a gut feeling, I haven't checked what is usually done. Minor is what is usually done for a new demuxer. Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
Re: [FFmpeg-devel] [PATCH] avformat/flvdec: added support for KUX container
On Tue, Mar 26, 2019 at 00:24:40 +0100, Carl Eugen Hoyos wrote: > > Fixes ticket #4519. > I will push this if there are no objections. While trying to see if the proposed XV demuxer[1] can be simplified, I tripped over the fact that this KUX patch introduces a new demuxer for what looks like "only" a minor variant of FLV. Could the patch not just as well extend the flv probe and the extension list? Of course, this would need to be implemented differently (perhaps a flag within the Class?): > if(!strcmp(s->iformat->name, "kux")) Or is the difference between "a variant" and "a demuxer of its own" too small to justify the former? Just wondering. And I personally would have only bumped micro, not minor, but that's just a gut feeling, I haven't checked what is usually done. Cheers, Moritz P.S.: I will subsequently propose a similar initial XV patch. Either as this KUX patch does, or as I suggested above. [1] http://ffmpeg.org/pipermail/ffmpeg-devel/2019-March/241483.html ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
Re: [FFmpeg-devel] [PATCH] avformat/flvdec: added support for KUX container
2019-03-19 18:11 GMT+01:00, swarajhota...@gmail.com : > From: Swaraj Hota > > Fixes ticket #4519. I will push this if there are no objections. Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
Re: [FFmpeg-devel] [PATCH] avformat/flvdec: added support for KUX container
Oh, I really wished to do it ':D Anyway, thanks :) Tell me if any other change can be made. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
Re: [FFmpeg-devel] [PATCH] avformat/flvdec: added support for KUX container
2019-03-22 19:29 GMT+01:00, Swaraj Hota : > I have the meta chunk loaded into a buffer. > I searched a lot but I couldn't find a way to decompress > a password protected zip buffer. > > I saw zlib is used in the code base, so went through it. > Found a function "decompress()" but it didn't take a > password as argument. Then this currently can't be done. I will look at your original patch again tomorrow and likely commit it. Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
Re: [FFmpeg-devel] [PATCH] avformat/flvdec: added support for KUX container
Okay I found "fmemopen()". I'll see if I can use it with unzOpenCurrentFilePassword(). Please do tell if there is a better solution though. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
Re: [FFmpeg-devel] [PATCH] avformat/flvdec: added support for KUX container
I have the meta chunk loaded into a buffer. I searched a lot but I couldn't find a way to decompress a password protected zip buffer. I saw zlib is used in the code base, so went through it. Found a function "decompress()" but it didn't take a password as argument. I found another function under minizip/unzip.h called unzOpenCurrentFilePassword() which does take password but it takes a file as an argument and not a buffer. I am not sure what to do. I thought of using mmap and pass it to unzOpenCurrentFilePassword() but I guess its not portable and might not work with windows. Or will it? Please suggest something I can do here. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
Re: [FFmpeg-devel] [PATCH] avformat/flvdec: added support for KUX container
2019-03-21 21:41 GMT+01:00, Swaraj Hota : > The title is in unicode so I guess we can just encode it to utf8. > I can write a simple C code to do that. > What should I change in the patch though regarding this? > Can you give me more details on what exactly to implement? There is metadata in FFmpeg, for example the tag "title" is known and shown by "ffmpeg -i". You can use av_dict_set() (as done elsewhere in flvdec.c) to set the tag "title" to whatever is in the unicode block we both see. Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
Re: [FFmpeg-devel] [PATCH] avformat/flvdec: added support for KUX container
The title is in unicode so I guess we can just encode it to utf8. I can write a simple C code to do that. What should I change in the patch though regarding this? Can you give me more details on what exactly to implement? ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
Re: [FFmpeg-devel] [PATCH] avformat/flvdec: added support for KUX container
2019-03-21 0:04 GMT+01:00, Swaraj Hota : > Okay, I found out the metadata block contains fields like: > logo (a url to thumbnail), tags (in Chinese), > title (in Chinese as well), Can you convert the title to something that your terminal (set to utf8) can display? There is "title" metadata, so this can be useful. Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH] avformat/flvdec: added support for KUX container
Okay, I found out the metadata block contains fields like: logo (a url to thumbnail), tags (in Chinese), title (in Chinese as well), and other information which mostly seem to be related to streaming. I'm not sure though if anything is much useful. Please do suggest if you think any of these can be used. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH] avformat/flvdec: added support for KUX container
2019-03-19 18:11 GMT+01:00, swarajhota...@gmail.com : > From: Swaraj Hota > > Fixes ticket #4519. > --- > This is my qualification task for GSoC 2019. > Please suggest any more changes if required. There is a zipped section in the files starting at 0xe4, the password is "meta". Is there anything in this metadata block that may be useful for a player? Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
