Skip to content

Fixed invalid subitem handling in Win32 table tooltips - #3753

Open
fdesbiens wants to merge 1 commit into
eclipse-platform:masterfrom
fdesbiens:fix-win32-table-tooltip-bounds
Open

fdesbiens wants to merge 1 commit into
eclipse-platform:masterfrom
fdesbiens:fix-win32-table-tooltip-bounds

Conversation

@fdesbiens

Copy link
Copy Markdown

Fixes #3752

A Win32 subitem hit test can return a valid row with column -1 for a point outside the table. Tooltip positioning then indexes a per-cell font array with that column.

Rejected invalid hits before acquiring a device context and retained the row-font fallback for invalid font indices. Added Win32 regressions covering zero, one and 22 explicit columns, including the implicit first column and valid tooltip measurement.

Compiled all 651 Win32 Java sources against their matching native libraries. Native Windows Java 21 passed all six new cases. Unmodified code failed all six; reverting either guard failed its three cases. The full Tycho test matrix was not run locally, and the original spontaneous hover is not automated.

Fixes eclipse-platform#3752

A Win32 subitem hit test can return a valid row with column -1 for a point outside the
table. Tooltip positioning then indexes a per-cell font array with that column.

Rejected invalid hits before acquiring a device context and retained the row-font
fallback for invalid font indices. Added Win32 regressions covering zero, one and 22
explicit columns, including the implicit first column and valid tooltip measurement.

Compiled all 651 Win32 Java sources against their matching native libraries. Native
Windows Java 21 passed all six new cases. Unmodified code failed all six; reverting
either guard failed its three cases. The full Tycho test matrix was not run locally, and
the original spontaneous hover is not automated.

Signed-off-by: Frédéric Desbiens <frederic.desbiens@eclipse-foundation.org>
Assisted-by: Codex (gpt-6) <noreply@openai.com>
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.

[Win32] Table tooltip indexes cell font with subitem -1

1 participant