Skip to content

Remove vestigial EndSyncMode() in ReadOrResizeOrSignal. Add a Flush() for fps ticks mode start#214

Merged
ldemailly merged 4 commits intomainfrom
cleanup
Jan 19, 2026
Merged

Remove vestigial EndSyncMode() in ReadOrResizeOrSignal. Add a Flush() for fps ticks mode start#214
ldemailly merged 4 commits intomainfrom
cleanup

Conversation

@ldemailly
Copy link
Copy Markdown
Member

@ldemailly ldemailly commented Jan 19, 2026

That EndSyncMode was on the wrong if anyway and also not necessary, likely an remaining issue from the FPS ticks always doing the EndSyncMode too (preventing cursor blinking)

will also avoid needing a flush in https://github.com/fortio/tev/pull/42/files#diff-6cb7e1b4ec6265a9fe71b51395a7ec1e6109e78d180384551a07bed65bd0498eR234

@codecov
Copy link
Copy Markdown

codecov bot commented Jan 19, 2026

Codecov Report

❌ Patch coverage is 0% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 17.23%. Comparing base (9547a7b) to head (4230676).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
ansipixels/ansipixels.go 0.00% 1 Missing ⚠️
brick/cli/brick.go 0.00% 1 Missing ⚠️
brick/cli/replay.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #214   +/-   ##
=======================================
  Coverage   17.23%   17.23%           
=======================================
  Files          26       26           
  Lines        4329     4329           
=======================================
  Hits          746      746           
  Misses       3552     3552           
  Partials       31       31           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ldemailly ldemailly requested a review from geofpwhite January 19, 2026 20:15
@ldemailly ldemailly merged commit 179422e into main Jan 19, 2026
9 checks passed
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.

2 participants