swh:1:snp:af87cd67498ef4fe47c76ed3e7caffe5b61facaf
Revision 4d4c701fb01b8391344f901252cf46d6173c57f8 authored by Jonas Rembser on 14 May 2022, 10:02:23 UTC, committed by Sergey Linev on 27 September 2022, 06:48:28 UTC
The logic in `TDecompLU` where a separate `bool` was tracking if a given
pointer should be "deleted" caused some compiler errors warnings like:

```
root/math/matrix/src/TDecompLU.cxx: In static member function ‘static Bool_t TDecompLU::InvertLU(TMatrixD&, Double_t, Double_t*)’:
root/math/matrix/src/TDecompLU.cxx:883:17: warning: ‘void operator delete [](void*)’ called on unallocated object ‘workd’ [-Wfree-nonheap-object]
  883 |       delete [] pWorkd;
      |                 ^~~~~~
root/src/root/math/matrix/src/TDecompLU.cxx:847:13: note: declared here
  847 |    Double_t workd[kWorkMax];
```

This commit proposes to rewrite the logic that there is always a
well-defined owning pointer that has to be "deleted" if it's not
`nullptr`.
1 parent a3c94ea
History
Tip revision: 6c9118fb23c981c28a53dc215c68f2be00c04e3e authored by Jonas Rembser on 12 April 2024, 19:22:15 UTC
[RF] Enable `roofit_multiprocess` on the CI
Tip revision: 6c9118f

README.md

back to top