Issue1129

Title enable more performance-related clang-tidy checks
Priority wish Status resolved
Superseder Nosy List jendrik, malte, silvan
Assigned To jendrik Keywords
Optional summary
Pull request: https://github.com/aibasel/downward/pull/202

Created on 2024-01-07.22:17:35 by jendrik, last changed by jendrik.

Summary
Pull request: https://github.com/aibasel/downward/pull/202
Messages
msg11511 (view) Author: jendrik Date: 2024-01-09.12:05:00
Thanks for the comments! Merged.
msg11505 (view) Author: silvan Date: 2024-01-08.11:32:11
The induced changes look good to me, so I think it makes sense to have the extra checks. Most of the added checks don't induce any changes, so I assume having them doesn't hurt either.
msg11501 (view) Author: jendrik Date: 2024-01-07.22:17:35
I'd like to add some new clang-tidy checks. What are your thoughts on the changes?
History
Date User Action Args
2024-01-09 12:05:00jendriksetstatus: reviewing -> resolved
messages: + msg11511
2024-01-08 11:32:11silvansetnosy: + silvan
messages: + msg11505
2024-01-07 22:17:35jendrikcreate