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

maybe-uninitialized variable in TestFKDTree #42356

Closed
iarspider opened this issue Jul 25, 2023 · 4 comments
Closed

maybe-uninitialized variable in TestFKDTree #42356

iarspider opened this issue Jul 25, 2023 · 4 comments

Comments

@iarspider
Copy link
Contributor

In CPP20 IBs, the following warnings are emitted:

.../src/CommonTools/RecoAlgos/test/FKDTree_t.cpp: In member function 'test2D':
  .../src/CommonTools/RecoAlgos/test/FKDTree_t.cpp:48:13: warning: 'y' may be used uninitialized [-Wmaybe-uninitialized]
    48 |       float y;
      |             ^
.../src/CommonTools/RecoAlgos/test/FKDTree_t.cpp: In member function 'test3D':
  .../cms/cmssw/CMSSW_13_3_CPP20_X_2023-07-24-1100/src/CommonTools/RecoAlgos/test/FKDTree_t.cpp:98:13: warning: 'y' may be used uninitialized [-Wmaybe-uninitialized]
    98 |       float y;
      |             ^

(this was spotted initially in #31379 (review), but not followed up)

@cmsbuild
Copy link
Contributor

A new Issue was created by @iarspider .

@Dr15Jones, @perrotta, @dpiparo, @rappoccio, @makortel, @smuzaffar can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

@iarspider
Copy link
Contributor Author

assign reconstruction

@cmsbuild
Copy link
Contributor

New categories assigned: reconstruction

@mandrenguyen,@clacaputo you have been requested to review this Pull request/Issue and eventually sign? Thanks

@iarspider
Copy link
Contributor Author

Closed via #42626

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

2 participants