Skip to content

Remove unused code: CommonCOMDlgs#391

Closed
mark-sil wants to merge 2 commits intorelease/9.3from
cleanup_CommonCOMDlgs
Closed

Remove unused code: CommonCOMDlgs#391
mark-sil wants to merge 2 commits intorelease/9.3from
cleanup_CommonCOMDlgs

Conversation

@mark-sil
Copy link
Contributor

@mark-sil mark-sil commented Jul 8, 2025

Remove the code in Src\ CommonCOMDlgs, which does not appear to
be used. It looks like this was being built for Linux, but not
Windows.


This change is Reviewable

mark-sil added 2 commits June 26, 2025 14:27
Change-Id: I88c3f7c9e7d8c3d9eb24a20a6b773980ab81d4d8
Remove the code in Src\ CommonCOMDlgs, which does not appear to
be used. It looks like this was being built for Linux, but not
Windows.

Change-Id: Id4bf821787c62a02d9a93ff9f1884427980c3f46
@mark-sil mark-sil closed this Jul 8, 2025
@mark-sil
Copy link
Contributor Author

mark-sil commented Jul 8, 2025

This change needs to be applied on top of #380, but what I did seems to be causing a review of all the files in 380 as well as the change in this PR. So, closing this and will try again.

@mark-sil mark-sil deleted the cleanup_CommonCOMDlgs branch August 13, 2025 12:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant