- User Since
- Jun 8 2023, 6:28 AM (16 w, 4 d)
Jun 11 2023
Fix broken style in TokenAnnotator.cpp. I don't know how i keep missing these, trying to get better.
Added missing test case for default operator declarations as requested, as well as blank lines between the blocks inside the test functions
as visual separators.
Revamped the formatting logic again and added the requested unit tests.
This revision of the diff changes the names of the new style options to SpaceAfterOperatorOverload and SpaceAfterOperatorCall, which clarifies their scope a lot better i think.
I also rewrote the logic to apply formatting that fits exactly what their name implies respectively,
the new tests i added should cover all common use cases (except i forgot some again).
Jun 9 2023
Jun 8 2023
Clean up the formatting with clang-format.
I had some issues with Git there because i messed up a merge, so the diff was bad. Here's the proper diff.
Some additional changes because i exceeded the maximum line width on multiple occasions.
Applied all suggested changes and added a suiting option for aformentioned edge-case for call expressions.
Also added the missing release notes to the apropriate section.
I took the liberty and extended the unit tests for SpaceAfterTemplateKeyword and SpaceAfterOperator keyword to cover both possible cases, which lead to me discovering that my code was not quite right because of an early-out return path in a calling function.
I removed the redundant anonymous parameter in the unit test source string. Locally all formatting tests passed fine.
I made a stupid mistake and messed up the formatting of the anonymous parameter in the unit test code.
Fixing this right now, don't know why it slipped through.