Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Deprecate --glx-swap-method #131

Merged
merged 1 commit into from
Mar 12, 2019
Merged

Deprecate --glx-swap-method #131

merged 1 commit into from
Mar 12, 2019

Conversation

yshui
Copy link
Owner

@yshui yshui commented Mar 12, 2019 •
edited
Loading

  • Change config_libconfig.c to print warning message for option
  • Change options.c to print warning message for option
  • Remove option variable from config.h
  • Remove old usage message from options.c
    • If added a replacement, add new usage message
  • Remove usage info from man/compton.1.asciidoc
    • If added a replacement, add new usage info
  • Remove the use of the deprecated option from compton.sample.conf
    • If added a replacement, add new example

@yshui
Deprecate --glx-swap-method
dd240d0 
Setting glx-swap-method to value other than "undefined" and "buffer-age"
could potentially cause rendering problems. So remove them, the meaning
of the remaining options can be more precisely captured by "use-damage",
so create a new option under that name.

--glx-swap-method is deprecated in favor of the new option --use-damage.

Signed-off-by: Yuxuan Shui <yshuiv7@gmail.com>
@yshui yshui force-pushed the deprecate-glx-swap-method branch from e10a591 to dd240d0 Compare March 12, 2019 00:35
@yshui yshui merged commit cbe22a1 into next Mar 12, 2019
@yshui yshui deleted the deprecate-glx-swap-method branch March 12, 2019 00:37
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@yshui