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

[burek]

[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=3a3d24c693806900dbec1de0b3c94912f121ac1e;hb=HEAD#l56
[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