Correct return value of CTaskDialog::OnHelp()

[JoshHarmon]

It returns S_FALSE, not S_OK, according to afxtaskdialog.cpp in 14.42.34433.

In testing, returning S_OK from an overridden implementation to match the described "default" implementation may result in another window (e.g., a main frame window) also presenting help.

[prmerger-automator]

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

[learn-build-service-prod]

Learn Build status updates of commit 629fc83:

✅ Validation status: passed

File	Status	Preview URL	Details
docs/mfc/reference/ctaskdialog-class.md	✅Succeeded

For more details, please refer to the build report.

For any questions, please:

• Try searching the learn.microsoft.com contributor guides
• Post your question in the Learn support channel

[JoshHarmon]

I should note that the fact there is a behavior change based on return value suggests to me that the page for TDN_HELP also is likely to contain bogus claims, such the claim that "The return value is ignored."

[TylerMSFT]

@JoshHarmon, thank you for this fix. I verify the same.
I'll notify the owner of the other doc.
As an aside, my verification process included looking at another topic about the CTaskDialog, which I was able to substantially update. So, your good turn had good knock-on effects.

[TylerMSFT]

#sign-off