Skip to content

[clang-format] Fix alignment in presence of template functions #68029

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Oct 3, 2023
Merged

[clang-format] Fix alignment in presence of template functions #68029

merged 1 commit into from
Oct 3, 2023

Conversation

HazardyKnusperkeks
Copy link
Contributor

@HazardyKnusperkeks HazardyKnusperkeks commented Oct 2, 2023 •
edited
Loading

Fixes #68102.

@HazardyKnusperkeks HazardyKnusperkeks self-assigned this Oct 2, 2023
@owenca
Copy link
Contributor

owenca commented Oct 2, 2023

Can you add an issue to show the incorrect format before this patch?

owenca
owenca reviewed Oct 2, 2023
View reviewed changes
@HazardyKnusperkeks
Copy link
Contributor Author

Can you add an issue to show the incorrect format before this patch?

Done.

@HazardyKnusperkeks HazardyKnusperkeks merged commit 1f01269 into llvm:main Oct 3, 2023
@HazardyKnusperkeks HazardyKnusperkeks deleted the align-template-func branch October 3, 2023 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@owenca owenca owenca approved these changes

@mydeveloperday mydeveloperday mydeveloperday approved these changes

Assignees

@HazardyKnusperkeks HazardyKnusperkeks

Labels
clang-format
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[clang-format] Missing indention of an aligned template function call
3 participants
@HazardyKnusperkeks @owenca @mydeveloperday