[00:00] fixing 1299977 is oddly cathartic [00:01] also, how much code should we share between our examples and our production code? I say ideally 'nothing' and the examples should only be based on the public headers === chihchun_afk is now known as chihchun === chihchun is now known as chihchun_afk === chihchun_afk is now known as chihchun === chihchun is now known as chihchun_afk === chihchun_afk is now known as chihchun === chihchun is now known as chihchun_afk === fginther` is now known as fginther === chihchun_afk is now known as chihchun === tedg is now known as ted [13:51] Morning! [13:52] racarr__: Afternoon here [13:53] racarr__: got time to make a couple of trivial fixes to https://code.launchpad.net/~mir-team/mir/cursor-spike-lightning-round-fix-input-area-contains/+merge/220111 - so we can land it? [13:54] anpok_: racarr__ here's a real easy one to review: https://code.launchpad.net/~alan-griffiths/mir/fix-1325602/+merge/221846 [13:56] alan_g: Thanks! as soon as I finish breakfast I will [14:14] dednick: What do you think of: [14:14] s/trusted_participant_starting/participant_added/g [14:14] s/trusted_participant_stopping/participant_removed/g [14:15] alan_g: i have no problem with that. [14:15] * alan_g makes it so === marrusl_ is now known as marrusl === dandrader is now known as dandrader|afk === dandrader|afk is now known as dandrader [15:42] we're in this funny position where we have an api/abi between the production code and examples/ [15:43] my particular problem is that both the demo compositor and the default compositor need a renderer and some occlusion detection [15:44] What's funny about production code having an api/abi and example code using it? [15:45] well, i'd be okay if the api/abi was only the stuff in include/ of course [15:45] but in our examples, we're recommending modifying the production code to suit the purposes of the examples [15:45] That's odd [15:46] particularly mc::filter_occlusions_from and mc::GLRenderer [15:47] * alan_g suspects the answer lies in bzr blame [15:47] well, the answer lies in how we're sharing GLRenderer [15:48] like, its useful because its the default, but do we want to share the code that is useful to the mir defaults in the hope that shell writers can make use of it too? [15:48] I say we don't, if you override a default, you have to write a full alternative [15:50] kdub_: I'm not familiar enough with the current state to know how reasonable it is. (Can one just wrap the default and change things that way?) [15:51] well, one could wrap the default, but we allow overridding various internally-called functions in GLRenderer [15:52] kdub_: I mean through the public api [15:54] right, like take the_gl_renderer() and wrap the default GLRenderer with shadow/frame painting [15:56] My feeling is that we don't have enough use-cases (yet) to drive a correct design, and that we've got [premature] generalization in the wrong places. [15:57] right, and I think the right place to move to overridding is the DisplayBufferCompositor (which just has a bool composite() function) [15:57] instead of Renderer which is funny interface with begin(), render(), end(), set_rotation(), etc [15:58] I just want to figure out if I should try to share mc::filter_occlusions_from with the demo shell, or just write a demo shell occlusion algorithm that doesn't use production code [16:00] I'm not voting on the "correct" answer. I just know the code is hard to adapt every time we try supporting something new. It needs good test cases and a refactor. [16:02] Im voting for decorations and shadows in the scene :p [16:03] kdub_: I don't think it hurts to share that algorithm. [16:03] it doesn't hurt to share, but hurts a bit to expand it to think about fancier decorations :) [16:09] kdub_: you were happy with the previous version (but CI wasn't). Could you check: https://code.launchpad.net/~alan-griffiths/mir/rework-test_client_library.cpp-mk2/+merge/221854 - thanks [16:09] okay [16:20] alan_g: Oh updated lightning-round btw [16:25] racarr__: top approved an hour ago btw [16:35] alan_g: Aha, thanks :) [16:54] anpok_: OK to land https://code.launchpad.net/~alan-griffiths/mir/fix-1325602/+merge/221846? === chihchun is now known as chihchun_afk === alan_g is now known as alan_g|EOD === dandrader is now known as dandrader|lunch [17:41] alan_g|EOD: yes [18:20] [18:28] mwahaha I replaced my keybaord [18:28] with my laptop [18:28] and set up synergy for mouse sharing [18:28] its quite nice [18:28] (+ keybooooooard) [18:28] ...though apparently with some key repeat glitches [18:30] digging in to the emulator bug now...its a little weird... (https://bugs.launchpad.net/ubuntu/+source/android/+bug/1319582) [18:31] Ubuntu bug 1319582 in Mir "emulator: 'Failed to start RenderThread' after opening/closing applications" [High,Triaged] [18:31] I dont know where the fix is but it cant be in mir/qtubuntu, etc, because how are they supposed to clean up [18:32] a clients eglWindowSurface (which in the emulator model has driver/host ssssssssssssssside reeeeeesources) [18:32] ...wow this keyrepeat ttttthing is great [18:34] I mean its like you allocate an eglContext and then crash and nothing leaks, presumably the kernel side component of the driver cleans up any hardware resources [18:34] so this is an emulator driver bug that android has worked around somehow (or we are misusing the emulator) [18:37] Who ever thought "apt-get source android" would be a thing [18:43] failed to fetch...hash sum mismatch?!?! === dandrader|lunch is now known as dandrader === ted is now known as tedg [19:57] typedef SomeSortOfSmartPtr TypePtr is an ugly pattern :p [19:57] I think I tracked down the emulator bug...and have a strong guess thats what lead to it. [20:40] anaybody else with canonical irc connection issues? [20:41] nah. launchpad connection issues though === chihchun_afk is now known as chihchun === Sarvatt_ is now known as Sarvatt === ValicekB_ is now known as ValicekB [23:59] hey there...for anyone wanting to lend a hand in testing [23:59] we're trying to promote our mir 0.2.0