Skip to content

F_CMD_ARGS and F_PASS_ARGS have been replaced by normal parameter list#1287

Open
peripherium wants to merge 1 commit intofvwmorg:mainfrom
peripherium:me/remove-F_PASS_ARGS-and-F_CMD_ARGS
Open

F_CMD_ARGS and F_PASS_ARGS have been replaced by normal parameter list#1287
peripherium wants to merge 1 commit intofvwmorg:mainfrom
peripherium:me/remove-F_PASS_ARGS-and-F_CMD_ARGS

Conversation

@peripherium
Copy link
Copy Markdown
Contributor

  • Expanded F_*_ARGS macros into regular parameter lists, mainly for CMD_* functions.
  • Removed unused parameters that were previously passed via these macros (e.g. bindings_cmd() in bindings.h, menustyle_parse_style() in menustyle.c).
  • Removed variables that only existed to satisfy F_PASS_ARGS (notably in menus.c).
  • Updated the existing trivia comment in fvwm.h to match the current code.

@ThomasAdam ThomasAdam self-assigned this Jan 25, 2026
@ThomasAdam ThomasAdam added this to FVWM3 Jan 25, 2026
@github-project-automation github-project-automation bot moved this to PRs in FVWM3 Jan 25, 2026
@ThomasAdam ThomasAdam added the type:internal-improvement Issue is an internal improvement (cleanup, etc.) label Jan 25, 2026
@ThomasAdam ThomasAdam added this to the 1.1.5 milestone Jan 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:internal-improvement Issue is an internal improvement (cleanup, etc.)

Projects

Status: PRs

Development

Successfully merging this pull request may close these issues.

2 participants