Remove otherwise unused variables which only exist to be passed to GetTextExtent() (PR #26434)

20 views
Skip to first unread message

paulcor

unread,
May 8, 2026, 7:45:01 PMMay 8
to wx-...@googlegroups.com, Subscribed

You can view, comment on, or merge this pull request online at:

  https://github.com/wxWidgets/wxWidgets/pull/26434

Commit Summary

  • 671eb19 Remove otherwise unused variables which only exist to be passed to GetTextExtent()

File Changes

(19 files)

Patch Links:


Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you are subscribed to this thread.Message ID: <wxWidgets/wxWidgets/pull/26434@github.com>

paulcor

unread,
May 8, 2026, 8:55:52 PMMay 8
to wx-...@googlegroups.com, Subscribed

Merged #26434 into master.


Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

You are receiving this because you are subscribed to this thread.Message ID: <wxWidgets/wxWidgets/pull/26434/issue_event/25328400942@github.com>

VZ

unread,
May 9, 2026, 9:55:12 AMMay 9
to wx-...@googlegroups.com, Subscribed
vadz left a comment (wxWidgets/wxWidgets#26434)

FWIW as long as we're modifying these calls, I'd prefer to switch to the overload returning wxSize to avoid ugly output parameters.


Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

You are receiving this because you are subscribed to this thread.Message ID: <wxWidgets/wxWidgets/pull/26434/c4412681445@github.com>

Reply all
Reply to author
Forward
0 new messages