Skip to content

Make PackMENUITEMINFO: please use dwItemData instead of dwTypeData an actual DeprecationWarning#2595

Merged
Avasam merged 3 commits intomhammond:mainfrom
Avasam:Make-PackMENUITEMINFO--please-use-dwItemData-instead-of-dwTypeData-an-actual-DeprecationWarning
Jun 17, 2025
Merged

Make PackMENUITEMINFO: please use dwItemData instead of dwTypeData an actual DeprecationWarning#2595
Avasam merged 3 commits intomhammond:mainfrom
Avasam:Make-PackMENUITEMINFO--please-use-dwItemData-instead-of-dwTypeData-an-actual-DeprecationWarning

Conversation

@Avasam
Copy link
Copy Markdown
Collaborator

@Avasam Avasam commented Apr 30, 2025

Comment mentions wanting to "nuke" this waaaaay in the past. And it does warn users. But just like for #2503, it wasn't an actual DeprecationWarning.

Also set stacklevel=2 to help point to faulty caller code (see #2594 to help remember setting the stacklevel)

@Avasam Avasam requested a review from mhammond April 30, 2025 17:34
@Avasam Avasam changed the title Make PackMENUITEMINFO: please use dwItemData instead of dwTypeData an actual DeprecationWarning Make PackMENUITEMINFO: please use dwItemData instead of dwTypeData an actual DeprecationWarning Apr 30, 2025
@Avasam Avasam force-pushed the Make-PackMENUITEMINFO--please-use-dwItemData-instead-of-dwTypeData-an-actual-DeprecationWarning branch from 3bd3e4f to 1e3ca4e Compare April 30, 2025 23:08
@Avasam Avasam force-pushed the Make-PackMENUITEMINFO--please-use-dwItemData-instead-of-dwTypeData-an-actual-DeprecationWarning branch from 1e3ca4e to eb4ee66 Compare April 30, 2025 23:09
@Avasam Avasam merged commit 3a21a5a into mhammond:main Jun 17, 2025
30 checks passed
@Avasam Avasam deleted the Make-PackMENUITEMINFO--please-use-dwItemData-instead-of-dwTypeData-an-actual-DeprecationWarning branch June 17, 2025 21:08
@Avasam
Copy link
Copy Markdown
Collaborator Author

Avasam commented Jun 17, 2025

And back under 70 PRs ! Thanks a lot for the steady PR reviews.

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.

2 participants