Skip to content

Diagnostics - Add button to monitor watch expressions#1766

Merged
PabstMirror merged 5 commits intoCBATeam:masterfrom
BrettMayson:debug_watch
Jun 17, 2025
Merged

Diagnostics - Add button to monitor watch expressions#1766
PabstMirror merged 5 commits intoCBATeam:masterfrom
BrettMayson:debug_watch

Conversation

@BrettMayson
Copy link
Copy Markdown
Contributor

When merged this pull request will:
image
image

@veteran29 veteran29 self-requested a review June 15, 2025 20:36

GVAR(watchControls) = [controlNull, controlNull, controlNull, controlNull];
GVAR(watchStatements) = ["","", "", ""];
addMissionEventHandler ["Draw2D", {call FUNC(watchUpdate)}];
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From wiki for Draw2d, "Will stop executing when UI loses focus". Does it stop if game gets unfocused with -noPause? If so I would go for EachFrame.

Copy link
Copy Markdown
Contributor Author

@BrettMayson BrettMayson Jun 16, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Keeps running with noPause on borderless window, stops with exclusive fullscreen

BrettMayson and others added 2 commits June 16, 2025 16:58
Co-authored-by: Filip Maciejewski <veteran29@users.noreply.github.com>
@PabstMirror
Copy link
Copy Markdown
Contributor

they seem to get desynced if you start and stop
e.g. start 1, 2,
stop 2, start 4
ss

@BrettMayson
Copy link
Copy Markdown
Contributor Author

Wasn't able to reproduce that in single or multiplayer, using that order or just randomly quickly toggling them, never had an issue and it worked as expected

@PabstMirror PabstMirror added this to the 3.18.4 milestone Jun 17, 2025
@PabstMirror PabstMirror merged commit e0c4f68 into CBATeam:master Jun 17, 2025
4 checks passed
@BrettMayson BrettMayson deleted the debug_watch branch June 23, 2025 12:02
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.

3 participants