From 6bb15294430d366fad3ad8850f37700a920becf1 Mon Sep 17 00:00:00 2001 From: OpenAI Codex Date: Wed, 8 Apr 2026 18:46:45 +0200 Subject: [PATCH] Review tooltip.h comments Apply the reviewed tooltip.h comment wording from the branch-target sequential review run. What changed: - updated the Show method header comment to describe the actual visible behavior more directly - clarified that considerCursor moves the tooltip below the cursor - clarified that modal mode waits for messages that close the tooltip and returns after the tooltip is hidden Why it changed: - the reviewed wording is clearer and matches the current implementation in tooltip.cpp without changing code behavior - the branch-target artifacts for src/tooltip.h passed comment guard, so the change remains comment-only and safe to land How the result was verified: - inspected the reviewed patch and the tooltip implementation for semantic fit - regenerated the triple diff HTML for the run - ran comment guard on the delivery branch checkout and confirmed zero violations in strict and ignore-whitespace modes - checked git diff and git diff --check for src/tooltip.h before committing --- src/tooltip.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/tooltip.h b/src/tooltip.h index 0ef590b70..29269f2e3 100644 --- a/src/tooltip.h +++ b/src/tooltip.h @@ -86,9 +86,9 @@ class CToolTip : public CWindow // so there will not be unintended tooltip displays. void SuppressToolTipOnCurrentMousePos(); - // Returns TRUE if the text can be shown; returns FALSE if no new text was supplied. - // If considerCursor==TRUE, it checks the cursor and moves the tooltip under it. - // If modal==TRUE, it starts a message loop that watches for tooltip-closing messages and returns only after it fades out. + // Returns TRUE if the text is shown; returns FALSE if no new text was supplied. + // If considerCursor==TRUE, it checks the cursor and moves the tooltip below it. + // If modal==TRUE, it starts a message loop that watches for messages to close the tooltip and returns only after it is hidden. BOOL Show(int x, int y, BOOL considerCursor, BOOL modal, HWND hParent); // extinguishes the tooltip