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

cleanup: Return braced initialiser list where possible. #82

Merged
merged 1 commit into from
Jan 8, 2025

Conversation

iphydf
Copy link
Member

@iphydf iphydf commented Nov 18, 2024

run-clang-tidy -p _build -fix $(find . -name "*.h" -or -name "*.cpp" \
  -or -name "*.c" | grep -v "/_build/") \
  -checks="-*,modernize-return-braced-init-list"

This change is Reviewable

@iphydf iphydf added this to the v1.18.0 milestone Nov 18, 2024
@github-actions github-actions bot added the cleanup Internal code cleanup, possibly affecting semantics, e.g. deleting a deprecated feature. label Nov 18, 2024
@iphydf iphydf modified the milestones: v1.18.0, v1.18.x, v1.18.1 Nov 18, 2024
@iphydf iphydf modified the milestones: v1.18.1, v1.18.x Jan 1, 2025
@iphydf iphydf force-pushed the return-braced-init-list branch 2 times, most recently from a8c9592 to ba28476 Compare January 8, 2025 00:03
@iphydf iphydf modified the milestones: v1.18.x, v1.18.2 Jan 8, 2025
@iphydf iphydf force-pushed the return-braced-init-list branch 4 times, most recently from 3d4c1c7 to 93cdc7e Compare January 8, 2025 00:47
```
run-clang-tidy -p _build -fix $(find . -name "*.h" -or -name "*.cpp" \
  -or -name "*.c" | grep -v "/_build/") \
  -checks="-*,modernize-return-braced-init-list"
```
@iphydf iphydf force-pushed the return-braced-init-list branch from 93cdc7e to 5d87f67 Compare January 8, 2025 11:41
@toktok-releaser toktok-releaser merged commit 5d87f67 into TokTok:master Jan 8, 2025
32 checks passed
@iphydf iphydf deleted the return-braced-init-list branch January 8, 2025 11:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cleanup Internal code cleanup, possibly affecting semantics, e.g. deleting a deprecated feature.
Development

Successfully merging this pull request may close these issues.

3 participants