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

Delete timeline causes crash on windows #680 #693

Conversation

wolfseifert
Copy link
Contributor

No description provided.

delete-timeline-causes-crash-on-windows-Murmele#680
delete-timeline-causes-crash-on-windows-Murmele#680
@Murmele
Copy link
Owner

Murmele commented Feb 13, 2024

Can you fix the formatting?

@wolfseifert
Copy link
Contributor Author

I made the changes without reformatting, directly at github. I have no idea what is wrong with the formatting. Is the existing code compliant with this formatting check? Maybe you should disable the formatting check for PRs.

@Murmele
Copy link
Owner

Murmele commented Feb 14, 2024

I made the changes without reformatting, directly at github. I have no idea what is wrong with the formatting. Is the existing code compliant with this formatting check? Maybe you should disable the formatting check for PRs.

you have to execute the cl-fmt.sh script. Maybe because the line got longer and then it should be wrapped or something like this.

 ./src/ui/MainWindow.cpp:204:42: error: code should be clang-formatted [-Wclang-format-violations]
  connect(timeline, &QTimeLine::finished, [timeline] { timeline->deleteLater(); });
                                         ^
Failed on file: ./src/ui/MainWindow.cpp

./src/ui/RepoView.cpp:893:42: error: code should be clang-formatted [-Wclang-format-violations]
  connect(timeline, &QTimeLine::finished, [timeline] { timeline->deleteLater(); });
                                         ^
Failed on file: ./src/ui/RepoView.cpp

@wolfseifert
Copy link
Contributor Author

Done.

@Murmele
Copy link
Owner

Murmele commented Feb 14, 2024

Thanks :) i will merge when ci runs through

@Murmele Murmele merged commit d37a32f into Murmele:master Feb 20, 2024
11 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