hetrickcv icon indicating copy to clipboard operation
hetrickcv copied to clipboard

Warnings observed while building Cardinal

Open cosinekitty opened this issue 11 months ago • 0 comments

Hi Michael! Nothing urgent here; I just saw some warnings while building Cardinal this morning as I was preparing to submit my own Sapphire plugin update PR there. I thought you might be interested.

I would be willing to take a whack at resolving some of these. Would you be interested in a PR?

Compiling HetrickCV/src/2To4.cpp
Compiling HetrickCV/src/AnalogToDigital.cpp
Compiling HetrickCV/src/ASR.cpp
Compiling HetrickCV/src/BinaryGate.cpp
Compiling HetrickCV/src/BinaryNoise.cpp
Compiling HetrickCV/src/Bitshift.cpp
Compiling HetrickCV/src/BlankPanel.cpp
Compiling HetrickCV/src/Boolean3.cpp
Compiling HetrickCV/src/Chaos1Op.cpp
In file included from HetrickCV/Gamma/Gamma/gen.h:10,
                 from HetrickCV/Gamma/Gamma/rnd.h:12,
                 from HetrickCV/src/DSP/HCVRandom.h:3,
                 from HetrickCV/src/DSP/HCVChaos.h:4,
                 from HetrickCV/src/Chaos1Op.cpp:2:
HetrickCV/Gamma/Gamma/Types.h: In function ‘gam::Vec<N, T>& gam::operator+=(Vec<N, T>&, const Complex<U>&)’:
HetrickCV/Gamma/Gamma/Types.h:409:9: warning: this ‘for’ clause does not guard... [-Wmisleading-indentation]
  409 |         for(auto i:{0,1}) v[i] op##= c[i]; return v;\
      |         ^~~
HetrickCV/Gamma/Gamma/Types.h:409:9: note: in definition of macro ‘DEF_CVOP’
  409 |         for(auto i:{0,1}) v[i] op##= c[i]; return v;\
      |         ^~~
HetrickCV/Gamma/Gamma/Types.h:409:44: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the ‘for’
  409 |         for(auto i:{0,1}) v[i] op##= c[i]; return v;\
      |                                            ^~~~~~
HetrickCV/Gamma/Gamma/Types.h:409:44: note: in definition of macro ‘DEF_CVOP’
  409 |         for(auto i:{0,1}) v[i] op##= c[i]; return v;\
      |                                            ^~~~~~
HetrickCV/Gamma/Gamma/Types.h: In function ‘gam::Complex<U>& gam::operator+=(Complex<U>&, const Vec<N, T>&)’:
HetrickCV/Gamma/Gamma/Types.h:413:9: warning: this ‘for’ clause does not guard... [-Wmisleading-indentation]
  413 |         for(auto i:{0,1}) c[i] op##= v[i]; return c;\
      |         ^~~
HetrickCV/Gamma/Gamma/Types.h:413:9: note: in definition of macro ‘DEF_CVOP’
  413 |         for(auto i:{0,1}) c[i] op##= v[i]; return c;\
      |         ^~~
HetrickCV/Gamma/Gamma/Types.h:413:44: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the ‘for’
  413 |         for(auto i:{0,1}) c[i] op##= v[i]; return c;\
      |                                            ^~~~~~
HetrickCV/Gamma/Gamma/Types.h:413:44: note: in definition of macro ‘DEF_CVOP’
  413 |         for(auto i:{0,1}) c[i] op##= v[i]; return c;\
      |                                            ^~~~~~
HetrickCV/Gamma/Gamma/Types.h: In function ‘gam::Vec<N, T>& gam::operator-=(Vec<N, T>&, const Complex<U>&)’:
HetrickCV/Gamma/Gamma/Types.h:409:9: warning: this ‘for’ clause does not guard... [-Wmisleading-indentation]
  409 |         for(auto i:{0,1}) v[i] op##= c[i]; return v;\
      |         ^~~
HetrickCV/Gamma/Gamma/Types.h:409:9: note: in definition of macro ‘DEF_CVOP’
  409 |         for(auto i:{0,1}) v[i] op##= c[i]; return v;\
      |         ^~~
HetrickCV/Gamma/Gamma/Types.h:409:44: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the ‘for’
  409 |         for(auto i:{0,1}) v[i] op##= c[i]; return v;\
      |                                            ^~~~~~

... and so on ...

cosinekitty avatar Apr 01 '24 16:04 cosinekitty