Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Regression: Waveform overview seeking is broken in 2.6/main #13946

Closed
fwcd opened this issue Nov 29, 2024 · 4 comments · Fixed by #13947
Closed

Regression: Waveform overview seeking is broken in 2.6/main #13946

fwcd opened this issue Nov 29, 2024 · 4 comments · Fixed by #13947

Comments

@fwcd
Copy link
Member

fwcd commented Nov 29, 2024

Bug Description

Seeking one of the waveforms doesn't seem to work correctly on the latest 2.6/main builds:

Screen.Recording.2024-11-29.at.15.12.20.mov

A quick bisect shows that the issue was introduced somewhere between 437db42 (good) and 0eccb1b (bad) (diff), so very likely here:

cc @Swiftb0y

Version

2.6-alpha-138-g0eccb1b2db (HEAD)

OS

macOS 15.1.1

@Swiftb0y
Copy link
Member

ACK. Thanks for the report. I'll try to find some time to look into it.

@fwcd
Copy link
Member Author

fwcd commented Nov 29, 2024

Did a finer-grained bisect, the issue was introduced somewhere in 3aea4ed.

@ronso0 ronso0 changed the title Regression: Waveform seeking is broken in 2.6/main Regression: Waveform overview seeking is broken in 2.6/main Nov 29, 2024
@Swiftb0y
Copy link
Member

I see, that kinda makes sense. I will double check there.

@Swiftb0y
Copy link
Member

I'm fairly certain I made a copy-paste error here with Right, it should've been None. I'll verify locally and open a PR.

if (directionOption & ControlParameterWidgetConnection::DIR_TO_WIDGET) {
pWidget->addAndSetDisplayConnection(std::move(pConnection),
WBaseWidget::ConnectionSide::Right);
} else {
pWidget->addConnection(std::move(pConnection),
WBaseWidget::ConnectionSide::Right);
}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants