[Ffmpeg-devel-irc] ffmpeg-devel.log.20120811

[burek]

[02:05] <CIA-41> ffmpeg: 03Michael Niedermayer 07master * rbd6ece4609 10ffmpeg/libavformat/mp3enc.c: 
[02:05] <CIA-41> ffmpeg: mp3enc: fix CODEC_ID/CodecID that leaked in from older patch
[02:05] <CIA-41> ffmpeg: Found-by: jamal <jamrial at gmail.com>
[02:05] <CIA-41> ffmpeg: Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
[02:06] <CIA-41> ffmpeg: 03Michael Niedermayer 07master * rf9505923a3 10ffmpeg/libavcodec/cavsdec.c: 
[02:06] <CIA-41> ffmpeg: cavsdec: check dimensions being valid.
[02:06] <CIA-41> ffmpeg: Fixes crash
[02:06] <CIA-41> ffmpeg: Fixes Ticket1628
[02:06] <CIA-41> ffmpeg: Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
[02:25] <CIA-41> ffmpeg: 03Michael Niedermayer 07master * r58825a18aa 10ffmpeg/libavcodec/msrle.c: 
[02:25] <CIA-41> ffmpeg: msrle: fix regression causing null ptr dereference
[02:25] <CIA-41> ffmpeg: Fixes Ticket1630
[02:25] <CIA-41> ffmpeg: Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
[05:46] <CIA-41> ffmpeg: 03Michael Niedermayer 07master * r994923ec00 10ffmpeg/libavcodec/dwt.c: 
[05:46] <CIA-41> ffmpeg: dwt: switch to av_assert
[05:46] <CIA-41> ffmpeg: Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
[05:46] <CIA-41> ffmpeg: 03Michael Niedermayer 07master * r48d20b918c 10ffmpeg/libavcodec/snowdec.c: 
[05:46] <CIA-41> ffmpeg: snowdec: increase stack size
[05:46] <CIA-41> ffmpeg: Fixes Ticket1632
[05:46] <CIA-41> ffmpeg: Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
[11:04] <ubitux> saste: i think the env thing is in case SDL_getenv() would not be mapped on a getenv
[11:04] <ubitux> and would use some internal stuff in SDL or something
[11:05] <saste> i was asking why atoi() was not used in the first place
[11:05] <ubitux> and thus could include the type or something
[11:05] <ubitux> (so the sdl getenv would return a int directly or something)
[11:06] <ubitux> anyway that shouldn't happen
[11:06] <ubitux> :)
[11:46] <nyuhu> hum& I compared my hue port to the mplayer one using the showinfo filter ; the plane checksums are the same but not always for the same 'n' values (sequential number), is this an issue ?
[12:53] <saste> nyuhu: can you post an example
[15:11] <CIA-41> ffmpeg: 03Anton Khirnov 07master * rc223d79945 10ffmpeg/libavcodec/ (Makefile avcodec.h codec_desc.c): 
[15:11] <CIA-41> ffmpeg: lavc: add codec descriptors.
[15:11] <CIA-41> ffmpeg: They describe properties that are inherent to a codec (as described by
[15:11] <CIA-41> ffmpeg: an AVCodecID) without referring to a specific implementation.
[15:11] <CIA-41> ffmpeg: 03Anton Khirnov 07master * r51efed152d 10ffmpeg/libavcodec/ (avcodec.h codec_desc.c): lavc: add an intra-only codec property.
[15:11] <CIA-41> ffmpeg: 03Anton Khirnov 07master * r885da7b082 10ffmpeg/libavformat/utils.c: lavf: simplify is_intra_only() by using codec descriptors.
[15:11] <CIA-41> ffmpeg: 03Michael Niedermayer 07master * rf5f3684fb8 10ffmpeg/: (log message trimmed)
[15:11] <CIA-41> ffmpeg: Merge remote-tracking branch 'qatar/master'
[15:11] <CIA-41> ffmpeg: * qatar/master:
[15:11] <CIA-41> ffmpeg:  lavf: simplify is_intra_only() by using codec descriptors.
[15:11] <CIA-41> ffmpeg:  lavc: add an intra-only codec property.
[15:11] <CIA-41> ffmpeg:  lavc: add codec descriptors.
[15:11] <CIA-41> ffmpeg:  lavc: fix mixing CODEC_ID/AV_CODEC_ID in C++ code.
[15:11] <CIA-41> ffmpeg: 03Mans Rullgard 07master * r987170cb9d 10ffmpeg/ (6 files in 3 dirs): 
[15:11] <CIA-41> ffmpeg: dict: add av_dict_count()
[15:11] <CIA-41> ffmpeg: This adds a function to retrieve the number of entries in a
[15:11] <CIA-41> ffmpeg: dictionary and updates the places directly accessing what should
[15:11] <CIA-41> ffmpeg: be an opaque struct to use this new function instead.
[15:12] <CIA-41> ffmpeg: 03Mans Rullgard 07master * r33de86db2b 10ffmpeg/libavutil/ (dict.c internal.h): 
[15:12] <CIA-41> ffmpeg: dict: move struct AVDictionary definition to dict.c
[15:12] <CIA-41> ffmpeg: This makes struct AVDictionary fully opaque now that nothing
[15:12] <CIA-41> ffmpeg: needs to access it directly any more.
[15:12] <CIA-41> ffmpeg: Signed-off-by: Mans Rullgard <mans at mansr.com>
[15:14] <iive> so, now in addition to codec flags we'll have codec descriptors....
[15:15] <nevcairiel> the flags have the problem that they need a decoder/encoder to be set
[15:15] <nevcairiel> and may even be implementation dependent
[15:38] <michaelni> nevcairiel, the descriptors just have the problem of being 200 times slower, and still are implementation dependant as they still are based on implementation ids
[15:39] <michaelni> ill fix the speed issue
[15:39] <michaelni> but iive is right its silly design
[15:46] <michaelni> its not silly enough to break ABI OTOH just some change that wasnt really neccessary
[15:48] <michaelni> nevcairiel, additioally the descriptors duplicate alot of info now and it might become inconsistent quite easily ...
[15:50] <michaelni> and nevcairiel to be more specific, if one wanted just the implementations disabled and the AVCodec available that is quite possible, just have the function pointers in it be NULL instead of pointing to the codec implementation, but there are other ways too
[16:23] <michaelni> nevcairiel, ive posted a patch that fixes the speed problem, your comments/review is welcome. Antons comments are welcome too if he isnt afraid ...
[17:05] <CIA-41> ffmpeg: 03Michael Niedermayer 07master * r5a49482cc0 10ffmpeg/libavcodec/dsputil_template.c: 
[17:05] <CIA-41> ffmpeg: dsputil_template: switch to av_assert
[17:05] <CIA-41> ffmpeg: Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
[17:21] <Compn> "This very evil hack should fix this problem."
[17:21] <Compn> ...
[17:22] <Compn> did this just completely break mplayer compilation?
[19:01] <iive> just took a look of the source. I don't get it. the descriptors seems completely redundant.
[19:03] <iive> the only benefit I could possible see is the ability to list more descriptions than codecs. e.g. having xvid, divx, and mpeg4asp descriptiors all sharing single codec.
[19:16] <nevcairiel> right now they add an advantage for codecs that simply have no implementation. Not saying its the best way to do it, but they add some additional information there
[19:54] <CIA-41> ffmpeg: 03Michael Niedermayer 07master * r7d23a65f0d 10ffmpeg/libavformat/asfdec.c: 
[19:54] <CIA-41> ffmpeg: asfdec: fix FRAME_HEADER_SIZE
[19:54] <CIA-41> ffmpeg: Fixes Ticket1477
[19:54] <CIA-41> ffmpeg: Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
[20:07] <ubitux> this makes me wonder if we shouldn't open a ffmpeg-libav-dmz mailing list to have some kind of communication medium between the project, were anyone willing to collaborate would be welcome to discuss such issues
[20:07] <ubitux> this would be typically used for RFC
[20:07] <ubitux> (iirc, Anton sent this as a RFC at first for example)
[20:18] <iive> to libav maillist?
[20:24] <ubitux> yes
[20:25] <ubitux> if we had such shared ml, maybe michael or iive would have comment on this
[20:25] <ubitux> basically it could be a ml for collaboration when working on api improvement or such
[20:26] <ubitux> or a flamewar ml, still isolated from the projects
[20:26] <ubitux> anyway, just an idea :)
[20:27] <nevcairiel> judging by the comments from certain individuals on either side, it wouldnt end well
[20:36] <ubitux> :(
[20:38] <iive> ubitux: can you give me a link to the rfc?
[00:00] --- Sun Aug 12 2012