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

Allow modules to use fvwm_debug #1144

Merged
merged 2 commits into from
Dec 30, 2024
Merged

Allow modules to use fvwm_debug #1144

merged 2 commits into from
Dec 30, 2024

Conversation

ThomasAdam
Copy link
Member

  • log: expose ability query/set fd
  • modules: convert to use fvwm_debug

@ThomasAdam ThomasAdam added this to the 1.1.2 milestone Dec 18, 2024
@ThomasAdam ThomasAdam added type:enhancement Augmenting an existing feature type:internal-improvement Issue is an internal improvement (cleanup, etc.) labels Dec 18, 2024
@ThomasAdam ThomasAdam changed the title ta/modules use fvwmdebug Allow modules to use fvwm_debug Dec 18, 2024
@ThomasAdam ThomasAdam marked this pull request as draft December 18, 2024 15:16
@ThomasAdam ThomasAdam force-pushed the ta/modules-use-fvwmdebug branch 2 times, most recently from c4e4217 to 66f7f2b Compare December 18, 2024 15:33
@ThomasAdam ThomasAdam added relates:module Issue is in module code and removed type:enhancement Augmenting an existing feature labels Dec 22, 2024
@ThomasAdam ThomasAdam self-assigned this Dec 26, 2024
@ThomasAdam ThomasAdam force-pushed the ta/modules-use-fvwmdebug branch 3 times, most recently from 9db3e23 to 8cc41e2 Compare December 30, 2024 14:08
@ThomasAdam ThomasAdam marked this pull request as ready for review December 30, 2024 14:10
@ThomasAdam
log: expose ability query/set fd
1c15e5b 
This allows for the logging FD to be queried and set -- which is useful
for modules to be able to know the FD for being able to use fvwm_debug()
@ThomasAdam
modules: convert to use fvwm_debug
82a9800
@ThomasAdam ThomasAdam force-pushed the ta/modules-use-fvwmdebug branch from 8cc41e2 to 82a9800 Compare December 30, 2024 14:45
@ThomasAdam ThomasAdam merged commit 8db8fa3 into main Dec 30, 2024
12 checks passed
@ThomasAdam ThomasAdam deleted the ta/modules-use-fvwmdebug branch December 30, 2024 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@ThomasAdam ThomasAdam

Labels
relates:module Issue is in module code type:internal-improvement Issue is an internal improvement (cleanup, etc.)
Projects
FVWM3
Status: Done
Milestone
1.1.2
Development

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