Drop unused Xserver/miext/cw #439
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…. Never used by nxagent (only consumer: XAA). Picked from the below X.org commit: commit 6ed08949af4f7ac09170d3d9581e4092b24a84ee Author: Eric Anholt <[email protected]> Date: Tue Mar 27 17:31:28 2007 -0700 Move libcw setup to the only renderer requiring it (XAA). Additionally, protect libcw setup behind checks for Render, to avoid segfaulting if Render isn't available (xnest). The previous setup was an ABI-preserving dance, which is better nuked now. Now, anything that needs libcw must explicitly initialize it, and miDisableCompositeWrapper (previously only called by EXA and presumably binary drivers) is gone.
This change lets nxagent segfault in fb/fbsolid.c if the Composite extension gets enabled (nxagent +extension Composite :<disp>). However, as Composite support is broken anyway, currently, let's approach Composite support without the deprecated CompositeWrapper. Backported from X.org: commit a44a379a2a4c198dbd6ffd6cc0db376f8fb64327 Author: Adam Jackson <[email protected]> Date: Thu Jul 19 15:00:03 2012 -0400 cw: Remove XAA was the only consumer. Signed-off-by: Adam Jackson <[email protected]> Reviewed-by: Dave Airlie <[email protected]> Reviewed-by: Alan Coopersmith <[email protected]> Reviewed-by: Alex Deucher <[email protected]> Reviewed-by: Daniel Stone <[email protected]> Acked-by: Aaron Plattner <[email protected]> Signed-off-by: Keith Packard <[email protected]> Backported-to-NX-by: Mike Gabriel <[email protected]>
Ionic
reviewed
May 10, 2017
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Xserver/miext/cw: Remove. The nxagent Xserver was never a consumer.
This change lets nxagent segfault in fb/fbsolid.c if the Composite
extension gets enabled (nxagent +extension Composite :).
However, as Composite support is broken anyway, currently, let's
approach Composite support without the deprecated CompositeWrapper.