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

chore(clang-tidy): remove deprecated/unsupported AnalyzeTemporaryDtors setting #4960

Merged

Conversation

mojomex
Copy link
Contributor

@mojomex mojomex commented Jul 10, 2024

Description

This PR aims to make Clang-Tidy usable in IDEs, e.g. with Clangd as a language server. For VS Code, the default Clangd version is 18, which does not support the AnalyzeTemporaryDtors option anymore, which was deprecated in version 16.

This PR removes the setting which, according to the Clang 16 release notes is no longer in use:

Deprecate the global configuration file option AnalyzeTemporaryDtors, which is no longer in use. The option will be fully removed in clang-tidy version 18.

Related links:

Tests performed

Before the change, Clangd 18 would show errors when parsing .clang-tidy:

E[10:48:30.356] tidy-config error at /home/.../autoware/.clang-tidy:205:0: unknown key 'AnalyzeTemporaryDtors'
E[10:48:30.356] Error parsing clang-tidy configuration in /home/.../autoware/.clang-tidy: Invalid argument

After the change, the errors are gone.

Effects on system behavior

Not applicable.

Interface changes

Not applicable.

Pre-review checklist for the PR author

The PR author must check the checkboxes below when creating the PR.

In-review checklist for the PR reviewers

The PR reviewers must check the checkboxes below before approval.

Post-review checklist for the PR author

The PR author must check the checkboxes below before merging.

  • There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@mojomex mojomex force-pushed the fix/clang-tidy-deprecated-setting branch from 2766bb7 to f23477e Compare July 10, 2024 02:51
@mojomex mojomex self-assigned this Jul 10, 2024
@mitsudome-r
Copy link
Member

NOTE:
The default clang-tidy version in Ubuntu 22.04 is 14, but the default config for AnalyzeTemporaryDtors seems to be false so this modification shouldn't have impact on environments with older versions of clang-tidy as well.

@mitsudome-r mitsudome-r merged commit 240fe86 into autowarefoundation:main Jul 10, 2024
14 checks passed
@mojomex mojomex deleted the fix/clang-tidy-deprecated-setting branch July 10, 2024 05:31
pravinkmr26 pushed a commit to pravinkmr26/autoware that referenced this pull request Jul 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants