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

[burek]

[00:55] <cone-872> ffmpeg.git 03Carl Eugen Hoyos 07master:401f6a7126fe: Do not suggest to use gas-preprocessor if using it would break compilation.
[00:55] <cone-872> ffmpeg.git 03Tim.Nicholson 07master:ae4c912bcecb: Forward interlaced field information from mov to ffv1 decoder.
[00:55] <cone-872> ffmpeg.git 03Michael Niedermayer 07master:bdccfc3fc354: Merge remote-tracking branch 'cehoyos/master'
[02:41] <wm4> hm getting lots of valgrind errors with valgrind ffmpeg -i test.mkv -vf edgedetect,gradfun out.mkv
[02:54] <wm4> actually it just crashes
[03:14] <KGB> 01[13FFV101] 15michaelni pushed 2 new commits to 06master: 02http://git.io/bXCgjA
[03:14] <KGB> 13FFV1/06master 147210c5c 15Michael Niedermayer: ffv1: Document run length VLC coding
[03:14] <KGB> 13FFV1/06master 14dc84a70 15Michael Niedermayer: ffv1: update year
[03:22] <llogan> now we have two git commit bots?
[03:43] <michaelni> github has one source.ffmpeg.org ha another
[03:43] <michaelni> the ffv1 spec is just at github
[08:59] <cone-751> ffmpeg.git 03Vittorio Giovara 07master:0d8b943d204b: h264_sei: Return meaningful values
[08:59] <cone-751> ffmpeg.git 03Michael Niedermayer 07master:7cd13f618c3c: Merge commit '0d8b943d204bd16fcf2f4a59c742e65a401dd3d0'
[09:13] <cone-751> ffmpeg.git 03Gavriloaie Eugen-Andrei 07master:0d6fa3977b01: rtmp: Add seek support
[09:13] <cone-751> ffmpeg.git 03Michael Niedermayer 07master:8e970a58614f: Merge commit '0d6fa3977b016f1b72b0b24b8834ff9222498548'
[09:22] <cone-751> ffmpeg.git 03Diego Biurrun 07master:b5a138652ff8: Give less generic names to global library option arrays
[09:22] <cone-751> ffmpeg.git 03Michael Niedermayer 07master:a8e963835a43: Merge commit 'b5a138652ff8a5b987d3e1191e67fd9f6575527e'
[09:26] <cone-751> ffmpeg.git 03Diego Biurrun 07master:79be2c325c5e: doc/print_options: Move options headers to a saner place
[09:26] <cone-751> ffmpeg.git 03Michael Niedermayer 07master:fa5410f61a75: Merge commit '79be2c325c5ee8f7ac9e28399e51986ebe99bb3c'
[09:38] <cone-751> ffmpeg.git 03Diego Biurrun 07master:3a7050ffed5c: build: Add _Pragma macro to disable deprecated declaration warnings
[09:38] <cone-751> ffmpeg.git 03Michael Niedermayer 07master:85fc1a18ca7b: Merge commit '3a7050ffed5ce061b114a11e4de4b77aba8efa0b'
[10:35] <cone-751> ffmpeg.git 03Diego Biurrun 07master:7950e519bb09: Disable deprecation warnings for cases where a replacement is available
[10:35] <cone-751> ffmpeg.git 03Diego Biurrun 07master:038c4f65ee6c: configure: Check for GCC diagnostic pragma support inside of functions
[10:35] <cone-751> ffmpeg.git 03Michael Niedermayer 07master:20be5e0a0e75: Merge commit '7950e519bb094897f957b9a9531cc60ba46cbc91'
[10:35] <cone-751> ffmpeg.git 03Michael Niedermayer 07master:1607a9854552: avcodec/mlp: Fix bugs in libavs warning fixes
[10:53] <cone-751> ffmpeg.git 03Yusuke Nakamura 07master:a8b19271c3b4: avcodec: Add output_picture_number to AVCodecParserContext
[10:53] <cone-751> ffmpeg.git 03Michael Niedermayer 07master:82fdfe8e51c5: Merge commit 'a8b19271c3b40ac3c3dc769fe248887acf14ba5a'
[11:03] <cone-751> ffmpeg.git 03Diego Biurrun 07master:6da5b57da11b: configure: Check for GCC diagnostic pragma support inside of functions
[11:03] <cone-751> ffmpeg.git 03Michael Niedermayer 07master:62f616ed58ba: Merge remote-tracking branch 'qatar/master'
[12:59] <cone-751> ffmpeg.git 03Michael Niedermayer 07master:5ad4e2933748: MAINTAINERS: add myself as maintainer for the interface code to swresample & swscale in libavfilter
[12:59] <cone-751> ffmpeg.git 03Andrey Utkin 07master:b7ed18b9bd0f: mpegtsenc: add option tables_version
[15:28] <cone-751> ffmpeg.git 03Michael Niedermayer 07master:62738157dd54: pnmdec: always output native pixel format
[15:28] <cone-751> ffmpeg.git 03Carl Eugen Hoyos 07master:34d48dac252f: avcodec/pnmdec: support more pnm files
[16:24] <cone-751> ffmpeg.git 03Michael Niedermayer 07master:d6fd1242f318: avdevice/timefilter-test: provide more space for the printout to allow larger values
[16:24] <cone-751> ffmpeg.git 03Michael Niedermayer 07master:bc4e79856281: avdevice/timefilter: 2nd try at avoiding rounding issues
[16:31] <wm4> this command crashes for me after about 33 frames: ffmpeg -f rawvideo -video_size 1280x720 -i /dev/zero  -vf edgedetect,gradfun /tmp/out.mkv
[16:31] <wm4> *** Error in `../build_libs/bin/ffmpeg': corrupted double-linked list: 0x091d66f8 ***
[16:32] <wm4> could be a series issue? maybe gradfun's asm is writing past the frame memory allocation
[16:33] <wm4> *serious
[16:39] <pippin> \o/ my custom non-bayer ordered dither works =) http://pippin.gimp.org/dither/
[19:22] <michaelni> wm4, crash fixed
[19:23] <cone-751> ffmpeg.git 03Michael Niedermayer 07master:e43a0a232dbf: avfilter: fix plane validity checks
[19:24] <wm4> michaelni: uh I don't get it
[19:24] <wm4> michaelni: shouldn't it use the plane count the pixel format implies?
[19:24] <wm4> because the unused fields could contain random values
[19:24] <wm4> and in this case, they apparently did
[19:25] <ubitux> ah thx for that commit
[19:25] <wm4> you just made the bogus check a bit tighter
[19:25] <wm4> but still bogus
[19:25] <wm4> should use av_pix_fmt_count_planes() instead
[19:27] <wm4> anyway, must have been hard to find...
[19:47] <michaelni> data wasnt random data[1] was a 256 entry palette, the planes could be cheked differently, this just seemed the simplest
[20:34] <ubitux> michaelni: so in case of palette in data[1], we must make sure that linesize is *not* set
[20:34] <ubitux> shouldn't an assert be added somewhere?
[20:34] <michaelni> sounds like a good idea
[20:38] <wm4> huh, in my code I explicitly set linesize for palette...
[20:38] <wm4> hm or actually I don't
[20:39] <wm4> so must unused planes (plus the palette) have linesize set to 0?
[21:39] <cone-751> ffmpeg.git 03Andrey Utkin 07master:5b76c3a12049: doc/muxers: Document previously undocumented mpegts muxer options
[21:39] <michaelni> wm4, i think they are always null, do you know of anyting that could set them to non null ?
[21:40] <wm4> michaelni: for example when code doesn't explicitly initialize the fields (not all code necessarily uses the libavutil functions to fill a frame)
[21:49] <michaelni> with sizeof(AVFrame) not being part of the public API how can an application even allocate a AVFrame without uisng libavutil functions
[21:49] <michaelni> ?
[21:49] <michaelni> and if it doesnt use the functions how could it set the fields to defaults
[21:49] <wm4> it can allocate an avframe with libavutil, but set the plane pointers itself
[21:50] <nevcairiel> they are initialized to null, so it sets them to something invalid?
[21:50] <wm4> (and overwrite unused plane pointers with crap)
[21:50] <nevcairiel> sounds like a bug in the application
[21:50] <wm4> well, it's not really clear that ffmpeg would use the unused plane pointers
[21:51] <michaelni> we should probably check that they are null on public entry points if we keep the null checks
[21:53] <michaelni> and it should be documented
[21:54] <michaelni> or all these checks could be replaced if people prefer and someone volunteers to implement it
[00:00] --- Sun Aug  4 2013