04:42d_ed[m]: >hmm, is mpv's resize behavior not technically a protocol error?
04:42d_ed[m]: See: https://gitlab.freedesktop.org/wayland/wayland-protocols/-/issues/33
04:42d_ed[m]: ifreund: ^
08:26pq: JoshuaAshton, that's kind of the whole problem: scRGB *cannot* be used without more information, either explicit or assumed. As a protocol designer I cannot assume, so I require explicit, and leave the guessing for the apps.
08:31JEEB: yea so the content color volume should be known by the app. for example if I'm trying to pass Display P3 wide gamut SDR as scRGB, then I should be able to flag that and then the compositor can decide how to show that content
08:43ifreund: d_ed[m]: thanks for the link
11:27emersion: daniels, feel free to merge !309 if you like it
11:37daniels: emersion: cool, ta
11:41wlb: wayland/main: Joshua Ashton * event-loop: Handle EINTR and EAGAIN in wl_event_loop_dispatch https://gitlab.freedesktop.org/wayland/wayland/commit/3bac2e5fb8ee src/event-loop.c
11:41wlb: wayland Merge request !309 merged \o/ (event-loop: Handle EINTR and EAGAIN in wl_event_loop_dispatch https://gitlab.freedesktop.org/wayland/wayland/-/merge_requests/309)
12:55wlb: weston Merge request !1230 opened by Daniel Stone (daniels) Draft: wet_process: Fully split wet_process and wl_client https://gitlab.freedesktop.org/wayland/weston/-/merge_requests/1230
13:11pq: swick[m], emersion, as we are talking about the H.273 enums, and putting those in XML, one more aspect comes to mind: headers generated from XML are hard to use in library public API. Would you need to define an identical enum anyway just to expose public API?
13:11pq: do you have such libraries?
13:12emersion: pq, btw the kernel patch is exactly the same situation: defines for values in the specs
13:12pq: emersion, but it's not generated, and the names are unambiguous.
13:13emersion: not sure what difference that makes…
13:13emersion: the generated header issue is annoying to handle
13:13emersion: historically wlroots just expects compositors to run the generator and include the file before wlr headers
13:13emersion: which is … not nice
13:15emersion: if we had widely available headers with the defines, it would be easier
13:15pq: Most H.273 enum values list several standards. Which one to use for the name is somewhat arbitrary. It could be confusing for someone implementing the-other-spec, why is it using seemingly wrong value.
13:18pq: If I need to invent my own enum in libweston, I can use it for both internals and public API, and I can choose the naming most appropriate for Weston.
13:18pq: of course, it doesn't hurt if XML defined another enum I just don't use
13:21pq: so I suppose the arguments of "generated headers are annoying" and "which name to pick" are moot, then?
13:21pq: in that case, we can certainly have those enums in XML
13:22pq: ...with the caveat that also values outside of the enum are allowed, which means the argument is not actually an enum
13:22pq: which I think language bindings people don't like?
13:23pq: maybe language bindings would just ignore that XML enum too
13:26pq: as long as we rely on H.273 for the value definitions, I don't really care
14:18swick[m]: honestly, I really dislike having potentially non-exhausting enums... but if people really want it then I won't stop it
14:18swick[m]: we can add those enums at any point in the future though so maybe try without them first?
14:21emersion: wayland enums are non-exhaustive, and we can't change that'
14:29wlb: weston Merge request !1231 opened by Simon Ser (emersion) clients/scaler: check viewporter availability https://gitlab.freedesktop.org/wayland/weston/-/merge_requests/1231 [Clients]
14:29swick[m]: usually if you send values which are not covered by an enum that results in a protocol error. having both the enum and the CICP spec to choose values from is just adding complexity.
14:30emersion: "usually", it's totally manual
14:30emersion: and there are cases where it's not an error
14:31emersion: e.g. wl_shm formats
14:31kchibisov: Given that you can add enums to wl_shm without bumping any version you have non-exhaustive enums handling in every library around wayland protocol.
14:31emersion: the issue with not having the enum is that then users need to hardcode constants
14:32emersion: and then we end up with https://github.com/ValveSoftware/gamescope/blob/master/src/drm.cpp#L400
14:33wlb: weston Merge request !1229 merged \o/ (Integrate libdisplay-info (optional at first) https://gitlab.freedesktop.org/wayland/weston/-/merge_requests/1229)
14:33wlb: weston/main: Marius Vlad * 8 commits https://gitlab.freedesktop.org/wayland/weston/compare/8c9dd4febbe129a29d045974a89eec8e28bd3418...8ca6c24c8fa08651ef75ff37f28677de0e01553d
14:33emersion: noice
14:34swick[m]: \o/
14:35swick[m]: the other issue was naming all the crap
14:35swick[m]: so if someone makes something that's not horrible... sure. I don't want to do that though.
14:36emersion: ci failing, i don't think it's my fault…
14:45mvlad: emersion, nope, it's some networking stuff.
15:07wlb: weston/main: Simon Ser * clients/scaler: check viewporter availability https://gitlab.freedesktop.org/wayland/weston/commit/fe495344882a clients/scaler.c
15:07wlb: weston Merge request !1231 merged \o/ (clients/scaler: check viewporter availability https://gitlab.freedesktop.org/wayland/weston/-/merge_requests/1231)
15:30daniels: emersion: I hear you wrt including extension enums in compositor headers - having the same pain now Weston does tablets
15:30emersion: yeah ;_;
15:31emersion: there's an old wlr thread listing all of the options and none are good
15:31daniels: w-p generating headers … ?
15:32daniels: like, installed alongside XML - at least just for the enumerated definitions and not the marshalling?
18:07emersion: pq: when color-repr is not used, what are good defaults?
18:08emersion: midpoint or cosited? REC.601 or REC.709? full or limited range?
19:32swick[m]: defo full range for rgb and limited range for ycc. siting has almost no effect and rec.601 is probably fine
20:04wlb: wayland Issue #377 opened by shrekni (shrekni5) Some games (specially source and tf2c) dont run under wayland https://gitlab.freedesktop.org/wayland/wayland/-/issues/377
20:06LaserEyess: 🤔