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

[burek]

[00:34] <llogan> dur..how do i add a comment to my message body that isn't part of the patch? with git send-email
[00:34] <mathstuf> llogan: git format-patch --cover-letter ?
[00:37] <cone-911> ffmpeg.git 03Mickaël Raulet 07master:3c3ece24ea1e: hevc : cosmetic changes(cherry picked from commit 7308c0ccf13f18cebe4851e6dcd6b5c0b09be1dd)
[00:37] <cone-911> ffmpeg.git 03Mickaël Raulet 07master:3106cbd32118: hevc: more cosmetic(cherry picked from commit 9697abe41daa234602915f85bf6b1c0ca0252cff)
[00:37] <cone-911> ffmpeg.git 03Mickaël Raulet 07master:2707cca78f70: hevc: cosmetic change(cherry picked from commit 3b57513b3f39c04337801fb9d159c7ca8dfa9deb)
[00:37] <cone-911> ffmpeg.git 03Mickaël Raulet 07master:c1882e801d64: hevc: clean up mvs(cherry picked from commit 955317c09b877a513d3fcfcd1615909b2f4f651c)
[00:37] <cone-911> ffmpeg.git 03Anton Khirnov 07master:2f77894cccec: hevc: better mt implementation
[00:37] <cone-911> ffmpeg.git 03Mickaël Raulet 07master:0ddd3c5ba634: hevc: add decode hrd (cherry picked from commit ab4061dff796b1dd2bc884101226aab48c2c875e)
[00:38] <llogan> mathstuf: ah, thanks. I was reading "send-email" in the docs when it was actually "format-patch"...tired.
[00:38] <mathstuf> and send-email doesnt complain when it gets --cover-letter either :/
[00:39] <mathstuf> i should send a patch to git to dump out a message when that happens
[00:40] <llogan> i have some sort of disability when it comes to git
[00:41] <mathstuf> there are simple parts and...not so simple parts (hand-editing {add,checkout,reset} -p hunks, splitting a patch in rebase -i, and more, but those aren't really hit upon day-to-day)
[00:43] <wm4> mathstuf: btw. do you want to try to implement metadata update for mp3 too?
[00:43] <mathstuf> i can take a look
[00:44] <wm4> it would help with some web radios
[00:44] <wm4> oh btw., didn't you ask where ICY info comes from
[00:44] <wm4> it's from http.c
[00:45] <mathstuf> i guess the popular formats for internet radio are mp3, aac, and maybe some apple thing for that end of the internet?
[00:45] <wm4> yeah
[00:45] <wm4> not sure what aac radio uses
[00:45] <mathstuf> i get ICY from somafm
[00:46] <mathstuf> havent done a dumpstream on it to see if theres anything in that part of the stream
[00:46] Action: mathstuf tests
[00:51] <mathstuf> nope, nothing from soma at least
[01:02] <cone-911> ffmpeg.git 03Michael Niedermayer 07master:72d1f668c40c: Changelog: add 2.1
[01:02] <cone-911> ffmpeg.git 03Michael Niedermayer 07master:f11a917ac012: MAINTAINERS: update which releases i maintain
[01:02] <cone-911> ffmpeg.git 03Michael Niedermayer 07master:b85bf3423c47: doc/APIchanges: add 2 missing hashes & versions
[01:02] <cone-911> ffmpeg.git 03Michael Niedermayer 07master:d041f1251347: avdevice/pulse_audio_enc: remove double ;
[01:15] <cone-911> ffmpeg.git 03Michael Niedermayer 07master:0fef19b15ad9: doc/RELEASE_NOTES: update for 2.1
[01:27] <mathstuf> wm4: well, i can test flac, mp3, and opus in mpd as well
[01:40] <cone-911> ffmpeg.git 03Michael Niedermayer 07fatal: ambiguous argument 'refs/heads/release/2.1': unknown revision or path not in the working tree.
[01:40] <cone-911> Use '--' to separate paths from revisions
[01:40] <cone-911> refs/heads/release/2.1:HEAD: doc/RELEASE_NOTES: update for 2.1
[01:42] <mathstuf> hmm, so flac streaming has ICY info in it
[01:42] <mathstuf> ogg doesnt
[01:43] <mathstuf> mp3 and opus also have ICY
[01:47] <mathstuf> ok...wtf?
[01:47] <mathstuf> cant get any sound from mplayer or mpv
[02:02] <mathstuf> whatever...looks like mpd only puts tags into ogg/vorbis streams anyways
[02:02] <mathstuf> (yay curl >.> )
[02:14] <cone-911> ffmpeg.git 03Michael Niedermayer 07fatal: ambiguous argument 'refs/tags/n2.1': unknown revision or path not in the working tree.
[02:14] <cone-911> Use '--' to separate paths from revisions
[02:14] <cone-911> refs/tags/n2.1:HEAD: doc/RELEASE_NOTES: update for 2.1
[09:15] <durandal_1707> why examples still use AUDIO_INBUF_SIZE ?
[09:16] <ubitux> examples are broken
[09:17] <durandal_1707> thus useless
[09:44] <cone-911> ffmpeg.git 03Stefano Sabatini 07master:6baf9c4406bc: cmdutils: fix expected signature for show_colors() function
[09:49] <durandal_1707> s/useless/harmful
[09:57] <saste> ubitux: string variable interpolation, why did you work on that?
[09:57] <saste> also, any progress on that?
[09:57] <saste> why was it rejected?
[10:04] <durandal_1707> as far i see it wasn't, just incomplete
[10:06] <ubitux> saste: i worked on it for segment format mainly
[10:06] <ubitux> to be able to set a custom output format name
[10:06] <durandal_1707> ffmpeg.org down ?!?!?
[10:07] <saste> durandal_1707, so it seems
[10:07] <ubitux> durandal_1707: http://www.downforeveryoneorjustme.com/ffmpeg.org
[10:07] <saste> now it's up (I had to wait a few seconds)
[10:08] <saste> ubitux, that's my use case as well
[10:08] <saste> ubitux, then what happened?
[10:08] <ubitux> dunno, i didn't care anymore i guess
[10:08] <cone-911> ffmpeg.git 03Justin Ruggles 07master:211ca69b13eb: lavr: check that current_buffer is not NULL before using it
[10:08] <cone-911> ffmpeg.git 03Michael Niedermayer 07master:b96dddd34403: Merge commit '211ca69b13eb0a127a9ef7e70ddaccdab125d1c5'
[10:08] <saste> do you have a link to the mailing-list, what should i look for
[10:08] <saste> ?
[10:08] <ubitux> mmh
[10:08] <ubitux> just a min
[10:10] <ubitux> http://ffmpeg.org/pipermail/ffmpeg-devel/2012-February/120446.html
[10:11] <ubitux> next or prev msg probably
[10:11] <saste> ubitux, merci
[10:11] <ubitux> :)
[10:12] <ubitux> saste: see printf("%s\n", av_stringinterp_interpolate(si, "%(filename)_%(ts1)-%(ts2)_%(nokey)%(ext)"));
[10:12] <ubitux> in the example/main()
[10:12] <saste> ok
[10:13] <mraulet> michaelni: I added profile indication warning when there is no profile indication in the bitstream
[10:18] <cone-911> ffmpeg.git 03Vittorio Giovara 07master:fc06ee6ee377: mmvideo: fix uninitialized variable use in mm_decode_intra
[10:18] <cone-911> ffmpeg.git 03Michael Niedermayer 07master:893063781926: Merge commit 'fc06ee6ee377cc3b512dff8f02057e26311bc4da'
[10:21] <durandal_1707> saste: did you found equivalent of splitplanes/components?
[10:22] <saste> durandal_1707, no, there is any?
[10:22] <durandal_1707> looks like lavfi have to many filters for you to remmember all of them...
[10:23] <durandal_1707> *too
[10:23] <saste> i suppose i could find some hack to use geq or lut, even if not very efficient
[10:24] <durandal_1707> why you can't leart how to use grep?
[10:24] <durandal_1707> *learn
[10:25] <durandal_1707> saste: you need it for something specifically or?
[10:25] <saste> extractplanes?
[10:27] <durandal_1707> you find it useful?
[10:28] <saste> durandal_1707, could be, i never personally used most of the stuff i wrote
[10:28] <saste> was just curious
[10:32] <durandal_1707> if you never use it, why you wrote it then?
[10:34] <durandal_1707> i wrote extractplanes for sole reason of that undocumented hack in img2 (de)muxer
[10:37] <cone-911> ffmpeg.git 03Vittorio Giovara 07master:529a9893d769: avframe: mark source frame const in _ref and _clone
[10:37] <cone-911> ffmpeg.git 03Michael Niedermayer 07master:078dab551d53: Merge commit '529a9893d769f381b72785c500662be2020da5fe'
[10:45] <cone-911> ffmpeg.git 03Anton Khirnov 07master:4db81f081743: hevc: add irap checks (cherry picked from commit 3d3bbe35541a308937d0fe72b20a1c29d1c4100d)
[10:45] <cone-911> ffmpeg.git 03Anton Khirnov 07master:cb148e56dc15: hevc: refactor pic_arrays and set_sps (cherry picked from commit a6686c6d83b50c0962269f2c487f4f0c57e0df79)
[10:46] <cone-911> ffmpeg.git 03Mickaël Raulet 07master:a21839149cdd: hevc: add profile idc warning (cherry picked from commit 15f7a481fd19529b13631bfff5b3d65bfe5729d5)
[10:53] <cone-911> ffmpeg.git 03Anton Khirnov 07master:ddc589ce98c2: avconv: drop a now useless variable
[10:53] <cone-911> ffmpeg.git 03Michael Niedermayer 07master:ed49e91fd704: Merge commit 'ddc589ce98c2bba1e59318b5b0224717325eac46'
[11:01] <cone-911> ffmpeg.git 03Paul B Mahol 07master:79ef4b19bfca: FATE: add bitexact sws flags to the fieldorder test
[11:01] <cone-911> ffmpeg.git 03Michael Niedermayer 07master:bdd3a746775f: Merge commit '79ef4b19bfcab8b984682a53bb8561e5c8324731'
[11:21] <cone-911> ffmpeg.git 03Anton Khirnov 07master:94603feb1b3a: h264_ps: when parsing a VUI fails, only abort when explode is set
[11:21] <cone-911> ffmpeg.git 03Michael Niedermayer 07master:ac3fa95e73e5: Merge commit '94603feb1b3ad01a821a1a1cef1570b13f471821'
[11:32] <cone-911> ffmpeg.git 03Anton Khirnov 07master:0b357a8095e7: AVOptions: do not range check flag options.
[11:32] <cone-911> ffmpeg.git 03Michael Niedermayer 07master:f0e43e60cd09: Merge commit '0b357a8095e72b092cc5c2aacc2f806db75ecae3'
[11:45] <cone-911> ffmpeg.git 03Luca Barbato 07master:de6061203e2d: configure: Disable -Wmaybe-uninitialized by default
[11:45] <cone-911> ffmpeg.git 03Michael Niedermayer 07master:c985966a1059: Merge commit 'de6061203e2d509579ab110fb1873aade34320f5'
[11:55] <durandal_1707> nice, more and more hardly tested stuff get comitted
[11:58] <ubitux> what are you refering to?
[11:59] <durandal_1707> anything from Merge....
[11:59] <ubitux> i don't like much that warning drop
[12:03] <cone-911> ffmpeg.git 03Luca Barbato 07master:074931488639: h263: Return meaningful errors
[12:03] <cone-911> ffmpeg.git 03Michael Niedermayer 07master:d57a6fe6abda: Merge commit '0749314886390f6ec81d45e0ba424fcb36c945cf'
[12:03] <michaelni> ubitux, feel free to revert
[12:03] <durandal_1707> there is no no-maybe-uninitialized on clang only no-uninitialized
[12:04] <ubitux> rhaa and again luca trying to remove assert
[12:04] <ubitux> ...and he get reviewed by cosmetic guy #3 for h264 code :(
[12:05] <durandal_1707> i assume cosmetic guy #0 is Don
[12:06] <ubitux> i started counting at #1, but yeah
[12:07] <durandal_1707> but lavfi counts from #0 so should you too
[12:08] <ubitux> :)
[12:15] <cone-911> ffmpeg.git 03Michael Niedermayer 07master:aaaf2dc023d3: h263: Check init_get_bits return value
[12:15] <cone-911> ffmpeg.git 03Michael Niedermayer 07master:a66ee3dc87c5: Merge commit 'aaaf2dc023d31f30eeec874f24b50f44b9295185'
[12:25] <cone-911> ffmpeg.git 03Luca Barbato 07master:53151723e377: avio: K&R formatting cosmetics
[12:25] <cone-911> ffmpeg.git 03Michael Niedermayer 07master:0436ffc294ba: Merge commit '53151723e377b9c43f876e20d7f27a17993256c8'
[12:27] <cone-911> ffmpeg.git 03Anton Khirnov 07master:f354f30836a3: error resilience: check error_concealment, not err_recognition.
[12:27] <cone-911> ffmpeg.git 03Michael Niedermayer 07master:528f5cdd2cf1: Merge commit 'f354f30836a3148275ce60d19bbc581310249ad2'
[12:29] <saste> will cosmetics ever end?
[12:31] <ubitux> they're 3 full time on it now
[12:32] <saste> what when they will end?
[12:35] <ubitux> maybe they'll start merging commits from ffmpeg to reindent them
[12:35] <ubitux> ...or maybe they will change or create another convention
[12:45] <cone-911> ffmpeg.git 03Anton Khirnov 07master:23a211cbba0b: lavc: change all decoders to behave consistently with AV_EF_CRCCHECK.
[12:46] <cone-911> ffmpeg.git 03Michael Niedermayer 07master:84f972f9944a: Merge commit '23a211cbba0b7c9ee694040031b2e5da1be54a00'
[13:06] <cone-911> ffmpeg.git 03Anton Khirnov 07master:97de206b44a4: lavc: disable CRC checking by default
[13:06] <cone-911> ffmpeg.git 03Michael Niedermayer 07master:252f56636ca0: Merge commit '97de206b44a48da726807cc3e7b9448a8112760b'
[13:13] <cone-911> ffmpeg.git 03Vittorio Giovara 07master:5c439b41d048: avfilter: have avfilter_get_by_name return const for next bump
[13:13] <cone-911> ffmpeg.git 03Michael Niedermayer 07master:a1ce060c5191: Merge commit '5c439b41d0489412c0a4cf6dfb98915251677b8e'
[13:21] <cone-911> ffmpeg.git 03Vittorio Giovara 07master:884c7a6eb86a: avfilter: fix const use of avfilter_next
[13:21] <cone-911> ffmpeg.git 03Michael Niedermayer 07master:054e4bab83b7: Merge commit '884c7a6eb86a9bc05abafc058b279018ded7de5f'
[13:26] <cone-911> ffmpeg.git 03Martin Storsjö 07master:f2521563d1a0: g722dec: Change bits_per_codeword to the right option type
[13:27] <cone-911> ffmpeg.git 03Michael Niedermayer 07master:683ab5fec994: Merge commit 'f2521563d1a0c3e2a21892f59f3327b82b3db7fc'
[13:37] Action: durandal_1707 wants to remove libmpcodecs
[13:53] <cone-911> ffmpeg.git 03Anton Khirnov 07master:2ba68dd044ca: lavf: remove unreliable timestamp guessing heuristic
[13:53] <cone-911> ffmpeg.git 03Michael Niedermayer 07master:feba750dcd49: Merge commit '2ba68dd044ca8fc591139c05563840f546a9c0c0'
[13:57] <ubitux> michaelni: http://lists.libav.org/pipermail/libav-devel/2013-October/052452.html
[13:57] <ubitux> it seems to fix one
[13:57] <nevcairiel> michaelni: to be fair, the field removed from AVStream is from a section of AVStream that explicitly states above that its not part of the API
[13:58] <nevcairiel> so anyone that used it was wrong :d
[13:58] <durandal_1707> API != ABI
[13:58] <michaelni> IIRC ffmpeg & avconc access these fields when i last looked
[13:59] <nevcairiel> then they are doing it wrong
[14:00] <ubitux> how can avconv access it and still build without changes?
[14:00] <nevcairiel> probably one of the other fields in that section
[14:01] <nevcairiel> fwiw, i had that block of code disabled in my repository for a long time because it broke some files
[14:01] <michaelni> ubitux, avconv accesses fields after the removed field
[14:01] <nevcairiel> not sure if thats still the case today
[14:02] <ubitux> michaelni: ah, ok
[14:03] <michaelni> nevcairiel, do you have a file that is happier without the block ?
[14:04] <michaelni> i sure would prefer to get rid of it ...
[14:04] <nevcairiel> then just do it and fix any samples that are broken afterwards properly
[14:05] <nevcairiel> if any even show up
[14:05] <nevcairiel> i wonder if i still have those files, it was quite some time ago
[14:07] <durandal_1707> michaelni: you merged err_detect change again
[14:07] <durandal_1707> also what about that flags option thing that remove range checking?
[14:08] <durandal_1707> all those things change behaviour, making our 'beloved' (l)users 'happy'
[14:09] <michaelni> durandal_1707, ill post a patchset about cleaning up err_detect, i just need to write and test it first
[14:10] <michaelni> but my opinion is that crc testing should not be enabled by default when its slow
[14:11] <michaelni> and i suspect this affects more than hevc
[14:15] <durandal_1707> changing default values is never good
[14:16] <durandal_1707> if users want speed they can change it
[14:18] <michaelni> durandal_1707, comment noted but i suspect this is not what the majority of the developers or users wants
[14:19] <durandal_1707> and it is certainly bad way to show that your e-penis is bigger than someone else. disabling crc checking is not performance boost - you are not improving code by making it faster, you just stopped calling it
[14:19] <durandal_1707> if you are so worried about performance why it was not 0 from start?
[14:20] <cone-911> ffmpeg.git 03Anton Khirnov 07master:a1c5cc429d99: lavc: don't set AVFrame.pts to random numbers in decoders.
[14:20] <cone-911> ffmpeg.git 03Michael Niedermayer 07master:17e47ec8be20: Merge commit 'a1c5cc429d99216406170eac7e8352860076d3e8'
[14:21] <durandal_1707> expect now posts like: ffmpeg lossless decoder is not lossless/bit-exact
[14:22] <nevcairiel> crc checks dont change that
[14:22] <durandal_1707> other decoders check it always
[14:23] <durandal_1707> but only here, suddenly, it get disabled
[14:23] <durandal_1707> without even small notice in Changelog
[14:23] <durandal_1707> but that is already known story, why would you ever care about sane API
[14:24] <michaelni> durandal_1707, like i said ill post a patchset to clean it up, everyone can comment then and we can discuss until everyone is happy
[14:25] <michaelni> my oppinon is that defaults should be what makes the majority of users happy
[14:25] <michaelni> and that can mean that sometimes for a codec a CRC check should be done by default and sometimes it should not
[14:26] <michaelni> for some decoders the CRC checks are important for detecting errors and doing error concealment
[14:26] <michaelni> for hevc it just slows things down
[14:26] <michaelni> (from an end users point of view)
[14:29] <ubitux> can someone on libav side tell them to stop doing this shit? http://pastie.org/8437239
[14:34] <durandal_1707> they will tell: do not merge it.
[14:34] <av500> do not merge it
[14:34] <av500> :)
[14:35] <durandal_1707> ubitux, you should allow kids to learn K&R
[14:44] <cone-911> ffmpeg.git 03Michael Niedermayer 07master:6c82c87dbbc0: ac3dec: fix outptr increment.
[14:44] <cone-911> ffmpeg.git 03Michael Niedermayer 07master:269b3c8799f4: Merge remote-tracking branch 'qatar/master'
[16:31] <durandal_1707> ubitux: how was in school?
[16:31] <ubitux> huh?
[16:31] <ubitux> you mean my cosmetics teaching?
[16:35] <durandal_1707> psss! there are sleepers here
[16:38] <ubitux> -                v= ff_mpeg4_default_non_intra_matrix[i];
[16:38] <ubitux> +                v =
[16:38] <ubitux> +                    ff_mpeg4_default_non_intra_matrix[i];
[16:38] <ubitux> lol
[16:38] <ubitux> i forgot that one
[16:39] <durandal_1707> 'v' should be in separate line
[16:42] Action: durandal_1707 working on #3089
[17:13] <durandal_1707> michaelni: why init_get_bits work for size == 0?
[17:14] <durandal_1707> it appears that may cause that get_bits1 always return 1 even if there are no bits left
[17:15] <durandal_1707> if size is 0, there should no bit !=0 be returned at all
[17:15] <durandal_1707> why such an oversight from programmer?
[17:18] <durandal_1707> i have patch here already, that causes segv if s/< 0/<= 0
[17:24] <durandal_1707> same happens if bit size is < 0
[17:28] <durandal_1707> that points to unrelated bug
[17:34] <durandal_1707> looks like there is another overread...
[17:51] <ubitux> oh, spi donation, that is a cool thing
[17:52] <durandal_1707> if you actually get something donated
[17:53] <durandal_1707> and if one never check return code of init_get_bits(8) crash happens
[17:54] <durandal_1707> i expected sane implementation would not crash but just returns 0
[18:02] <ubitux> 2.2 ’ titanium?
[18:04] <cone-911> ffmpeg.git 03Michael Niedermayer 07master:c00686518c0b: avcodec/h263dec: fix handling of AV_EF_EXPLODE
[18:04] <cone-911> ffmpeg.git 03Michael Niedermayer 07master:162126bb174c: avutil/opt: check flags validity in write_number()
[18:05] <durandal_1707> 2.2 -> Clement
[18:06] <ubitux> o_o
[18:06] <ubitux> wut?
[18:07] <durandal_1707> we are not allowed to put our names?
[18:08] <ubitux> oh, you are also named Clément? @_@
[18:09] <durandal_1707> nope
[18:10] <durandal_1707> if you insist it can be Paul
[18:11] <ubitux> :)
[18:25] <durandal_1707> is it alowed to set pointer to NULL + X ? even if pointer is never tuched after that
[18:26] <nevcairiel> technically its invalid to even have a pointer to a forbidden memory address
[18:26] <nevcairiel> not that it will ever cause issues
[18:28] <durandal_1707> buffer_end is used by some decoders: h263 and mpeg12
[18:30] <durandal_1707> ah, but it is NULL + 0
[18:30] <durandal_1707> so just setting buffer to NULL causes crash
[18:44] <durandal_1707> hmm, perhaps it only crash for get_bits1
[18:52] <llogan> burek: why have your ffmpeg builds stopped on 2013-10-25?
[18:55] <durandal_1707> michaelni: i added check in get_bits1 if buffer is null, but perhaps using stub buffer is better, as that avoid check
[19:06] <michaelni> get_bits/get_bits1 is speed critical code, why do you want to add a check in it ?
[19:07] <michaelni> nevcairiel, its allowed to have a pointer to at least 1 element after an array, like its commonly used with end pointers
[19:08] <nevcairiel> well ok, i was thinking more of total random stuff =)
[19:11] <durandal_1707> michaelni: i found nice trick and wrote 'hack'
[19:11] <durandal_1707> michaelni: btw, iirc there are code that use get_bits(gb, 1)
[19:24] <llogan> someone asked me on twitter if ffmepg 1.1.7 builds with Visual Studio 2013. I have no clue. Anyone have an idea?
[19:25] <nevcairiel> master does
[19:25] <nevcairiel> thats all i can tell you
[19:25] <llogan> ok, good enough for me. thanks.
[19:35] <wm4> hm the documented ogg maintainer (David Conrad) doesn't seem to be too active
[19:35] <wm4> so I suppose it won't happen he will be around to ok mathstuf's ogg metadata update patches
[19:36] <wm4> how does it go from there?
[19:37] <durandal_1707> in past, just nothing
[19:38] <durandal_1707> if you care for ogg you could review it
[19:41] <wm4> durandal_1707: so should I just post a bunch of replies containing just "ok"?
[19:41] <durandal_1707> if you actually review code, do i need to explain what that means?
[19:42] <durandal_1707> there is also some doxy ...
[19:55] <Compn> wm4 : if there is no maintainer, michaelni reviews patches before committing. but its always great if someone else does a review too...
[19:56] <durandal_1707> so get_bits1(gb) is faster than get_bits(gb, 1) ?
[20:00] <durandal_1707> kierank: so you pay people before they do work?
[20:00] <kierank> i have a proper contract
[20:00] <durandal_1707> do you have place for me?
[20:01] <kierank> ?
[20:01] <kierank> you're taking a random guy's word
[20:01] <kierank> and all these super secret companies
[20:02] <durandal_1707> i agree, you are lucky if you get 50% now and 50% after
[20:04] <kierank> legally (IANAL) that guy has made a public promise but the supporting companies may not have
[20:06] <kierank> also the fact these companies are secret makes it an imbalanced marketplace
[20:07] <durandal_1707> imbalanced in what sense?
[20:08] <kierank> you don't know the value of the goods (yadif) to the company
[20:09] <kierank> like is it for an ipod encoder software thats £20 or is it for a crazy expensive five figure equipment
[20:09] <durandal_1707> so it should all be open like source code is?
[20:09] <kierank> what should be open?
[20:10] <kierank> if you are hidden behind a shell you don't know what the net value of lgpl yadif is
[20:10] <durandal_1707> that secret company say we use it for that and that and gross is that and that
[20:10] <kierank> in practice you sign an nda but yes
[20:10] <durandal_1707> then pick what x264 did
[20:11] <kierank> there's no problem doing the one time payment for relicensing as long as you know who is funding it and how much utility it gives them
[20:14] <cone-911> ffmpeg.git 03Peter Ross 07master:69a042ee95c6: mpegts: demux synchronous SMPTE 336M Key-Length-Value (KLV) metadata
[20:15] <kierank> wow klv metadata :)
[20:15] <kierank> Compn won't like that
[20:29] <llogan> is that changelog worthy?
[20:37] <ubitux> yay almost done with 2x2
[20:44] <ubitux> BBB: ping :)
[20:47] <durandal11707> ubitux: you dissapointed me
[20:47] <ubitux> ah? :(
[20:48] <durandal11707> talking with dark side
[20:48] <ubitux> well i didn't want that shit to reach ffmpeg
[20:48] <wm4> darkside?
[20:48] <durandal11707> worse shit reached already
[20:48] <ubitux> and anyway, i'm supposed to contribute if i want to make a complain
[20:48] <durandal11707> indeed
[20:48] <ubitux> so i'm stopping for now
[20:51] <ubitux> oh i can simplify even further..
[20:53] <durandal11707> michaelni: that amv test is fucking retarded
[20:53] <durandal11707> and decoder
[20:55] <durandal11707> michaelni: all what my code does it change that 0 behaviour
[20:56] <durandal11707> that means amv decoder is in ugly state
[20:56] <ubitux> BBB: patch on the ml
[20:56] <durandal11707> shit, same happens with jpegs and mjpeg
[20:57] <nevcairiel> ubitux: why is there idct twice in the name? I'm sure there must be some sense in there
[20:58] <ubitux> because it can be idct+idct, idct+iasdt, iadst+idct, iadst+iadst
[21:01] <durandal11707> anything that use mjpeg broke
[21:01] <durandal11707> ubitux: benchmark numbers?
[21:01] <ubitux> durandal11707: yeah i should :p
[21:22] <ubitux> durandal11707: done
[21:25] <durandal11707> and what is gain overall?
[21:25] <durandal11707> you can post it here
[21:27] <ubitux> no difference
[21:27] <ubitux> which is expected
[21:28] <durandal11707> perhaps you use wrong sample?
[21:28] <ubitux> i use a ~40sec 1080p sample
[21:28] <ubitux> anyway, it's only idct idct 4x4
[21:28] <ubitux> there is 8x8, 16x16 and 32x32 to do
[21:29] <ubitux> and probably the iadst combinations
[21:29] <durandal11707> you can do them faster now or?
[21:29] <ubitux> now?
[21:29] <ubitux> like in a few minutes?
[21:30] <ubitux> i did the simplest one :p
[21:31] <durandal11707> well it should be somethinkg like 1st: 8h -> 2nd: 6h -> 3th: 3h -> 4th: 1h -> 5th: 5 minutes
[21:33] <ubitux> :D
[21:46] <durandal11707> michaelni: mjpeg calls init_get_bits8 with bit_size = 0
[21:46] <durandal11707> should it not error out if size is 0, eg just return 0?
[21:46] <durandal11707> and emulate /dev/zero
[22:22] <Compn> kierank : what does klv metadata have to do with me? :)
[22:22] <kierank> Compn: UAVs
[22:23] <llogan> they'll have the metadata location of your beard at all times
[22:23] <Compn> nooooooooooooooo
[22:23] <llogan> here...use this beard net
[22:23] <llogan> it has a painting of a "coldchin" on it so they won't know it's you
[22:25] <Compn> how quickly 'academic research' turns into 'military weapon' :(
[22:26] <Compn> also, 1millionth feature request to play and record video at the same time. this time from an rc plane pilot who wants to fly via video
[22:26] <Compn> without pipe'ing video or nonsense like that
[22:27] <Compn> vlc had 20 sec lag, so user couldnt use it effectively for realtime flight
[22:27] <llogan> ubitux: hurry up and apply it so i can add "more optimizations" in my HEVC 'n VP9 decoding news patch
[22:27] <durandal11707> that is sf
[22:28] <Compn> michaelni : feature request for ffplay, add encoding support . or alternatively, add sdl output to ffmpeg
[22:28] <Compn> should be easy...
[22:28] <Compn> probably is not
[22:28] <durandal11707> direct that to trac.ffmpeg.org
[22:28] <Compn> ok
[22:29] <Compn> whats it called? sdl outdev ?
[22:29] <durandal11707> man ffmpeg-devices
[22:31] <Compn> oh its already got one
[22:31] <Compn> ffmpeg -i INPUT -vcodec rawvideo -pix_fmt yuv420p -window_size qcif -f sdl "SDL output"
[22:32] <Compn> wow neat , ffmpeg -i INPUT OUTPUT -f xv display
[22:33] <durandal11707> now you can see your beard at once
[22:38] <cone-911> ffmpeg.git 03Stephen Hutchinson 07master:bd97ba72dc6f: avisynth: Introduce USING_AVISYNTH macro
[22:38] <cone-911> ffmpeg.git 03Stephen Hutchinson 07master:1549122d2698: avisynth: Change most of the comments to /* */ from //
[22:38] <cone-911> ffmpeg.git 03Stephen Hutchinson 07master:2c18bfe6af84: avisynth: Cosmetics
[22:38] <cone-911> ffmpeg.git 03Stephen Hutchinson 07master:d10d60be6882: avisynth: Compact AvxSynth's avoidance of 2.6's colorspaces.
[22:38] <cone-911> ffmpeg.git 03Stephen Hutchinson 07master:da9852670c69: avisynth: Don't declare structs anonymously
[22:38] <cone-911> ffmpeg.git 03Stephen Hutchinson 07master:df6279e73701: avisynth: Remove a couple of useless AviSynthContext casts
[22:38] <cone-911> ffmpeg.git 03Stephen Hutchinson 07master:cf31c1d4f835: avisynth: Remove outdated undef block
[22:38] <cone-911> ffmpeg.git 03Stephen Hutchinson 07master:c7f9aab80142: avisynth: Use AV_* prefixes for video and audio IDs
[22:38] <cone-911> ffmpeg.git 03Stephen Hutchinson 07master:7ac67583c3a8: avisynth: Switch a couple of AVERROR_UNKNOWNs to AVERROR(ENOMEM)
[22:38] <cone-911> ffmpeg.git 03Stephen Hutchinson 07master:ac9529ceec19: avisynth: Simplify a stray av_log message
[22:38] <cone-911> ffmpeg.git 03Stephen Hutchinson 07master:f87a2e1272b6: avisynth: Factor out a couple of returns
[23:01] <llogan> my shitty tests shows native vp9 slower to decode, but i don't know what i'm doing
[23:01] <durandal11707> send patch
[23:02] <llogan> yeah, sure. let me just whip up that 32x32 for you in a minute
[23:33] <BBB> ubitux: ok will review hopefully tonight
[23:33] <BBB> ubitux: looked good on a quick glance earlier today
[23:55] <cone-911> ffmpeg.git 03Michael Niedermayer 07master:f0b26bf0f56e: avcodec/dvdec: dont try to decode ac when theres no input
[23:55] <cone-911> ffmpeg.git 03Michael Niedermayer 07master:e510171f31b9: avcodec: fix old codec ids
[23:55] <cone-911> ffmpeg.git 03Michael Niedermayer 07master:f1f0b01c4700: avformat/gxf: fix old codec id
[00:00] --- Tue Oct 29 2013