{"push_id":1218270007,"size":9,"distinct_size":9,"ref":"refs/heads/master","head":"46b60a3e72a0343611a204876522e36e7869b998","before":"16d276308acb88c607d832f743d7d75d8bf9e420","commits":[{"sha":"99d9921f39706b90b00d5b97d07ed12c60ea5786","author":{"name":"wm4","email":"2c532deb9ac3ad3ec7b00ece6290c05b79210235@nowhere"},"message":"vo_opengl: glctx can be NULL during init\n\nThis fixes a crash that can happen with the Cocoa backend: it calls\nvo_wakeup() during init, which calls vo_opengl.c/wakeup().\n\nFixes #3360.","distinct":true,"url":"https://api.github.com/repos/Akemi/mpv/commits/99d9921f39706b90b00d5b97d07ed12c60ea5786"},{"sha":"d60db967bda3e9910ec6bc78d0d0cb284f5d68fa","author":{"name":"wm4","email":"2c532deb9ac3ad3ec7b00ece6290c05b79210235@nowhere"},"message":"demux_lavf: remove subtitle seeking special-case\n\nIt used not to work - but now it apparently does. Not sure when that got\nfixed in FFmpeg, but there's no longer a reason to keep this hack.\n\nThis also gets rid of the check for the read_seek2 field, which is not\npart of the public API.","distinct":true,"url":"https://api.github.com/repos/Akemi/mpv/commits/d60db967bda3e9910ec6bc78d0d0cb284f5d68fa"},{"sha":"3623cec7d2753ec7e7bf1ca73580203acb3347c8","author":{"name":"wm4","email":"2c532deb9ac3ad3ec7b00ece6290c05b79210235@nowhere"},"message":"af_lavcac3enc: use common code for AVFrame setup","distinct":true,"url":"https://api.github.com/repos/Akemi/mpv/commits/3623cec7d2753ec7e7bf1ca73580203acb3347c8"},{"sha":"6bb48a10dba13a7bd119bea7522f6810c9c27870","author":{"name":"wm4","email":"2c532deb9ac3ad3ec7b00ece6290c05b79210235@nowhere"},"message":"player: disable DS with spdif transcoding too\n\nOtherwise it behaves dumb. (Although you could argue it shouldn't try to\nguess whether speed changes work, but instead simply disable DS if they\ndon't work.)","distinct":true,"url":"https://api.github.com/repos/Akemi/mpv/commits/6bb48a10dba13a7bd119bea7522f6810c9c27870"},{"sha":"dcfde2934dd7b5401b29d6a604fa3eca1b867d5c","author":{"name":"wm4","email":"2c532deb9ac3ad3ec7b00ece6290c05b79210235@nowhere"},"message":"audio: use idiotic FFmpeg ABI rules for public-except-not-public fields\n\nThe FFmpeg API is incredibly weird and inconsistent about this. This is\nalso a FFmpeg-only issue and nothing like this is in Libav - which\ndoesn't really show FFmpeg in a very positive light.\n\n(To make it even worse: this is a full-blown Libav API incompatibility,\neven though this crap was added for Libav ABI-compatibility. It's\nabsurd.)\n\nQuoting the FFmpeg header for the AVFrame.channels field:\n\n    /**\n     * number of audio channels, only used for audio.\n     * Code outside libavutil should access this field using:\n     * av_frame_get_channels(frame)\n     * - encoding: unused\n     * - decoding: Read by user.\n     */\n    int channels;\n\nIt says \"should\" not must, and it doesn't even mention\nav_frame_set_channels(). It's also in the section for public fields (not\nbelow a marker that indicates private fields in a public struct, like\nit's done e.g. in AVCodecContext).\n\nBut not using the accessor will cause silent failures on ABI changes.\nThe failure that happened due to this code didn't even make it apparent\nwhat was wrong. So just use the idiotic accessor.\n\nAlso harmonize the FFmpeg-cursing in the code. (It's fully justified.)\n\nFixes #3295.\n\nNote that mpv will still check the exact library version numbers, and\nreject mismatches - to protect itself from such issues in the future.","distinct":true,"url":"https://api.github.com/repos/Akemi/mpv/commits/dcfde2934dd7b5401b29d6a604fa3eca1b867d5c"},{"sha":"ebe798a333fe547b77a5c0169a7fd8aadf8b8beb","author":{"name":"Niklas Haas","email":"5bf00c93c9250d44d769b81890fa6584403ae7e0@haasn.xyz"},"message":"vo_opengl: increase 3DLUT accuracy at lower LUT sizes\n\nThis code had the exact same texture indexing bug that the original\nscaler code had before the introduction of the LUT_POS macro to fix it.\n\nWe can re-use this same macro here, and the performance drop is\nvirtually entirely negligible. The benefit is greatly improved LUT\naccuracy as the 3DLUT size decreases - in particular, the old LUT\nstarted introducing more and more black crush the lower your LUT size is\n(because the error was essentially an over-contrast bias, with a\nmagnitude linearly related to the lut size).\n\nThe new code improves black stability as the LUT size decreases, and\nonly at very low values (16 and below) do black levels start noticeably\ngetting affected (due to crude linearization of the nonlinear response\ncurve).\n\nThe default value of 3dlut-size is definitely generous enough for this\nto make no difference out of the box, but it also causes no performance\ndrop at all on my machine so I see no harm in improving the logic.\nFurthermore, this means we could easily decrease the default 3dlut size\nin a future commit, perhaps even down to 64x64x64 as a default. (But\nmore testing is warranted here)","distinct":true,"url":"https://api.github.com/repos/Akemi/mpv/commits/ebe798a333fe547b77a5c0169a7fd8aadf8b8beb"},{"sha":"cb4342dda4389abb2d5c81f7bc660675199a313d","author":{"name":"wm4","email":"2c532deb9ac3ad3ec7b00ece6290c05b79210235@nowhere"},"message":"x11: do not accidentally discard event flags\n\nIt seems vo_x11_check_events() was supposed to return the currently\nflagged events and reset them. But there are many places where\nvo_x11_check_events() is called without checking its return value. This\ncould lead to forgotten events.\n\nChange the code such that they can't get lost.","distinct":true,"url":"https://api.github.com/repos/Akemi/mpv/commits/cb4342dda4389abb2d5c81f7bc660675199a313d"},{"sha":"f127869037f50c1382424e89e0cba5ac3443c094","author":{"name":"Niklas Haas","email":"5bf00c93c9250d44d769b81890fa6584403ae7e0@haasn.xyz"},"message":"vo_opengl: reduce default 3dlut-size to 64x64x64\n\nFollowing testing after ebe798a, this is a more than sufficient size to\ncover our use case.\n\nThe old default was a drop of about 58 dB PSNR using the old code, and\nthis new default is about 65 dB PSNR, so it's actually an improvement\ndespite resulting in a smaller size.\n\nThere was no outlier whatsoever when comparing sizes around the 64\nneighbourhood (with every step corresponding to a PSNR drop of about\n0.07 dB), so I picked this since it's a power of two and requires no\nchange to the current 3dlut-size parsing logic.\n\nI also tested smaller sizes such as 32x32x32 which performed almost as\nwell on colorful samples, but this results in noticeable black boost in\nthe dark regions, which is pretty undesirable. Therefore, we should\navoid going much further below 64x64x64.\n\nEither way, this new size is so fast to compute that the 3dlut cache is\nalmost useless on my end. In fact, it might even be slower to load the\nprofile from the cache than to recompute it from scratch. (For caches on\na disk. For cache on a tmpfs, it makes no difference)","distinct":true,"url":"https://api.github.com/repos/Akemi/mpv/commits/f127869037f50c1382424e89e0cba5ac3443c094"},{"sha":"46b60a3e72a0343611a204876522e36e7869b998","author":{"name":"Niklas Haas","email":"5bf00c93c9250d44d769b81890fa6584403ae7e0@haasn.xyz"},"message":"vo_opengl: remove the 3dlut-size npot2 restriction\n\nThis requires changing the pixel upload alignment because the odd sizes\nmight not be aligned to multiples of 4.\n\nAnyway, the restriction has no real benefit and the sizes in between 32\nand 64 might be worth using, so just drop it.","distinct":true,"url":"https://api.github.com/repos/Akemi/mpv/commits/46b60a3e72a0343611a204876522e36e7869b998"}]}
{"actor":{"display_login":"Akemi"}}