Sorry, not awake. You're right that some attributes will be read only. What I should have said is that visibility is not something that would ever change. At least not in the traditional windowing sense we don't use the visibility flag.
What I didn't realize is that while I vacationed we implemented occlusion notifications, and called it "visibility". That's a misnomer that confused me and we should probably fix. So yes, mir_surface_attrib_visibility would need a getter. Hopefully under a different name.
Sorry, not awake. You're right that some attributes will be read only. What I should have said is that visibility is not something that would ever change. At least not in the traditional windowing sense we don't use the visibility flag.
What I didn't realize is that while I vacationed we implemented occlusion notifications, and called it "visibility". That's a misnomer that confused me and we should probably fix. So yes, mir_surface_ attrib_ visibility would need a getter. Hopefully under a different name.