Skip to content

[clang-format] Possible regression clang-format-19? #93678

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

Closed
prj- opened this issue May 29, 2024 · 8 comments
Closed

[clang-format] Possible regression clang-format-19? #93678

prj- opened this issue May 29, 2024 · 8 comments

Comments

@prj-
Copy link

prj- commented May 29, 2024

Could you please let me know which one is correct?

$ clang-format foo.cpp 
struct func_traits_impl<Ret (C::*)(Args...) const> {};
$ clang-format-19 foo.cpp                                                                               
struct func_traits_impl<Ret (C:: *)(Args...) const> {};
$ clang-format-19 --version
Debian clang-format version 19.0.0 (++20240527100630+434ee06d7761-1~exp1~20240527220812.2113)
@owenca owenca added the question A question, not bug report. Check out https://llvm.org/docs/GettingInvolved.html instead! label May 29, 2024
@owenca
Copy link
Contributor

owenca commented May 29, 2024

Version 19 is correct. See #86253.

@owenca owenca closed this as completed May 29, 2024
@prj-
Copy link
Author

prj- commented May 29, 2024

Thank you!

@prj-
Copy link
Author

prj- commented Aug 27, 2024

@owenca, this has changed yet again, so which version is correct, 18 and 20 past 2024-08-25, or 19?

$ clang-format-20 foo.cpp 
struct func_traits_impl<Ret (C::*)(Args...) const> {};
$ clang-format-20 --version
Debian clang-format version 20.0.0 (++20240825100934+5b41eb3a6dcd-1~exp1~20240825221045.2294)
$  clang-format-20 foo.cpp  
struct func_traits_impl<Ret (C:: *)(Args...) const> {};
$ clang-format-20 --version                          
Debian clang-format version 20.0.0 (++20240821111322+a80dd44b0d96-1~exp1~20240821111500.2285)

@owenca owenca added bug Indicates an unexpected problem or unintended behavior and removed question A question, not bug report. Check out https://llvm.org/docs/GettingInvolved.html instead! labels Aug 27, 2024
@llvmbot
Copy link
Member

llvmbot commented Aug 27, 2024

@llvm/issue-subscribers-bug

Author: Pierre Jolivet (prj-)

Could you please let me know which one is correct? ```cpp $ clang-format foo.cpp struct func_traits_impl<Ret (C::*)(Args...) const> {}; $ clang-format-19 foo.cpp struct func_traits_impl<Ret (C:: *)(Args...) const> {}; $ clang-format-19 --version Debian clang-format version 19.0.0 (++20240527100630+434ee06d7761-1~exp1~20240527220812.2113) ```

@owenca
Copy link
Contributor

owenca commented Aug 27, 2024

See #86253 (comment) and the followup commit 714033a, which has been backported to the 19.x Release branch. So the soon-to-be released 19.1.0 and the trunk (20.0.0git) are deemed to be correct right now.

@owenca
Copy link
Contributor

owenca commented Aug 27, 2024

Fixed by 714033a.

@owenca owenca added regression and removed bug Indicates an unexpected problem or unintended behavior labels Aug 27, 2024
@prj-
Copy link
Author

prj- commented Aug 27, 2024

OK, nice, no change in between 18, 19, and 20 then!

@owenca
Copy link
Contributor

owenca commented Aug 27, 2024

Yes, 18 is also correct.

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

No branches or pull requests

3 participants