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

burek burek021 at gmail.com
Fri Sep 29 03:05:03 EEST 2017

[00:00:18 CEST] <BBB> oh boy
[00:00:40 CEST] <BBB> btw
[00:00:46 CEST] <Compn> triggered
[00:05:07 CEST] <cone-906> ffmpeg 03Diego Biurrun 07master:dad7a9c7c0ae: configure: Rework dependency handling for conflicting components
[00:05:08 CEST] <cone-906> ffmpeg 03James Almer 07master:7659f35638e2: Merge commit 'dad7a9c7c0ae8ebc56f2e3a24e6fa4da5c2cd491'
[00:07:33 CEST] <cone-906> ffmpeg 03Martin Storsjö 07master:57ec83e4246b: omx: Use the EOS flag to handle flushing at the end
[00:07:34 CEST] <cone-906> ffmpeg 03James Almer 07master:f1cdfd29590c: Merge commit '57ec83e4246b21c2f0c068b9151d806737d4497f'
[00:09:23 CEST] <jkqxz> jamrial:  d1e7e4fbe wasn't quite enough.  <http://sprunge.us/gPeD>?
[00:09:42 CEST] <cone-906> ffmpeg 03Diego Biurrun 07master:c546147db07d: configure: Correctly recurse in do_check_deps()
[00:09:43 CEST] <cone-906> ffmpeg 03James Almer 07master:57db1faf7a47: Merge commit 'c546147db07d16a76c2fb698d2e8a3057f393475'
[00:10:40 CEST] <jamrial> jkqxz: yeah, saw it on fate. I don't have such a system so i couldn't test if it was enough back then
[00:10:59 CEST] <jamrial> jkqxz: the current error is missing size_t, so including stddef.h should fix it, i think
[00:11:04 CEST] <jkqxz> I've actually tested it, now works for me.
[00:11:22 CEST] <jkqxz> avcodec.h is enough for everything currently missing (size_t, AVFrame, AVPacket).
[00:12:07 CEST] <jamrial> ok, push it then
[00:12:47 CEST] <wm4> hm I noticed the v4l encoder is preferred over libx264
[00:12:50 CEST] <wm4> that's probably not right
[00:12:58 CEST] <cone-906> ffmpeg 03Mark Thompson 07master:da53c424b96f: lavc/v4l2: Add missing header include
[00:13:08 CEST] <jkqxz> jamrial:  ^  Thanks.
[00:13:29 CEST] <jamrial> thanks to you for fixing it, if anything :p
[00:13:47 CEST] <jkqxz> wm4:  Huh, so it is.
[00:13:53 CEST] <JEEB> call for pokes on https://ffmpeg.org/pipermail/ffmpeg-devel/2017-September/216816.html , https://ffmpeg.org/pipermail/ffmpeg-devel/2017-September/216699.html :3
[00:13:57 CEST] <jkqxz> (Barfs immediately on x86, of course.)
[00:13:59 CEST] <jkqxz> Let me fix that too.
[00:15:44 CEST] <jamrial> wm4, jkqxz: should be a matter of changing the order in allcodecs.c
[00:16:37 CEST] <jkqxz> Yeah, on it.
[00:24:27 CEST] <jkqxz> wm4: jamrial:  <http://sprunge.us/JGQe>?
[00:25:03 CEST] <jamrial> jkqxz: no, just move the encoders. keep the decoders where they are
[00:25:32 CEST] <jamrial> also, i don't think h263 needs to be moved
[00:26:00 CEST] <jamrial> the internal encoder entry is before the v4l2 entry, so it's fine
[00:26:11 CEST] <jkqxz> Why does that help?  CUVID/NVENC is a similar case, and is in the lower list.
[00:26:40 CEST] <wm4> hm I'm too tired to see the difference
[00:27:04 CEST] <jamrial> the point here is to make sure the hw encoders are after the external software libraries like libx264/5, libvpx and such
[00:27:23 CEST] <jamrial> hw decoders being right below the internal decoder in the list is ok
[00:27:32 CEST] <jkqxz> Yes.  And moving them all like this is the most consistent way to do it.
[00:27:38 CEST] <jkqxz> Otherwise they are very mixed up.
[00:28:13 CEST] <jkqxz> Hmm, qsv is the way you are saying.
[00:28:21 CEST] <jamrial> jkqxz: something like https://pastebin.com/raw/ENyr3c4s
[00:28:44 CEST] <jamrial> ah, there's a mistake there now that i see
[00:28:59 CEST] <jamrial> added H264_V4L2M2M decoder twice :p
[00:29:00 CEST] <jkqxz> It should move all of them.
[00:29:02 CEST] <jamrial> second should be encoder
[00:29:13 CEST] <jkqxz> I'll do the same thing as qsv, then.
[00:29:51 CEST] <jamrial> the decoders down there like vp9_cuvid and such could be moved up IMO
[00:30:41 CEST] <jamrial> as long as they are below the native decoder it should be ok
[00:30:50 CEST] <jamrial> but that's some other patch
[00:31:11 CEST] <jamrial> right now libx264/5/vpx should have priority
[00:31:57 CEST] <jkqxz> Really those encoders just shouldn't be used at all without being requested.
[00:32:42 CEST] <jkqxz> VAAPI should never be picked by default.
[00:35:36 CEST] <jkqxz> ... maybe there should be a "don't use this by default" flag.
[00:35:53 CEST] <JEEB> doesn't sound like a bad idea
[00:35:55 CEST] <jkqxz> (Kindof like AVOID_PROBING.)
[00:40:10 CEST] <cone-906> ffmpeg 03Martin Storsjö 07master:0331c3f5e8cb: arm: vp9itxfm: Make the larger core transforms standalone functions
[00:40:11 CEST] <cone-906> ffmpeg 03Martin Storsjö 07master:115476018d2c: aarch64: vp9itxfm: Make the larger core transforms standalone functions
[00:40:12 CEST] <cone-906> ffmpeg 03Martin Storsjö 07master:47b3c2c18d18: arm: vp9itxfm: Move the load_add_store macro out from the itxfm16 pass2 function
[00:40:13 CEST] <cone-906> ffmpeg 03Martin Storsjö 07master:79d332ebbde8: aarch64: vp9itxfm: Move the load_add_store macro out from the itxfm16 pass2 function
[00:40:14 CEST] <cone-906> ffmpeg 03Martin Storsjö 07master:5eb5aec475aa: arm: vp9itxfm: Do a simpler half/quarter idct16/idct32 when possible
[00:40:15 CEST] <cone-906> ffmpeg 03Martin Storsjö 07master:a63da4511d0f: aarch64: vp9itxfm: Do separate functions for half/quarter idct16 and idct32
[00:40:16 CEST] <cone-906> ffmpeg 03Martin Storsjö 07master:3933b86bb93a: arm: vp9itxfm: Share instructions for loading idct coeffs in the 8x8 function
[00:40:17 CEST] <cone-906> ffmpeg 03Martin Storsjö 07master:4da4b2b87f08: aarch64: vp9itxfm: Share instructions for loading idct coeffs in the 8x8 function
[00:40:18 CEST] <cone-906> ffmpeg 03Martin Storsjö 07master:ed8d293306e1: aarch64: vp9itxfm: Use a single lane ld1 instead of ld1r where possible
[00:40:19 CEST] <cone-906> ffmpeg 03Martin Storsjö 07master:3dd7827258dd: aarch64: vp9itxfm: Use the right lane sizes in 8x8 for improved readability
[00:40:20 CEST] <cone-906> ffmpeg 03Martin Storsjö 07master:8476eb0d3ab1: aarch64: vp9itxfm: Update a comment to refer to a register with a different name
[00:40:21 CEST] <cone-906> ffmpeg 03Martin Storsjö 07master:0c0b87f12d48: aarch64: vp9itxfm: Fix incorrect vertical alignment
[00:40:22 CEST] <cone-906> ffmpeg 03James Almer 07master:276f035a4d0f: Merge commit '0c0b87f12d48d4e7f0d3d13f9345e828a3a5ea32'
[00:41:55 CEST] <jkqxz> jamrial:  <http://sprunge.us/LcZc>?
[00:43:13 CEST] <jamrial> jkqxz: yeah, that should be good
[00:45:40 CEST] <wm4> yeah looks good
[00:46:19 CEST] <jkqxz> Output looks right.  Building with libx264 only, it gets picked for mkv.  webm picks vp8_v4l2m2m and barfs.
[00:47:51 CEST] <thebombzen> while we're on automatic order, for NUT, mpeg4 gets picked by default rather than ffv1
[00:48:00 CEST] <thebombzen> this doesn't make sense
[00:48:24 CEST] <cone-906> ffmpeg 03Mark Thompson 07master:c1f22c295930: lavc: Move V4L2 encoders lower in the list
[00:49:22 CEST] <jkqxz> thebombzen:  Yes.  Change <http://git.videolan.org/?p=ffmpeg.git;a=blob;f=libavformat/nutenc.c;h=a92ff55c01373bd7db36088114888a1c1d15ef69;hb=HEAD#l1224>?
[00:49:48 CEST] <thebombzen> sure. I'll send a patch to the mailing list
[00:49:58 CEST] <wm4> lol at that mpeg audio logic
[00:50:23 CEST] <JEEB> lol
[00:51:25 CEST] <jkqxz> Matroska is libvorbis ? Vorbis : AC-3.  Wtf?
[00:52:33 CEST] <thebombzen> should I also change the audio to FLAC or libopus? 
[00:53:44 CEST] <thebombzen> I'm thinking changing the default for nut to FLAC off of libvorbis, if we're using FFV1 as the default video codec
[00:54:10 CEST] <jamrial> jkqxz: external library encoder first if possible, and if not available, a decent internal encoder
[00:54:37 CEST] <jkqxz> Well, AC-3 would not be my first choice of internal encoder.
[00:54:38 CEST] <jamrial> that's the logic used in these muxers
[00:54:44 CEST] <jkqxz> But yes, I suppose it's fair.
[00:55:00 CEST] <jamrial> it could maybe be changed to aac now, if it's no longer tagged as experimental
[00:55:17 CEST] <wm4> we don't have a proper internal opus encoder yet?
[00:55:27 CEST] <jamrial> yes, but tagged experimental i think?
[00:55:43 CEST] <thebombzen> I believe psy was added recently to internal opus but that doesn't mean it's ready
[00:56:10 CEST] <jamrial> for mkv aac is the better choice. opus for webm. but i don't know if the internal encoders for both are good enough for that
[01:01:28 CEST] <JEEB> aac encoder is pretty good
[01:01:31 CEST] <JEEB> at this point
[01:05:30 CEST] <thebombzen> would it make sense to set the default encoder for nut to be flac internal rather than libvorbis?
[01:05:47 CEST] <thebombzen> currently the order is libvorbis -> libmp3lame -> mp2
[01:16:14 CEST] <thebombzen> jkqxz: I just sent a patch to the mailing list. I also had to change the fate tests, because they do not specify the codec so the output files changed.
[01:28:04 CEST] <cone-906> ffmpeg 03Derek Buitenhuis 07master:77c23704c769: avcodec: Mark some codecs with threadsafe init as such
[01:28:05 CEST] <cone-906> ffmpeg 03James Almer 07master:d96363cc8437: Merge commit '77c23704c769168e4210956314775a1931f6aa0b'
[01:30:05 CEST] <cone-906> ffmpeg 03Timo Rothenpieler 07master:a52976c0feab: nvenc: make gpu indices independent of supported capabilities
[01:30:06 CEST] <cone-906> ffmpeg 03James Almer 07master:91feb62b3dbf: Merge commit 'a52976c0feab6e86138983c248bd01fa45cdda69'
[01:32:23 CEST] <cone-906> ffmpeg 03Martin Storsjö 07master:bc2589763042: utvideodec: Add a missing include
[01:32:24 CEST] <cone-906> ffmpeg 03James Almer 07master:09b1a9d74038: Merge commit 'bc2589763042dc2384b724b203ec778f35bcebad'
[01:34:48 CEST] <cone-906> ffmpeg 03Luca Barbato 07master:b6093e8c72a8: hlsenc: Correctly write down all 16 bytes in hex
[01:34:49 CEST] <cone-906> ffmpeg 03James Almer 07master:9cf1376aadcd: Merge commit 'b6093e8c72a80710f086c678ab0730cf30953b5c'
[01:36:07 CEST] <cone-906> ffmpeg 03Vittorio Giovara 07master:ce6d72d10776: imgutils: Document av_image_get_buffer_size()
[01:36:08 CEST] <cone-906> ffmpeg 03James Almer 07master:1a115a31d430: Merge commit 'ce6d72d10776b03c6780d4aa676414ce002285d4'
[01:42:22 CEST] <rcombs> isn't the AC-3 encoder hot garbage
[01:42:42 CEST] <JEEB> supposedly it's based on one of the other OSS ac3 encoders
[01:42:58 CEST] <JEEB> so it's top tier OSS quality, IIRC. no idea how it compares against non-OSS encoders
[01:43:01 CEST] <JEEB> lol
[01:44:00 CEST] <rcombs> or to things that aren't AC3
[01:44:21 CEST] <JEEB> I kind of expected one to actually somehow need AC3 if it came up
[01:44:37 CEST] <rcombs> well it's the default for matroska if libvorbis isn't available apparently
[01:44:54 CEST] <JEEB> top keks
[01:47:29 CEST] <cone-906> ffmpeg 03Vittorio Giovara 07master:53ea595eec98: mov: Rework stsc index validation
[01:47:30 CEST] <cone-906> ffmpeg 03James Almer 07master:b35f6d3aa359: Merge commit '53ea595eec984e3109310e8bb7ff4b5786d91057'
[01:50:12 CEST] <iive> JEEB: ffmpeg ac3 decoder is quite old. and I'm not even aware that there is another FOSS ac3 encoder
[01:53:23 CEST] <cone-906> ffmpeg 03Martin Storsjö 07master:5e0c2158fbc7: aarch64: vp9mc: Simplify the extmla macro parameters
[01:53:24 CEST] <cone-906> ffmpeg 03Martin Storsjö 07master:fea92a4b57d1: arm: vp9mc: Calculate less unused data in the 4 pixel wide horizontal filter
[01:53:25 CEST] <cone-906> ffmpeg 03Martin Storsjö 07master:388e0d2515bc: aarch64: vp9mc: Calculate less unused data in the 4 pixel wide horizontal filter
[01:53:26 CEST] <cone-906> ffmpeg 03Martin Storsjö 07master:a76bf8cf1277: arm: vp9itxfm: Optimize 16x16 and 32x32 idct dc by unrolling
[01:53:27 CEST] <cone-906> ffmpeg 03Martin Storsjö 07master:3fcf788fbbcc: aarch64: vp9itxfm: Optimize 16x16 and 32x32 idct dc by unrolling
[01:53:28 CEST] <cone-906> ffmpeg 03Martin Storsjö 07master:e1f9de86f454: arm/aarch64: vp9lpf: Calculate !hev directly
[01:53:29 CEST] <cone-906> ffmpeg 03Martin Storsjö 07master:435cd7bc9967: arm: vp9lpf: Use orrs instead of orr+cmp
[01:53:30 CEST] <cone-906> ffmpeg 03James Almer 07master:7f89e9621b9e: Merge commit '435cd7bc99671bf561193421a50ac6e9d63c4266'
[01:56:58 CEST] <cone-906> ffmpeg 03Anton Khirnov 07master:99684f3ae752: avio: add a destructor for AVIOContext
[01:56:59 CEST] <cone-906> ffmpeg 03Anton Khirnov 07master:6f554521afdf: Use the new AVIOContext destructor.
[01:57:00 CEST] <cone-906> ffmpeg 03James Almer 07master:4ad0264ab3d4: Merge commit '6f554521afdf7ab4edbfaa9536660a1dca946b19'
[02:01:01 CEST] <cone-906> ffmpeg 03Anton Khirnov 07master:5c8a5765dc5f: scale_npp: explicitly set the output frames context for passthrough mode
[02:01:02 CEST] <cone-906> ffmpeg 03James Almer 07master:5256a86da067: Merge commit '5c8a5765dc5f4e29afb85b95be393c30f45412a8'
[03:12:44 CEST] <ldts> wm4: it seems that I was handling correctly the end of stream and the only issue is that the driver I was using for testing was failing to interpret the new API (so the dummy buffer is not required); on my end all it will be needed is to remove the timeout...once I have a fixed version of the driver that allows me to test I will push the correction to the baseline. 
[03:14:50 CEST] <wm4> oh nice
[03:31:25 CEST] <jamrial> wm4: do i keep the return value as size_t, or can i change it to int? seems stupid used size_t for a value that will be 32 max right now, 64 next year or so, and 128 in like five years
[03:31:36 CEST] <jamrial> would the difference between the projects be too much of a headache?
[03:35:35 CEST] <wm4> sigh, that issue again
[03:35:58 CEST] <wm4> Libav seems hell-bent on introducing inconsistencies for a few decades until eventually everything is size_t
[03:36:22 CEST] <wm4> I'd say just keep it size_t since as a return value it doesn't introduce too much confusion?
[03:37:28 CEST] <jamrial> sure
[03:38:01 CEST] <jamrial> and yes, the whole size_t thing is getting silly. used for things where it makes no sense
[03:38:08 CEST] <atomnuker> well then don't use it
[03:38:26 CEST] <atomnuker> I think BBB also objected to using special types for non-special variables
[03:39:06 CEST] <atomnuker> e.g. uint8_ts always used for bools, etc., since the compiler may have to implicitly convert them to ints or whatever
[03:39:55 CEST] <wm4> jamrial: they insist on using it "semantically" correct, so alignment would have to be size_t of course
[03:49:47 CEST] <Ahti333> wm4 did you see my reply to your comment on my patch (http://ffmpeg.org/pipermail/ffmpeg-devel/2017-September/216885.html), i messed up the thread structure trying to reply manually
[03:50:42 CEST] <wm4> Ahti333: thread appears fine here, and I didn't reply yet
[03:51:19 CEST] <Ahti333> oh, weird, the archives show me replying to myself but whatever :D, take your time, thanks :)
[03:52:38 CEST] <wm4> Ahti333: doing it in a way that prevents overriding chapters in "proper" file formats like it's done for apic sounds good to me
[03:52:50 CEST] <wm4> Ahti333: do you think that would be hard to add?
[03:56:19 CEST] <Ahti333> I'm not sure, I think it would require adjusting all the places ff_id3v2_read(_dict) is used (10~ places in the different fomat handlers). splitting up the code that reads the tags and storing it in the extra_meta itself should be relatively simple.
[03:58:11 CEST] <wm4> Ahti333: you could also take a brief look at formats that support chapters (not many) and look whether id3-added chapters could make them crash/UB
[03:58:19 CEST] <wm4> if not, your patch would be fine
[04:15:48 CEST] <cone-906> ffmpeg 03Anton Khirnov 07master:e6bff23f1e11: cpu: add a function for querying maximum required data alignment
[04:15:49 CEST] <cone-906> ffmpeg 03James Almer 07master:522f87708653: Merge commit 'e6bff23f1e11aefb16a2b5d6ee72bf7469c5a66e'
[04:15:50 CEST] <cone-906> ffmpeg 03James Almer 07master:3b345d389be2: avutil/cpu: split flag checks per arch in av_cpu_max_align()
[04:18:32 CEST] <Ahti333> wm4 I don't see anything that would cause a crash. if both the format and id3 contain chapter info, it would probably mess up the order and/or cause duplicate chapters in s->chapters
[04:18:40 CEST] <cone-906> ffmpeg 03Anton Khirnov 07master:f44ec22e095c: lavc: use av_cpu_max_align() instead of hardcoding alignment requirements
[04:18:41 CEST] <cone-906> ffmpeg 03James Almer 07master:24ee1b8c6343: Merge commit 'f44ec22e095c5ba00ffeadd891655c456e3dd014'
[04:23:44 CEST] <Ahti333> (or if they don't contain identical info, some or all of the id3 info would be overwritten by the format chapters)
[04:34:56 CEST] <cone-906> ffmpeg 03Anton Khirnov 07master:4de220d2e375: frame: allow align=0 (meaning automatic) for av_frame_get_buffer()
[04:34:57 CEST] <cone-906> ffmpeg 03James Almer 07master:7aa6b8a68fce: Merge commit '4de220d2e3751c459f8739a08ac6ca52e63eba30'
[04:37:08 CEST] <cone-906> ffmpeg 03wm4 07master:04f3bd349651: AVFrame: add an opaque_ref field
[04:37:09 CEST] <cone-906> ffmpeg 03James Almer 07master:11f3a7ae108a: Merge commit '04f3bd349651694f30feeb8c4ed9bc58106fca54'
[04:39:04 CEST] <cone-906> ffmpeg 03wm4 07master:c2f97f050870: hwcontext_dxva2: support D3D9Ex
[04:39:05 CEST] <cone-906> ffmpeg 03James Almer 07master:596a4cbd0a1b: Merge commit 'c2f97f050870897575570708ac48c5c15e6a0dd8'
[04:42:24 CEST] <cone-906> ffmpeg 03Luca Barbato 07master:0ee78020cd41: configure: Move up the avbuild directory creation
[04:42:25 CEST] <cone-906> ffmpeg 03James Almer 07master:b02b43a823a6: Merge commit '0ee78020cd41d81eec651acd7fc65906207796f3'
[04:50:57 CEST] <cone-906> ffmpeg 03Luca Barbato 07master:ba30b74686f0: aac: Validate the sbr sample rate before using the value
[04:50:58 CEST] <cone-906> ffmpeg 03James Almer 07master:e9a388061371: Merge commit 'ba30b74686f0cb6c9dd465ac4820059c48bf9d08'
[04:53:11 CEST] <cone-906> ffmpeg 03Luca Barbato 07master:9c2d36fcaf87: dv: Convert to the new bitstream reader
[04:53:12 CEST] <cone-906> ffmpeg 03James Almer 07master:a1dcb057e3dc: Merge commit '9c2d36fcaf8748b9baa9aba9264abefce711d67b'
[04:58:55 CEST] <cone-906> ffmpeg 03James Almer 07master:774295a3e01a: doc/libav-merge: mention skipped or incomplete runtime alignment commits
[05:01:37 CEST] <cone-906> ffmpeg 03Martin Storsjö 07master:e18c39005ad1: arm: vp9lpf: Interleave the start of flat8in into the calculation above
[05:01:38 CEST] <cone-906> ffmpeg 03Martin Storsjö 07master:b0806088d3b2: aarch64: vp9lpf: Interleave the start of flat8in into the calculation above
[05:01:39 CEST] <cone-906> ffmpeg 03Martin Storsjö 07master:07b5136c481d: aarch64: vp9lpf: Fix broken indentation/vertical alignment
[05:01:40 CEST] <cone-906> ffmpeg 03James Almer 07master:d987f9cfe370: Merge commit '07b5136c481d394992c7e951967df0cfbb346c0b'
[05:05:10 CEST] <cone-906> ffmpeg 03Mark Thompson 07master:44f2eda39ff5: lavc: Add device context field to AVCodecContext
[05:05:11 CEST] <cone-906> ffmpeg 03James Almer 07master:d7458ca8d7c8: Merge commit '44f2eda39ff55c69d4d739fb12a42a10b7ce581c'
[05:06:28 CEST] <cone-906> ffmpeg 03Mark Thompson 07master:5dd9a4b88b28: vaapi: Implement device-only setup
[05:06:29 CEST] <cone-906> ffmpeg 03James Almer 07master:752bc6b402ae: Merge commit '5dd9a4b88b287bf8c93520afda7becb1ad0d1894'
[05:08:02 CEST] <cone-906> ffmpeg 03Mark Thompson 07master:e791b915c774: hwcontext_vaapi: Try to support the VDPAU wrapper
[05:08:03 CEST] <cone-906> ffmpeg 03James Almer 07master:2838ab65ccf5: Merge commit 'e791b915c774408fbc0ec9e7270b021899e08ccc'
[05:11:53 CEST] <cone-906> ffmpeg 03Luca Barbato 07master:b446f0e98f85: mov: Do not try to parse multiple stsd for the same track
[05:11:54 CEST] <cone-906> ffmpeg 03James Almer 07master:d99c3af7072d: Merge commit 'b446f0e98f85e2e931b476e52b319f1c49244660'
[05:14:14 CEST] <cone-906> ffmpeg 03Diego Biurrun 07master:871b4f365463: configure: Check for xcb as well as xcb-shape before enabling libxcb
[05:14:15 CEST] <cone-906> ffmpeg 03James Almer 07master:e2a5fa11b2a2: Merge commit '871b4f3654636ed64560e86b9faa33828d195ceb'
[05:41:49 CEST] <cone-906> ffmpeg 03Alexandra Hájková 07master:0539d84d985e: asfdec: Account for different Format Data sizes
[05:41:50 CEST] <cone-906> ffmpeg 03James Almer 07master:42f27d1b8eab: Merge commit '0539d84d985e811e5989ef27c13f7e2dda0f9b89'
[05:47:23 CEST] <cone-906> ffmpeg 03Martin Storsjö 07master:030de53e9cc2: libopenh264dec: Let the framework use the h264_mp4toannexb bitstream filter
[05:47:24 CEST] <cone-906> ffmpeg 03James Almer 07master:e9b6212de29a: Merge commit '030de53e9cc225dc767458aedcc87efd457b4f3b'
[05:50:21 CEST] <cone-906> ffmpeg 03James Almer 07master:93dfc4f174d8: avcodec/libopenh264dec: check for ff_set_dimensions() return value
[05:52:58 CEST] <cone-906> ffmpeg 03Mark Thompson 07master:82989bd98c7f: avconv: Move rescale to stream timebase before monotonisation
[05:52:59 CEST] <cone-906> ffmpeg 03James Almer 07master:e862a433d8e3: Merge commit '82989bd98c7f4e87f59af2147b645b8fd8f31c53'
[05:54:29 CEST] <cone-906> ffmpeg 03Martin Storsjö 07master:8847eeaa1418: aarch64: Add parentheses around the offset parameter in movrel
[05:54:30 CEST] <cone-906> ffmpeg 03James Almer 07master:d2ec2e27f819: Merge commit '8847eeaa141898850381400000fb2b8a7adc7100'
[06:06:09 CEST] <cone-906> ffmpeg 03Diego Biurrun 07master:d00a0d8e84fe: configure: Handle SDL version check through pkg-config
[06:06:10 CEST] <cone-906> ffmpeg 03James Almer 07master:47d6b02f6c3d: Merge commit 'd00a0d8e84fef1b9124bfaf71cc17df79ca464a6'
[08:13:08 CEST] <FishPencil> I feel like there's a type in configure, should be "-lmfx" instead of it looking for liblibmfx https://github.com/FFmpeg/FFmpeg/blob/master/configure#L5948
[08:13:17 CEST] <FishPencil> s/type/typo
[10:38:20 CEST] Action: JEEB does the annoying thing of the morning http://ffmpeg.org/pipermail/ffmpeg-devel/2017-September/216816.html , http://ffmpeg.org/pipermail/ffmpeg-devel/2017-September/216699.html
[16:00:43 CEST] <jamrial> michaelni: what do you think of 8f5de34c8f? does it apply to us?
[16:01:50 CEST] <jamrial> our code started to deviate with your commit bca59d7745e, followed by the merge commit aa40df483b2
[16:19:35 CEST] <jamrial> nicolas still thinking he can do whateve the fuck he wants...
[16:20:03 CEST] <wm4> wew
[16:20:13 CEST] <wm4> it's nice that he gives me arguments to easily
[16:20:21 CEST] <wm4> I'll ignore his comments as well in the future
[16:23:44 CEST] <JEEB> evening poke wrt http://ffmpeg.org/pipermail/ffmpeg-devel/2017-September/216816.html , http://ffmpeg.org/pipermail/ffmpeg-devel/2017-September/216699.html
[16:24:27 CEST] <wm4> JEEB: just push them, nobody said no
[16:24:40 CEST] <JEEB> E_NO_RIGHTS
[16:24:54 CEST] <JEEB> (it's funny how people think I have :D)
[16:26:02 CEST] <wm4> did you reject them? everybody and their dog have push rights
[16:26:33 CEST] <wm4> otherwise talk to michaelni (who apparently still manages these things...)
[16:26:57 CEST] <JEEB> I didn't reject them, just was never asked and I thought it was rude to ask for them :D
[16:27:09 CEST] <JEEB> since most people seemed to get asked if they want them
[16:27:09 CEST] <wm4> (I think you've been long enough active in multimedia open source that we can trust you)
[16:29:51 CEST] <jamrial> wm4: to prevent an escalation of the situation with nicolas, send an email explaining why you're against the patch
[16:32:01 CEST] <wm4> if he actually asks for clarification (which he could have done days ago), also I don't know why he wants to remove it, he doesn't state any reason, so it's hard to argue against it other than saying "nope"
[16:33:14 CEST] <jamrial> he will not ask, so please, just send an email explaining why you're against it. don't give him arguments, no matter how small
[16:33:51 CEST] <michaelni> jamrial, the fade commits are from 2013, but i think our fixes where complete and the problematic slice_h FFALIGN is also removed
[16:34:08 CEST] <jamrial> michaelni: ok, will skip then. thanks
[16:35:12 CEST] <michaelni> JEEB, are you in MAINTAINERs ? 
[16:40:49 CEST] <JEEB> michaelni: not that I remember
[16:46:18 CEST] <jamrial> wm4: thanks
[16:49:16 CEST] <jamrial> next time please do this from the get go. don't give nicolas reasons to start flamewars
[16:51:07 CEST] <JEEB> michaelni: so regarding the patches. I added the docs so anything else you'd want from them?
[16:51:16 CEST] <JEEB> second patch contains the FATE tests
[16:52:20 CEST] <wm4> maybe next time nicolas could actually give reasons instead of blaming me for it
[17:05:55 CEST] <michaelni> JEEB, ill look at the patches and ill apply them if i dont spot anything ... about maintainers, do you maintain anything or want to maintain anything ?
[17:06:42 CEST] <michaelni> if yes, please send a patch to add yourself to MAINTAINERS
[17:10:29 CEST] <JEEB> I'm not really stable regarding what I poke... http://git.videolan.org/?p=ffmpeg.git&a=search&h=HEAD&st=author&s=Jan+Ekstr%C3%B6m
[17:10:47 CEST] <JEEB> (and then there's the backports that I post which are of course not by me)
[17:12:03 CEST] <wm4> add yourself to "Developers with git write access who are currently not maintaining any specific part"
[17:15:51 CEST] <JEEB> k
[17:18:34 CEST] <wm4> I didn't know that adding git access was suddenly so public and formal, but I guess it's a good thing
[17:21:48 CEST] <JEEB> yea
[17:25:19 CEST] <kierank> wm4: https://www.thegazette.co.uk/notice/2311020
[17:25:21 CEST] <kierank> needs to be in there
[17:25:23 CEST] <kierank> A PROCLAMATION
[17:31:02 CEST] <wm4> wat
[17:31:34 CEST] <kierank> that's how the queen gives a proclamation
[17:32:05 CEST] <wm4> seems roundabout
[17:55:37 CEST] <cone-230> ffmpeg 03Martin Storsjö 07master:8f5de34c8fb1: vf_fade: Make sure to not miss the last lines of a frame
[17:55:37 CEST] <cone-230> ffmpeg 03James Almer 07master:e63e0b65446b: Merge commit '8f5de34c8fb18fa1416e77d2cb998773a49ddb3d'
[18:08:36 CEST] <cone-230> ffmpeg 03Mark Thompson 07master:17aeee5832b9: vaapi_encode: Discard output buffer if picture submission fails
[18:08:37 CEST] <cone-230> ffmpeg 03Mark Thompson 07master:2d518aec4c78: vf_deinterlace_vaapi: Create filter buffer after context
[18:08:38 CEST] <cone-230> ffmpeg 03James Almer 07master:bf209013bff8: Merge commit '2d518aec4c781316092be65893b47922c8f71b67'
[18:09:40 CEST] <jamrial> jkqxz: do we need 7cb9296db8 from libav?
[18:11:20 CEST] <wm4> we had vaapi vp8 for a while in ffmpeg, so I guess not
[18:11:58 CEST] <jkqxz> It's more fallout from the VP8 hwaccel which never got merged.  Some people hated on that part of the patch and I wasn't really interested in pursuing it further.
[18:13:02 CEST] <jamrial> it wasn't merged?
[18:13:21 CEST] <jkqxz> wm4:  There is no VP8 decode hwaccel in ffmpeg (there is encode).
[18:13:25 CEST] <wm4> ah
[18:13:59 CEST] <jamrial> what do i do then? and shouldn't that be merged at some point?
[18:14:37 CEST] <jkqxz> It should.
[18:14:52 CEST] <jkqxz> Skip the patch now, though.
[18:16:30 CEST] <jamrial> ok, thanks
[18:17:12 CEST] <cone-230> ffmpeg 03Martin Storsjö 07master:07e4be7ec94c: movenc: Add an option for enabling negative CTS offsets
[18:17:13 CEST] <cone-230> ffmpeg 03Martin Storsjö 07master:5455a44aa507: movenc-test: Add tests for negative cts offsets
[18:23:18 CEST] <cone-230> ffmpeg 03Mark Thompson 07master:7cb9296db872: webp: Fix alpha decoding
[18:23:19 CEST] <cone-230> ffmpeg 03James Almer 07master:983705cf137b: Merge commit '7cb9296db872c4221453e5411f242ebcfca62664'
[18:41:14 CEST] <cone-230> ffmpeg 03John Stebbins 07master:42cf7f91f1e9: dv: Don't return EIO upon EOF
[18:41:15 CEST] <cone-230> ffmpeg 03James Almer 07master:6c9f44a3c2cc: Merge commit '42cf7f91f1e9dabf494ff469d8f67ac8b33b0f63'
[18:43:24 CEST] <cone-230> ffmpeg 03Diego Biurrun 07master:00b160af117b: nvenc: Fix nvec vs. nvenc typo
[18:43:25 CEST] <cone-230> ffmpeg 03James Almer 07master:1a4d2e8b4da7: Merge commit '00b160af117b782292619c98effce6c8273792e5'
[18:47:40 CEST] <cone-230> ffmpeg 03Diego Biurrun 07master:54e39b102e29: configure: Explicitly spell out first require_pkg_config() parameter
[18:47:41 CEST] <cone-230> ffmpeg 03James Almer 07master:00a61f30a05a: Merge commit '54e39b102e29adcc2f59f1eca85be5f86c89454b'
[20:20:35 CEST] <BBB> jkqxz: what was the hate?
[20:20:50 CEST] <atomnuker> webp
[20:21:38 CEST] <BBB> . o O ( o m g )
[20:21:42 CEST] <BBB> ok
[20:24:09 CEST] <nevcairiel> all i ever heard about AMF is how horrible it is, do people really want to use that
[20:24:58 CEST] <nevcairiel> (also, the code has serious style issues just from the first sight)
[20:25:32 CEST] <nevcairiel> also, its C++ api with vtbl madness
[20:29:44 CEST] <wm4> seems like some sort of COM-made-portable like decklink (or whatever it was)
[20:30:00 CEST] <jkqxz> Um, these ffmpeg things in AMF are wrappers for parts of ffmpeg?  Sounds like a bad idea to include those bits...
[20:30:58 CEST] <wm4> wut, so it's recursive?
[20:31:01 CEST] <wm4> didn't look too closely
[20:31:02 CEST] <JEEB> lol
[20:31:16 CEST] <jkqxz> '#define FFMPEG_DLL_NAME    L"amf-component-ffmpeg64.dll"'
[20:31:37 CEST] <JEEB> ^_^
[20:32:25 CEST] <wm4> does it work on Linux at all?
[20:33:45 CEST] <jkqxz> I think it's meant to, but it requires the crazy proprietary stack.
[20:33:56 CEST] <jkqxz> (Which I have never run, so I can't confirm that.)
[20:36:22 CEST] <Gramner> that amf patch sure is something
[20:38:03 CEST] <jkqxz> "case $target_os in mingw32*|mingw64*|win32|win64|cygwin*) ;; *) disable amf ;; esac" - for this patch, no.
[20:38:23 CEST] <nevcairiel> we have a similar thing for cuvid
[20:39:43 CEST] <jkqxz> Batch file for testing, nice.
[20:40:34 CEST] <jkqxz> There is D3D11 integration, that's nice.  (Somewhere that libmfx falls over...)
[20:50:25 CEST] <wm4> nevcairiel: I mentioned that in my reply
[22:14:35 CEST] <cone-230> ffmpeg 03Gildas Fargeas 07master:cb8b729180cc: avdevice/decklink_dec: add support for more pixel formats
[22:14:36 CEST] <cone-230> ffmpeg 03Karthick J 07master:e6cdf30fb44f: avdevice/decklink_dec: Added VANC search for all resolutions
[22:14:37 CEST] <cone-230> ffmpeg 03Karthick J 07master:a8755785d7a9: avdevice/decklink_dec: Extraction of luma from V210 VANC modularized
[22:14:38 CEST] <cone-230> ffmpeg 03Karthick J 07master:b6cf66ae1c49: avdevice/decklink_dec: Added Closed caption decode from VANC
[22:14:39 CEST] <cone-230> ffmpeg 03Karthick J 07master:f7b7d51a37dd: avcodec/decode: Pass on the Closed Captions Side Data
[23:33:06 CEST] <BtbN> this AMF patchset
[23:34:27 CEST] <atomnuker> yeah
[23:34:34 CEST] <atomnuker> don't they expose those with vaapi?
[23:36:04 CEST] <BtbN> Isn't AMF Windows-Only?
[23:37:06 CEST] <wm4> on Windows you can just use MediaFoundation
[23:37:31 CEST] <BtbN> does ffmpeg support hw encoding using media foundation?
[23:37:48 CEST] <cone-230> ffmpeg 03Diego Biurrun 07master:7208e5b5d638: configure: Restructure the way check_pkg_config() operates
[23:37:49 CEST] <cone-230> ffmpeg 03James Almer 07master:d81b34069e80: Merge commit '7208e5b5d638d4b9c2784036b4fc5728f32233c7'
[23:37:51 CEST] <wm4> I have a patch
[23:38:10 CEST] <BtbN> I'd guess using AMF gives you more control?
[23:38:42 CEST] <wm4> as a vendor specific API, most likely?
[23:38:57 CEST] <BtbN> as generic media foundation
[23:41:52 CEST] <cone-230> ffmpeg 03Rostislav Pehlivanov 07master:05dfa21d47f3: opus_pvq: make max_den a float
[23:43:40 CEST] <rcombs> jkqxz: thoughts on the new VAAPI FEI thing?
[23:44:37 CEST] <wm4> jkqxz: while we're at it, does this include the magic that would make radeon GL mapping work?
[23:44:47 CEST] <jkqxz> Maybe usable to do stuff like lookahead.  Otherwise mostly only useful for people who want weird shit.
[23:45:06 CEST] <rcombs> mmm, that's what I was hoping
[23:45:06 CEST] <jkqxz> I'm just finishing that off now.
[23:47:21 CEST] <jkqxz> I suppose you mind if the mpv patch doesn't work with libva < 2?
[23:48:36 CEST] <cone-230> ffmpeg 03Pablo Montilla 07master:1015982f45d8: lavf/mov: Allow reading very large files.
[23:49:58 CEST] <wm4> jkqxz: you could just duplicate the implementation as another hwdec_ file
[23:50:43 CEST] <wm4> libva<2 will probably stay around for a longer time
[00:00:00 CEST] --- Fri Sep 29 2017

More information about the Ffmpeg-devel-irc mailing list