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

Clarify IContextMenu::QueryContextMenu return value #1807

Open
wants to merge 3 commits into
base: docs
Choose a base branch
from

Conversation

whindsaks
Copy link

It is very unhelpful to use 5 for both idCmdFirst and the menu id in the example!

I don't guarantee that my understanding of this calculation is correct. Please ask Raymond Chen to verify if the -5 in the example is supposed to be idCmdFirst or the lowest menu id.

It is very unhelpful to use 5 for both idCmdFirst and the menu id in the example.
Copy link

@whindsaks : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@whindsaks whindsaks changed the title Clarify return value Clarify IContextMenu::QueryContextMenu return value Apr 20, 2024
@whindsaks
Copy link
Author

There should probably also be a note somewhere that some shell extensions just return a count of items it added.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants