Skip to content

Conversation

@atsju
Copy link
Owner

@atsju atsju commented Jan 5, 2026

No description provided.

githubdoe and others added 6 commits January 4, 2026 21:26
Co-authored-by: Julien Staub <atsju2@yahoo.fr>
Co-authored-by: Julien Staub <atsju2@yahoo.fr>
Co-authored-by: Julien Staub <atsju2@yahoo.fr>
@atsju atsju changed the title Average profile test of Q_Unused Jan 5, 2026
@github-actions
Copy link

github-actions bot commented Jan 5, 2026

Cpp-Linter Report ⚠️

Some files did not pass the configured checks!

clang-tidy (v18.1.3) reports: 1 concern(s)
  • profileplot.cpp:431:5: warning: [clang-analyzer-security.FloatLoopCounter]

    Variable 'rad' with floating point type 'double' should not be used as a loop counter

      431 |     for (double rad = -1.0; rad < 1.0; rad += steps) {
          |     ^                       ~~~        ~~~
    profileplot.cpp:431:5: note: Variable 'rad' with floating point type 'double' should not be used as a loop counter
      431 |     for (double rad = -1.0; rad < 1.0; rad += steps) {
          |     ^                       ~~~        ~~~

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 v18.1.3

Have any feedback or feature suggestions? Share it here.

for (double rad = -1.; rad < 1.; rad += steps){
int dx, dy;
// Main Sampling Loop
for (double rad = -1.0; rad < 1.0; rad += steps) {
Copy link

Choose a reason for hiding this comment

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

clang-tidy diagnostic

profileplot.cpp:431:5: warning: [clang-analyzer-security.FloatLoopCounter]

Variable 'rad' with floating point type 'double' should not be used as a loop counter

  431 |     for (double rad = -1.0; rad < 1.0; rad += steps) {
      |     ^                       ~~~        ~~~
profileplot.cpp:431:5: note: Variable 'rad' with floating point type 'double' should not be used as a loop counter
  431 |     for (double rad = -1.0; rad < 1.0; rad += steps) {
      |     ^                       ~~~        ~~~

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.

4 participants