[00:00] <J_Darnley> llogan: Sorry, I misread [00:01] <J_Darnley> I assume qscale will be ignored and qp will be used [00:05] <J_Darnley> Ah then that's a bug I jusst made [00:05] <J_Darnley> It does use qp but tell the use user its using crf [00:05] Action: J_Darnley grumbles [00:18] <J_Darnley> llogan: And what about if the user uses both -qscale and -x264opts? Do I need to warn in that case? [00:31] <llogan> J_Darnley: i forgot about that, but if they're using x264opts then i'd consider them to be on their own (or hopefully they know what they're doing). [00:32] <llogan> or x264-params [00:34] <J_Darnley> Is that a 3rd way to set options? [00:35] <J_Darnley> :( [00:37] <michaelni> maybe just say that qscale is ignored with libx264 [00:37] <michaelni> without any other checks or details [00:41] <J_Darnley> I can move my patch further down the init function to catch all options. [00:41] <J_Darnley> Just a simple statement that qscale is ignored may be better. [00:42] <J_Darnley> For that case, where should I print the message? From the top of the init function like my first patch? [00:43] <michaelni> iam happy with any location that works [00:44] <J_Darnley> llogan: Any more comments? [00:46] <J_Darnley> Should I ask the original poster for comments on trac? [00:48] <J_Darnley> How about "-qscale is ignored with libx264, -crf is preferred"? [00:55] <J_Darnley> Do I need to mention libx264 in the message because ffmpeg will print "[libx264 @ address] in the log? [01:05] <michaelni> i guess you can skip the libx264 in the message [01:08] <llogan> J_Darnley: up to you how complex you want it to be. im also fine with a simple "-qscale is ignored with this encoder" or whatever [01:13] <J_Darnley> In that case I will say that a simple message is least likely to become incorrect, wrong, misleading, or broken in the future so I will submit a patch to do just that. [01:19] <J_Darnley> Oh dammit [01:20] <J_Darnley> look at what you're actually commiting James! [01:21] <J_Darnley> I think it is time for bed when I send yet another email [01:39] <cone-406> ffmpeg.git 03Michael Niedermayer 07master:d003a0cd2e58: avformat/mp3enc: use av_copy_packet() [03:30] <cone-406> ffmpeg.git 03Michael Niedermayer 07master:87ecefdab009: elbg: change utility_inc to 64bit [03:54] <cone-406> ffmpeg.git 03rogerdpack 07master:6abc56e892c2: dshow: use distinct buffers per stream. [07:22] <Zeranoe> Looks like some updates on the -ac 2 channel downmix thread: http://ffmpeg.zeranoe.com/forum/viewtopic.php?f=17&t=1851 [13:18] <cone-46> ffmpeg.git 03James Darnley 07master:f02832904dbc: lavc/libx264: add a warning when using qscale [13:26] <BBB> re bug 3486, it's bicublin, not bicubiclin [13:27] <BBB> (I don't have a bug tracker account so can't do it myself) [15:42] <pawelrutka> are there any people to ask about using ffmpeg C Api ? [16:17] <cone-46> ffmpeg.git 03Matt Oliver 07master:cd5cf395f65a: Additional icl inline asm fix. [16:42] Action: michaelni wonders why people ask a question and then leave the channel 3 minutes later [16:43] <Compn> michaelni : we could setup an irc script/bot if someone joins and asks a question within 1 minute of joining it will say 'slow down! we're here but we check irc every 30 minutes, please leave the channel connected and we will respond shortly' [16:44] <michaelni> why am i thinking they will just leave after 30sec then, hmm [16:45] <cone-46> ffmpeg.git 03rogerdpack 07master:0b4c5c99bc6f: avformat/udp: cleanup comments, add a warning if receive buffer is unexpectedly unable to be set high enough [17:04] <cone-46> ffmpeg.git 03rogerdpack 07master:3eafe1d71c09: udp: try to clarify verbiage on buffer_size [17:04] <cone-46> ffmpeg.git 03Michael Niedermayer 07master:ae4b11df0d9e: avcodec/snowenc: make snow non experimental [17:32] <Compn> http://askubuntu.com/questions/432893/libav-deinsterlace-option [17:32] <Compn> does that apply to ffmpeg ? cant test atm [17:37] <wm4> Compn: I would think so [17:37] <wm4> yadif is still the best deinterlacer in ffmpeg, so it should be used [17:37] <Compn> sure [17:37] <Compn> but i think we could make -deinterlace an alias for yadif [17:37] <Compn> so it wont break scripts [17:38] <Compn> michaelni : ^^ ? [17:38] <wm4> because apparently we don't care about backwards compatibility around here [17:38] <Compn> i care [17:39] <wm4> btw. does ffplay not use libavfilter for its audio display? [17:39] <wm4> seems like there's custom crap in ffplay.c [17:40] <Compn> ffplay's audio visualizer was made long before libavfilter [17:40] <Compn> dont think anyone ported it to libavfilter [17:40] <Compn> mr reading ffplay.c but not looking at git blame to see how old it is [17:40] <Compn> well bunch of useless hashes arent helpful anyhow [17:40] Action: Compn misses svn blame [17:41] <wm4> yeah that code is from 2003 [17:42] <wm4> written by fabrice himself [17:42] <Compn> lol [17:46] <michaelni> wm4, libav is not ffmpeg, we do care about compatibility [17:46] <michaelni> try ffmpeg -deinterace [17:46] <michaelni> try ffmpeg -deinterlace that is [17:47] <michaelni> it should insert yadif [17:48] <Compn> :) [17:48] Action: Compn afk bbl [17:51] <cone-46> ffmpeg.git 03Peter Ross 07master:54df8b4feaf4: fate: add cine demux test [17:59] <Actium> that's mine in case anyone would like to yell at me :) [18:19] <Case> latest sources give me some problems in mingw+msys with following configure command: [18:19] <Case> ./configure --disable-programs --disable-everything --disable-static --enable-shared --disable-swresample --disable-swscale --disable-network --disable-avdevice --disable-postproc --disable-avfilter --disable-debug --enable-runtime-cpudetect --enable-decoder=mp3float --enable-decoder=vorbis --enable-decoder=aac [18:20] <Case> make stops with error libavcodec/rawdec.o:rawdec.c:(.text.unlikely+0x34): undefined reference to `ff_dsputil_init' [18:20] <Case> it's my understanding this entire raw video decoder shouldn't be building with the above config command [18:30] <cone-46> ffmpeg.git 03Diego Biurrun 07master:7513234bdd93: avfilter: more correct printf format specifiers [18:30] <cone-46> ffmpeg.git 03Michael Niedermayer 07master:7ef8d97b1aa9: Merge commit '7513234bdd93f80593bfee3eb2da8c8f0d3992da' [18:51] <cone-46> ffmpeg.git 03Diego Biurrun 07master:cc8163e1a360: avcodec: more correct printf specifiers [18:51] <cone-46> ffmpeg.git 03Michael Niedermayer 07master:8d024c51071d: Merge commit 'cc8163e1a3601a56f722a4720516e860bf1c6198' [19:35] <cone-46> ffmpeg.git 03Diego Biurrun 07master:46caba4a6523: float_dsp: Replace arch optimization ifdefs by if cascade [19:35] <cone-46> ffmpeg.git 03Michael Niedermayer 07master:cd1d4b51e3ed: Merge remote-tracking branch 'qatar/master' [19:38] <michaelni> Case, cant reproduce a build issue with that command [19:38] <michaelni> that is on linux havnt tries mingw [19:40] <michaelni> Case, try a make distclean and if that doesnt help look at config.log [21:22] <cone-46> ffmpeg.git 03Nicolas George 07master:7b9a310d0e27: lavu: add AV_DYNARRAY_ADD() macro. [21:22] <cone-46> ffmpeg.git 03Nicolas George 07master:b0dcf7653064: lavu/mem: reimplement the dynarray functions with the macro. [21:22] <cone-46> ffmpeg.git 03Michael Niedermayer 07master:9ba119029796: avutil/mem: avoid using intptr_t to access void* in av_dynarray_add() [21:27] <wm4> this line is so wrong: https://git.videolan.org/?p=ffmpeg.git;a=blob;f=libavutil/dynarray.h;h=3a3d2... [21:27] <wm4> a log call just for resizing an array? [21:27] <wm4> probably even accidentally in here [21:53] <cone-46> ffmpeg.git 03Michael Niedermayer 07master:82c742377a16: avutil/dynarray: remove debug av_log() from AV_DYNARRAY_ADD [21:53] <michaelni> wm4, removed [22:00] <wm4> thanks [22:11] <cone-46> ffmpeg.git 03Diego Biurrun 07master:1df0b061621b: nuv: Reuse the DSPContext from RTJpegContext [22:11] <cone-46> ffmpeg.git 03Michael Niedermayer 07master:32da2fd42265: Merge commit '1df0b061621b10edde87e3ab7ea83aed381c574f' [22:16] <cone-46> ffmpeg.git 03Diego Biurrun 07master:1a8d0cf77ed2: x86: dsputil: Move inline assembly macros to a separate header [22:16] <cone-46> ffmpeg.git 03Michael Niedermayer 07master:8fbc6e59118d: Merge commit '1a8d0cf77ed2611e542ae98f341d4c43a04467bd' [22:23] <cone-46> ffmpeg.git 03Diego Biurrun 07master:600b854ad817: imgconvert: Move ff_deinterlace_line_*_mmx declarations out of dsputil [22:23] <cone-46> ffmpeg.git 03Michael Niedermayer 07master:d6d3cfb0aa23: Merge commit '600b854ad8173995518bd917e7f86120b5505088' [22:29] <cone-46> ffmpeg.git 03Diego Biurrun 07master:f9279ee74936: dsputil: Move ff_zigzag_direct and ff_crop_tab declarations to mathops.h [22:29] <cone-46> ffmpeg.git 03Michael Niedermayer 07master:672c610d8439: Merge commit 'f9279ee74936b4990fc46287c27816ef828cc06c' [22:40] <Case> michaelni: I tried it in a linux VM too where it worked. make distclean doesn't help. I don't know what to look for in the huge config.log file, but I uploaded it to http://www.saunalahti.fi/~cse/temp/config.log [22:40] <Case> I can make it compile by copying the rawdec.c file from FFmpeg 2.1 sources [23:03] <cone-46> ffmpeg.git 03Diego Biurrun 07master:322a1dda973e: dsputil: Refactor duplicated CALL_2X_PIXELS / PIXELS16 macros [23:03] <cone-46> ffmpeg.git 03Michael Niedermayer 07master:77bc34297596: Merge commit '322a1dda973e802db7b57f2007fad3efcd5bab81' [23:17] <cone-46> ffmpeg.git 03Diego Biurrun 07master:6655c933a887: x86: dsputil: Move fpel declarations to a separate header [23:17] <cone-46> ffmpeg.git 03Michael Niedermayer 07master:9333bba6ed32: Merge commit '6655c933a887a2d20707fff657b614aa1d86a25b' [23:21] <cone-46> ffmpeg.git 03Diego Biurrun 07master:ae116cd3ed90: ppc: dsputil: Move hpeldsp-related declarations to a separate header [23:21] <cone-46> ffmpeg.git 03Michael Niedermayer 07master:816d7936a714: Merge commit 'ae116cd3ed908d28b69d5198712217ec743d74f6' [23:26] <cone-46> ffmpeg.git 03Diego Biurrun 07master:82dd1026cfc1: x86: dsputil: Move hpeldsp-related declarations to a separate header [23:26] <cone-46> ffmpeg.git 03Michael Niedermayer 07master:74fed968d1b6: Merge commit '82dd1026cfc1d72b04019185bea4c1c9621ace3f' [23:44] <cone-46> ffmpeg.git 03Diego Biurrun 07master:57f09608e160: dsputil: Move thirdpel-related bits into their own context [23:44] <cone-46> ffmpeg.git 03Michael Niedermayer 07master:9d6a27d53365: Merge commit '57f09608e1600d1cf1679885a46f5004d522d68f' [23:50] <cone-46> ffmpeg.git 03Diego Biurrun 07master:4f4840377f80: lagarith: Fix typo in printf format string [23:50] <cone-46> ffmpeg.git 03Michael Niedermayer 07master:de182701ac9e: Merge remote-tracking branch 'qatar/master' [00:00] --- Sun Mar 23 2014