Skip to content
This repository was archived by the owner on Aug 16, 2026. It is now read-only.

refactor: modularize dwm core structure and extract layouts - #20

Merged
CasperGamingOne merged 2 commits into
CasperGamingOne:my_dwmfrom
DJSnowDogs:testing_1st_commit
Jul 29, 2026
Merged

CasperGamingOne merged 2 commits into
CasperGamingOne:my_dwmfrom
DJSnowDogs:testing_1st_commit

Conversation

@DJSnowDogs

Copy link
Copy Markdown
Contributor
- Create `dwm.h` header containing shared data structures (Client, Monitor, Arg, Button, Key, Layout, Rule), macros, function declarations, and extern global variables.
- Extract layout algorithms (`tile` and `monocle`) from `dwm.c` into a separate `layout.c` module.
- Update `dwm.c` to include `dwm.h` and expose necessary global variables across modules.
- Update `Makefile` build sources (`SRC = drw.c dwm.c layout.c util.c`) and distribution target dependencies.

DJSnowDogs and others added 2 commits July 30, 2026 01:24
    - Create `dwm.h` header containing shared data structures (Client, Monitor,
      Arg, Button, Key, Layout, Rule), macros, function declarations, and extern
      global variables.
    - Extract layout algorithms (`tile` and `monocle`) from `dwm.c` into a separate
      `layout.c` module.
    - Update `dwm.c` to include `dwm.h` and expose necessary global variables across
      modules.
    - Update `Makefile` build sources (`SRC = drw.c dwm.c layout.c util.c`) and
      distribution target dependencies.

Signed-off-by: DJSnowDogs <djsnowdogs@gmail.com>

@CasperGamingOne CasperGamingOne left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thx for contributing for the first time, codebase is a mess anyway that needs modernizing

@CasperGamingOne
CasperGamingOne merged commit 4080dd7 into CasperGamingOne:my_dwm Jul 29, 2026
7 checks passed
@DJSnowDogs
DJSnowDogs deleted the testing_1st_commit branch July 29, 2026 23:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants