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

burek burek021 at gmail.com
Mon Mar 20 03:05:02 EET 2017


[00:04:43 CET] <michaelni> commits like b53d8c3ccfeff77874f5ca7c68136b6d87a0a69c document fields that are currently not parsed/used
[00:08:20 CET] <michaelni> for alternative code implementations, i would suggest to use something that can be grep-ed i the commit message wen its removed so it can be easily found again
[00:10:15 CET] <michaelni> no more comments from me, i didnt look at all in depth, some of these are things i didnt touch since a long time
[00:19:36 CET] <ubitux> michaelni: ok, thanks. i'll merge the set tomorrow taking into account what you just said
[03:06:51 CET] <wm4> who did the work to relicense libswscale to lgpl?
[06:48:29 CET] <cone-362> ffmpeg 03Muhammad Faiz 07master:de1308429ae6: swresample/x86/resample: extend resample_double to support avx and fma3
[13:47:04 CET] <cone-890> ffmpeg 03Diego Biurrun 07master:e4d5b5519310: rangecoder: Kill non-compiling disabled cruft
[13:47:04 CET] <cone-890> ffmpeg 03Clément BSsch 07master:3f049646717f: Merge commit 'e4d5b55193109d08be47c42d320334546c006b51'
[13:54:23 CET] <ubitux> 23:59 <michaelni> the code removed from aa37d2bf4505afc106e2a23c44afc722bb204a8e looks cleaner than the remaining code // are you refering to the gray chunk or SWS_X one?
[13:58:18 CET] <michaelni> gray chunk
[14:02:11 CET] <ubitux> ok
[14:02:15 CET] <ubitux> speaking of this
[14:02:23 CET] <ubitux> we have the same case here about isPacked
[14:02:34 CET] <ubitux> (same file)
[14:04:29 CET] <michaelni> i think generic code to detect classes/proerties is better than litteral lists that need to be updated unless unexpectedly thats faster
[14:05:02 CET] <michaelni> quite possibly the generic code gives the wrong result on some corner case though, i dont know/remember
[14:05:07 CET] <ubitux> i agree
[14:07:05 CET] <ubitux> i may try something.
[14:56:00 CET] <ubitux> michaelni: should we consider monob and monow gray?
[15:01:10 CET] <ubitux> michaelni: anyway, patch on the ml
[15:09:21 CET] <ubitux> btw, isPacked is missing uyyvyy411
[15:30:23 CET] <cone-890> ffmpeg 03Diego Biurrun 07master:d5fda00efa75: mpeg4videoenc: Kill non-compiling disabled cruft
[15:30:24 CET] <cone-890> ffmpeg 03Clément BSsch 07master:efcba5a06acf: Merge commit 'd5fda00efa756387cffb4d7294691cd54cfe86cf'
[15:40:12 CET] <cone-890> ffmpeg 03Diego Biurrun 07master:aa37d2bf4505: swscale: Kill non-compiling disabled cruft
[15:40:13 CET] <cone-890> ffmpeg 03Clément BSsch 07master:8e950c9b4235: Merge commit 'aa37d2bf4505afc106e2a23c44afc722bb204a8e'
[15:43:27 CET] <cone-890> ffmpeg 03Diego Biurrun 07master:a972fc1c0ab6: wma: Kill non-compiling disabled cruft
[15:43:28 CET] <cone-890> ffmpeg 03Clément BSsch 07master:18cdef9ab7d7: Merge commit 'a972fc1c0ab6e7f169f9145d6da46e8cedbc291c'
[15:49:05 CET] <cone-890> ffmpeg 03Diego Biurrun 07master:562bec0e6907: pnm_parser: Drop broken disabled cruft
[15:49:06 CET] <cone-890> ffmpeg 03Clément BSsch 07master:8a403b00d1bd: Merge commit '562bec0e690760fb93deb2843a7237713103a191'
[15:49:30 CET] <cone-890> ffmpeg 03Diego Biurrun 07master:dab2034b8679: roqvideoenc: Drop broken disabled cruft
[15:49:31 CET] <cone-890> ffmpeg 03Clément BSsch 07master:4ded6f9b3127: Merge commit 'dab2034b8679aaacd8aef832cdeb71d0ee8a3358'
[15:50:07 CET] <cone-890> ffmpeg 03Diego Biurrun 07master:d9442d13033a: rm: Drop broken disabled cruft
[15:50:08 CET] <cone-890> ffmpeg 03Clément BSsch 07master:3eed90b1ed6c: Merge commit 'd9442d13033a24b14ebae149dcdb42709430e2d9'
[15:52:02 CET] <cone-890> ffmpeg 03Diego Biurrun 07master:263efc095e6c: jfdct: Kill broken cruft
[15:52:03 CET] <cone-890> ffmpeg 03Clément BSsch 07master:92cd2c04b1c2: Merge commit '263efc095e6c7ec2902119118b084cea29ea8916'
[15:55:30 CET] <cone-890> ffmpeg 03Diego Biurrun 07master:42c4c2d2a6dc: aac: Drop broken cruft
[15:55:31 CET] <cone-890> ffmpeg 03Clément BSsch 07master:842e7853c73d: Merge commit '42c4c2d2a6dc48adb0e901ef5617acfba0a3a18e'
[15:57:55 CET] <cone-890> ffmpeg 03Diego Biurrun 07master:b96f0ab3d29c: h264: Kill broken disabled cruft
[15:57:56 CET] <cone-890> ffmpeg 03Clément BSsch 07master:b6e88bf323bb: Merge commit 'b96f0ab3d29cdd9ea9ddabfb2052f72bf8615661'
[15:59:37 CET] <cone-890> ffmpeg 03Diego Biurrun 07master:17cb56b35672: ffv1: Remove broken disabled cruft
[15:59:38 CET] <cone-890> ffmpeg 03Clément BSsch 07master:2f42aef3e40c: Merge commit '17cb56b35672a2cd6ad7abe926e6cc772b8f4710'
[16:00:02 CET] <cone-890> ffmpeg 03Diego Biurrun 07master:a4b1b5aa281c: wc3movie: Drop unused cruft
[16:00:03 CET] <cone-890> ffmpeg 03Clément BSsch 07master:83706367e2b5: Merge commit 'a4b1b5aa281cacde8351d9947b54ccf82ff10cd0'
[16:00:46 CET] <cone-890> ffmpeg 03Diego Biurrun 07master:34c22a9ca656: faan(i)dct: Kill some disabled code
[16:00:47 CET] <cone-890> ffmpeg 03Clément BSsch 07master:7a6514861ecf: Merge commit '34c22a9ca656603428b2c3490d1339c5a5966961'
[16:04:38 CET] <cone-890> ffmpeg 03Diego Biurrun 07master:b53d8c3ccfef: mjpegdec: Drop disabled code
[16:04:39 CET] <cone-890> ffmpeg 03Clément BSsch 07master:1a48a51bfcd0: Merge commit 'b53d8c3ccfeff77874f5ca7c68136b6d87a0a69c'
[16:05:00 CET] <cone-890> ffmpeg 03Diego Biurrun 07master:be3363f664d7: nsv: Drop disabled cruft
[16:05:01 CET] <cone-890> ffmpeg 03Clément BSsch 07master:56d63208d824: Merge commit 'be3363f664d7314d55b42860bd4077154752d769'
[16:06:41 CET] <cone-890> ffmpeg 03Diego Biurrun 07master:be1db21ba88f: mathops: Drop disabled alternative mid_pred() implementation
[16:06:42 CET] <cone-890> ffmpeg 03Clément BSsch 07master:87cd8dc0b0b2: Merge commit 'be1db21ba88fe86036fea9f8d2c1a5f47c2a0a7e'
[16:07:08 CET] <cone-890> ffmpeg 03Diego Biurrun 07master:f2f145f3032b: msmpeg4: Drop disabled debug cruft
[16:07:09 CET] <cone-890> ffmpeg 03Clément BSsch 07master:95a29b1a8205: Merge commit 'f2f145f3032bc8808708a4bd694fbce5f1b8b63c'
[16:07:54 CET] <cone-890> ffmpeg 03Diego Biurrun 07master:0e285c2f9087: mpegvideo: Kill some disabled code
[16:07:55 CET] <cone-890> ffmpeg 03Clément BSsch 07master:adef752f1b18: Merge commit '0e285c2f908789e96e29bfd969ad5eaaa0eece65'
[16:09:05 CET] <cone-890> ffmpeg 03Diego Biurrun 07master:93fed46a92ba: timefilter: test: Drop some disabled debug cruft
[16:09:06 CET] <cone-890> ffmpeg 03Clément BSsch 07master:b2c5f5054b5c: Merge commit '93fed46a92bab8be176d3e67be4354189a8dbe7f'
[16:09:41 CET] <durandal_1707> remove merge cruft, please
[16:11:01 CET] <cone-890> ffmpeg 03Diego Biurrun 07master:7effebde7897: dvbsubdec: Remove disabled, near-duplicate debug code
[16:11:02 CET] <cone-890> ffmpeg 03Clément BSsch 07master:ff66ba6feb3e: Merge commit '7effebde78977fafce935776153ea2f7c0981fa3'
[16:11:33 CET] <cone-890> ffmpeg 03Diego Biurrun 07master:e2b9993558b6: simple_idct: x86: Drop disabled IDCT implementation
[16:11:34 CET] <cone-890> ffmpeg 03Clément BSsch 07master:8695ce73ca89: Merge commit 'e2b9993558b6adee42dcc6eb385a14943aaca974'
[16:12:26 CET] <cone-890> ffmpeg 03Diego Biurrun 07master:014852e932da: simple_idct: arm: Drop disabled code variant
[16:12:27 CET] <cone-890> ffmpeg 03Clément BSsch 07master:a754fae4a7e1: Merge commit '014852e932dab6e9cf2a53e7a17ce8321f3e922c'
[16:13:24 CET] <cone-890> ffmpeg 03Diego Biurrun 07master:83b92a855e8e: golomb: Drop disabled cruft
[16:13:25 CET] <cone-890> ffmpeg 03Clément BSsch 07master:01e188762fc4: Merge commit '83b92a855e8e08bdec484e13ee5a7c8996224772'
[16:17:56 CET] <ubitux> michaelni: ping on f5d46d332258dcd8ca623019ece1d5e5bb74142b; Anton seems to compare rect x+w against bw and rect y+h against bh while your fix seems to compare against respectively against w-i and h-j
[16:18:20 CET] <ubitux> do you think using bw and bh is less correct or not?
[16:19:31 CET] <ubitux> (your fix is in 6ba02602aa7fc7d38db582e75b8b093fb3c1608d)
[16:21:56 CET] <jamrial> that feeling when a git pull shows lines and lines full of red minus signs
[16:47:58 CET] <durandal_1707> ubitux: how many merges are left?
[16:48:21 CET] <ubitux> 930+
[16:48:26 CET] <michaelni> ubitux, maybe both do the same, maybe one breaks some valid files, maybe something else, i dont know
[16:48:36 CET] <ubitux> (920+ actually)
[16:49:04 CET] <ubitux> michaelni: do you prefer that i keep your condition?
[16:50:36 CET] <michaelni> i have no preferrance in this case, backports should stay the simpler though as thats simpler to do
[16:51:19 CET] <ubitux> well, i'd usually merge libav version, but this is a security issue you fixed, so i don't want to do a mistake here
[16:51:39 CET] <ubitux> but OTOH i don't want our code to be more security unsecure than Libav
[16:51:50 CET] <ubitux> -security
[17:30:47 CET] <cone-890> ffmpeg 03Anton Khirnov 07master:f5d46d332258: vmnc: check that subrectangles fit into their containing rectangles
[17:30:48 CET] <cone-890> ffmpeg 03Clément BSsch 07master:1080b7162f2c: Merge commit 'f5d46d332258dcd8ca623019ece1d5e5bb74142b'
[17:31:56 CET] <ubitux> is 796dca027be09334d7bbf4f2ac1200e06bb054cb fixed differently in FFmpeg?
[17:35:09 CET] <ubitux> maybe e11983bda073f8c63f60509ee753da9fba20ed10?
[17:38:44 CET] <cone-890> ffmpeg 03Anton Khirnov 07master:796dca027be0: alac: do not return success if nothing was decoded
[17:38:45 CET] <cone-890> ffmpeg 03Clément BSsch 07master:f09aa73b3050: Merge commit '796dca027be09334d7bbf4f2ac1200e06bb054cb'
[17:40:25 CET] <cone-890> ffmpeg 03Anton Khirnov 07master:bba9d8bdfb20: qpeg: fix an off by 1 error in the MV check
[17:40:26 CET] <cone-890> ffmpeg 03Clément BSsch 07master:a0220d949f2c: Merge commit 'bba9d8bdfb208b0ec2ccf182530347151ee3528b'
[17:50:22 CET] <cone-890> ffmpeg 03Anton Khirnov 07master:409d1cd2c955: cook: use the bytestream2 API for reading extradata
[17:50:23 CET] <cone-890> ffmpeg 03Clément BSsch 07master:d707e667c560: Merge commit '409d1cd2c955485798f8b0b0147c2b899b9144ec'
[17:52:25 CET] <cone-890> ffmpeg 03Anton Khirnov 07master:15ee419b7aba: pcx: properly pad the scanline
[17:52:26 CET] <cone-890> ffmpeg 03Clément BSsch 07master:2da66630dce7: Merge commit '15ee419b7abaf17f8c662c145fe93d3dbf43282b'
[17:56:15 CET] <cone-890> ffmpeg 03Anton Khirnov 07master:221402c1c88b: pcx: check that the packet is large enough before reading the header
[17:56:16 CET] <cone-890> ffmpeg 03Clément BSsch 07master:ca619cdf54ca: Merge commit '221402c1c88b9d12130c6f5834029b535ee0e0c5'
[17:58:33 CET] <ubitux> durandal_1707: opinion on 09b23786b3?
[18:00:30 CET] <ubitux> i should probably noop given 8cd1c0febe88b757e915e9af15559575c21ca728, but you may want to compare
[18:04:11 CET] <durandal_1707> ubitux: what its about?
[18:04:36 CET] <durandal_1707> i cant find that commit hash
[18:04:40 CET] <ubitux> https://git.libav.org/?p=libav.git;a=commitdiff;h=09b23786b3
[18:05:10 CET] <ubitux> our pcx code differs quite a bit, even after that commit
[18:05:31 CET] <ubitux> so i'm tempted to noop it, but you may find issues in our version or improvement to merge
[18:16:55 CET] <durandal_1707> ubitux: nothing interested, our code is nicer, noop it
[18:17:09 CET] <ubitux> alright, thanks :)
[18:17:45 CET] <cone-890> ffmpeg 03Anton Khirnov 07master:09b23786b398: pcx: use the bytestream2 API for reading from input
[18:17:46 CET] <cone-890> ffmpeg 03Clément BSsch 07master:81bc1782b6d0: Merge commit '09b23786b3986502ee88d4907356979127169bdd'
[18:18:17 CET] <cone-890> ffmpeg 03Anton Khirnov 07master:33f10546ec01: vc1: check that slices have a positive height
[18:18:18 CET] <cone-890> ffmpeg 03Clément BSsch 07master:d2f68be1e83d: Merge commit '33f10546ec012ad4e1054b57317885cded7e953e'
[18:20:45 CET] <cone-890> ffmpeg 03Anton Khirnov 07master:6755eb5b2123: mss12: validate display dimensions
[18:20:46 CET] <cone-890> ffmpeg 03Clément BSsch 07master:a45a623d4609: Merge commit '6755eb5b212384e0599f7f2c5de42df49fff57de'
[18:22:25 CET] <cone-890> ffmpeg 03Diego Biurrun 07master:46e3936fb04d: configure: Set __MSVCRT_VERSION__to 0x0700 for MinGW
[18:22:26 CET] <cone-890> ffmpeg 03Clément BSsch 07master:206a9fb29cbc: Merge commit '46e3936fb04d06550151e667357065e3f646da1a'
[18:25:12 CET] <cone-890> ffmpeg 03Luca Barbato 07master:24130234cd9d: rtpdec_mpeg4: validate fmtp fields
[18:25:13 CET] <cone-890> ffmpeg 03Clément BSsch 07master:f4a39ceea0d2: Merge commit '24130234cd9dd733116d17b724ea4c8e12ce097a'
[18:25:59 CET] <ubitux> hey
[18:26:11 CET] <ubitux> what's the policy wrt new codec id from libav?
[18:26:42 CET] <ubitux> adding it in the middle relying on our current offseting is fine, right?
[18:27:06 CET] <ubitux> ah, my bad, they actually have that offsetting as well, forget this
[18:27:32 CET] <Compn> we have to add the hex numbers or so ?
[18:27:41 CET] <Compn> to make sure the codec id dont conflict
[18:29:27 CET] <jamrial> ubitux: we dropped ABI compatibility, so just add it at the end of the corresponding part (video, audio, subtitle, etc)
[18:29:41 CET] <nevcairiel> didnt we drop the offseting
[18:30:17 CET] <nevcairiel> yeah we did
[18:30:25 CET] <jamrial> we probably should on the next bump
[18:30:25 CET] <nevcairiel> so just at the end of the section
[18:30:41 CET] <nevcairiel> we only have the media type offsets like we always had
[18:30:50 CET] <nevcairiel> the fourcc codec ids and the libav offset are gone already
[18:31:26 CET] <jamrial> no, just the fourcc. some of the libav offsets are still there
[18:31:32 CET] <jamrial> look for example the id for PCM_S64LE
[18:32:34 CET] <nevcairiel> oh right it looked like a section offset
[18:33:09 CET] <cone-890> ffmpeg 03Luca Barbato 07master:d42809f9835a: av1: Add codec_id and basic demuxing support
[18:33:10 CET] <cone-890> ffmpeg 03Clément BSsch 07master:ed223eeab34c: Merge commit 'd42809f9835a4e9e5c7c63210abb09ad0ef19cfb'
[18:35:58 CET] <cone-890> ffmpeg 03Luca Barbato 07master:963b3ab11f98: doc: Document FATE option HWACCEL
[18:35:59 CET] <cone-890> ffmpeg 03Clément BSsch 07master:33dc6fcc4c77: Merge commit '963b3ab11f98fcc4a311f0dc7b268890c5675da2'
[18:43:29 CET] <cone-890> ffmpeg 03Diego Biurrun 07master:6892df9294d9: vp3: Change type of stride parameters to ptrdiff_t
[18:43:30 CET] <cone-890> ffmpeg 03Clément BSsch 07master:6a42a54b9de4: Merge commit '6892df9294d93322d43255ada299507465bc93c8'
[18:43:59 CET] <ubitux> i'll stop the merge for an hour or two, feel free to take over
[18:48:42 CET] <jamrial> ubitux: i'll do a few. these ptrdiff_t sound like they could be tedious if they don't apply cleanly
[18:59:27 CET] <ubitux> the first one applied cleanly, we had no extra function nor asm implementation afaict
[19:00:01 CET] <ubitux> ETA: 906 commits
[19:00:51 CET] <jamrial> the next one also applied cleanly
[19:00:59 CET] <jamrial> i'm compiling currently
[19:06:22 CET] <cone-890> ffmpeg 03Diego Biurrun 07master:d9d26a3674f3: vp56: Change type of stride parameters to ptrdiff_t
[19:06:23 CET] <cone-890> ffmpeg 03James Almer 07master:4004d33fcbbb: Merge commit 'd9d26a3674f31f482f54e936fcb382160830877a'
[19:07:56 CET] <jkqxz> Anyone want to comment further on libturing?  I'm inclined to apply the most recent patch now that they've fixed the library to work with a normal install.  (Though still with the crazy boost libraries, they will need to clean that up to get into any distributions.)
[19:09:12 CET] <ubitux> what's the most recent patch?
[19:10:34 CET] <ubitux> the should probably require a minimal version of libturing
[19:10:45 CET] <jkqxz> <https://lists.ffmpeg.org/pipermail/ffmpeg-devel/2017-February/207105.html>
[19:11:38 CET] <jkqxz> I don't think they've heard of version numbering, so that may be tricky.
[19:11:50 CET] <ubitux> lol
[19:12:14 CET] <ubitux> that add_option() is crazy; it should use bprint api
[19:12:51 CET] <ubitux> which call also simplify a lot the caller
[19:13:23 CET] <ubitux> encoder_options should be memset to 0 instead of setting every single field and risking to forget one
[19:13:52 CET] <ubitux> illegal_option could use av_match_name
[19:14:50 CET] <ubitux> final error log should display the error code
[19:15:14 CET] <ubitux> mmh wait, that's not an error code for the lib, so the log should go away
[19:15:37 CET] <jkqxz> Please reply to it if you care.
[19:15:46 CET] <cone-890> ffmpeg 03Diego Biurrun 07master:87c6c78604e4: vp8: Change type of stride parameters to ptrdiff_t
[19:15:47 CET] <cone-890> ffmpeg 03James Almer 07master:e5623aafd8f6: Merge commit '87c6c78604e4dd16f1f45862b27ca006da010527'
[19:17:30 CET] <ubitux> will do, but i'll have to wait a few minutes for mutt to switch to the zomg big dir :p
[19:18:22 CET] <ubitux> they really need some API lessons though
[19:19:22 CET] <jkqxz> I think the whole point of the crazy string stuff is to avoid having to define any real API.
[19:19:47 CET] <cone-890> ffmpeg 03Diego Biurrun 07master:802727b538b4: vp8: Update some assembly comments left unchanged in bd66f073fe7286bd3c
[19:19:48 CET] <cone-890> ffmpeg 03James Almer 07master:4e4dfcac58f3: Merge commit '802727b538b484e3f9d1345bfcc4ab24cfea8898'
[19:25:41 CET] <ubitux> jkqxz: i'll make a review soon, sorry for delaying
[19:25:55 CET] <cone-890> ffmpeg 03Diego Biurrun 07master:f81be06cf614: cavs: Change type of stride parameters to ptrdiff_t
[19:25:56 CET] <cone-890> ffmpeg 03James Almer 07master:aec42ebc27c4: Merge commit 'f81be06cf614919d71ded29b8f595bef40123ad8'
[19:29:16 CET] <jkqxz> I don't mind; thank you for doing anything.  (I was inclined to just give in to attrition.)
[19:34:16 CET] <cone-890> ffmpeg 03Diego Biurrun 07master:3fd22538bc0e: prores: Change type of stride parameters to ptrdiff_t
[19:34:17 CET] <cone-890> ffmpeg 03James Almer 07master:663640d74527: Merge commit '3fd22538bc0e0de84b31335266b4b1577d3d609e'
[21:20:01 CET] <cone-890> ffmpeg 03Diego Biurrun 07master:721d57e608dc: vp56: Separate VP5 and VP6 dsp initialization
[21:20:02 CET] <cone-890> ffmpeg 03James Almer 07master:6966a5e4d79c: Merge commit '721d57e608dc4fd6c86f27c5ae76ef559d646220'
[21:20:14 CET] <jamrial> ^fuck that commit
[21:20:43 CET] <ubitux> :D
[21:21:33 CET] <jamrial> alpha channel was using a different context
[21:26:07 CET] <cone-890> ffmpeg 03Diego Biurrun 07master:4ab496261b12: libvpx: Cast a pointer to const to squelch a warning
[21:26:08 CET] <cone-890> ffmpeg 03James Almer 07master:54b19aaaeb3c: Merge commit '4ab496261b12e20ef293b7adca4fcaef1a67c538'
[21:26:25 CET] <jamrial> ubitux: ok, done
[21:29:14 CET] <ubitux> jamrial: i'll go back to it tomorrow :)
[21:29:17 CET] <ubitux> thanks
[21:29:26 CET] <jamrial> no prob
[21:30:23 CET] <ubitux> ETA: 899 commits
[22:02:03 CET] <cone-890> ffmpeg 03Martin Storsjö 07master:b7a565fe71d1: arm: vp9itxfm: Template the quarter/half idct32 function
[22:02:04 CET] <cone-890> ffmpeg 03Martin Storsjö 07master:70317b25aa35: arm/aarch64: vp9itxfm: Skip loading the min_eob pointer when it won't be used
[22:02:05 CET] <cone-890> ffmpeg 03Martin Storsjö 07master:21c89f3a26bb: arm/aarch64: vp9: Fix vertical alignment
[22:02:06 CET] <cone-890> ffmpeg 03Martin Storsjö 07master:b46d37e93ab1: arm: vp9itxfm16: Use the right lane size
[22:02:07 CET] <cone-890> ffmpeg 03Martin Storsjö 07master:c1619318e540: arm: vp9itxfm16: Fix vertical alignment
[22:02:08 CET] <cone-890> ffmpeg 03Martin Storsjö 07master:32e273c111d8: arm: vp9itxfm16: Avoid reloading the idct32 coefficients
[22:02:09 CET] <cone-890> ffmpeg 03Martin Storsjö 07master:25ced1eb1c6c: aarch64: vp9itxfm16: Fix a typo in a comment
[22:02:10 CET] <cone-890> ffmpeg 03Martin Storsjö 07master:d613251622d6: aarch64: vp9itxfm16: Avoid .irp when it doesn't save any lines
[22:02:11 CET] <cone-890> ffmpeg 03Martin Storsjö 07master:b76533f105cc: aarch64: vp9itxfm16: Restructure the idct32 store macros
[22:02:12 CET] <cone-890> ffmpeg 03Martin Storsjö 07master:0ea603203d1a: arm: vp9itxfm16: Make the larger core transforms standalone functions
[22:02:13 CET] <cone-890> ffmpeg 03Martin Storsjö 07master:0f2705e66b1f: aarch64: vp9itxfm16: Make the larger core transforms standalone functions
[22:02:14 CET] <cone-890> ffmpeg 03Martin Storsjö 07master:d564c9018f8a: aarch64: vp9itxfm16: Move the load_add_store macro out from the itxfm16 pass2 function
[22:02:15 CET] <cone-890> ffmpeg 03Martin Storsjö 07master:eabc5abf949b: arm: vp9itxfm16: Do a simpler half/quarter idct16/idct32 when possible
[22:02:16 CET] <cone-890> ffmpeg 03Martin Storsjö 07master:61b8a9ea2930: aarch64: vp9itxfm16: Do a simpler half/quarter idct16/idct32 when possible
[22:26:05 CET] <ubitux> michaelni: comment on patch 2?
[22:31:15 CET] <ubitux> michaelni: btw, libavutil/pixdesc.c:            if (!strncmp(d->name, "bayer_", 6))
[22:31:17 CET] <ubitux> :p
[22:35:28 CET] <nevcairiel> the bayer pixfmts are a huge hack =p
[22:37:37 CET] <ubitux> is it?
[22:43:06 CET] <jamrial> ubitux: return !!(desc->flags & AV_PIX_FMT_FLAG_BAYER) would be "more correct(tm)"
[22:46:12 CET] <michaelni> ubitux, about #2, no comment from me
[22:46:40 CET] <michaelni> i have no good comment :)
[23:32:24 CET] <ubitux> jamrial: sure, fixed locally
[23:32:32 CET] <ubitux> i'll apply the patchset tmr
[23:43:17 CET] <ubitux> michaelni: can i drop is{RGB,BGR}inBytes? they're unused
[00:00:00 CET] --- Mon Mar 20 2017


More information about the Ffmpeg-devel-irc mailing list