Skip to content

Conversation

@petersteneteg
Copy link
Member

No description provided.

@github-actions
Copy link

Cpp-Linter Report ⚠️

Some files did not pass the configured checks!

clang-tidy (v19.1.1) reports: 1 concern(s)
  • tensorvis/tensorvisio/include/inviwo/tensorvisio/processors/tensorfield3dimport.h:43:34: warning: [cppcoreguidelines-special-member-functions]

    class 'TensorField3DImport' defines a default destructor but does not define a copy constructor, a copy assignment operator, a move constructor or a move assignment operator

       43 | class IVW_MODULE_TENSORVISIO_API TensorField3DImport : public Processor {
          |                                  ^

Have any feedback or feature suggestions? Share it here.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cpp-linter Review

Used clang-tidy v19.1.1

Have any feedback or feature suggestions? Share it here.

* <Detailed description from a developer prespective>
*/

class IVW_MODULE_TENSORVISIO_API TensorField3DImport : public Processor {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

clang-tidy diagnostic

tensorvis/tensorvisio/include/inviwo/tensorvisio/processors/tensorfield3dimport.h:43:34: warning: [cppcoreguidelines-special-member-functions]

class 'TensorField3DImport' defines a default destructor but does not define a copy constructor, a copy assignment operator, a move constructor or a move assignment operator

   43 | class IVW_MODULE_TENSORVISIO_API TensorField3DImport : public Processor {
      |                                  ^

@petersteneteg petersteneteg merged commit a385071 into master Nov 21, 2025
17 checks passed
@petersteneteg petersteneteg deleted the fix/tensorvis-tidy branch November 21, 2025 08:19
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.

3 participants