Cherry Pick Macro fixes from randr-improvements branch#204
Cherry Pick Macro fixes from randr-improvements branch#204einsiedlerspiel wants to merge 2 commits intosharplispers:masterfrom
randr-improvements branch#204Conversation
Required for xrandr improvements, in particular "randr: fix mode-info functions".
|
I would prefer to finish the changes in their original form instead of in the form of a separate, temporary ASDF system that would provide a similar but slightly different interface compared to the "legacy" interface that would remain in clx proper. The reason for holding off with the |
|
Yeah having multiple versions of the API around wouldn't be optimal for an eventual consolidation. I'm not particularly familiar with the StumpWM code base but just did a quick search and it seems they have just like three calls to rr-functions in total. If that's really all it would be fairly trivial to provide them a patch, (that's assuming they don't do anything on their end to fix up buggy return values). Are there any other big project's you'd want to consider? I don't suspect many projects depend on the randr extension in it's current state. I'm kinda surprised stumpwm uses |
This PR cherry picks the changes @scymtym's
randr-improvementsbranch makes to clx macros. The original commits are 63ee5bb and cf46691.I'm looking to separate the fixed up randr extension into it's own asdf-system so it can be used with quicklisp versions of clx. This is mostly unproblematic because its changes are mostly in a separate name space already, but it does depend on these fixes to clx's macros. Merging them into master would be helpful.
This would also be a good first step to porting the fixes from that branch to master in a non-breaking manner.
The caveat is that I don't know how this affects other parts of clx and whether the changes were relevant to the down stream breakage the branch caused (#197) , though i don't think it's likely. Maybe @scymtym can help with that.