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.

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

[SHELL32] SHBrowseForFolder: Refactor brsfolder.c #6903

Merged
merged 7 commits into from
May 18, 2024

Conversation

katahiromz
Copy link
Contributor

@katahiromz katahiromz commented May 17, 2024

Purpose

dll/win32/shell32/wine/brsfolder.c. This file is too hard to read. Nobody reads.
And WineHQ doesn't show no progress recently in this file. OUT OF DATE!
JIRA issue: CORE-17340

Proposed changes

  • Rewrite some SendMessageW code by using TreeView_... macros.
  • Add BrsFolder_GetDataFromItem helper function and use it.

TODO

  • Do small tests.

@katahiromz katahiromz added refactoring For refactoring changes. shell All PR's related to the shell (and shell extensions) labels May 17, 2024
@katahiromz katahiromz changed the title [SHELL32] brsfolder.c: Refactor [SHELL32] Refactor brsfolder.c May 17, 2024
@katahiromz katahiromz changed the title [SHELL32] Refactor brsfolder.c [SHELL32] SHBrowseForFolder: Refactor brsfolder.c May 17, 2024
@katahiromz
Copy link
Contributor Author

Double checked. Let's go!

@katahiromz katahiromz merged commit ae02934 into reactos:master May 18, 2024
34 checks passed
@katahiromz katahiromz deleted the brsfolder-refactor branch May 18, 2024 11:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactoring For refactoring changes. shell All PR's related to the shell (and shell extensions)
Projects
None yet
1 participant