Skip to content

Missing include in otbStackIndiceFunctor.h

Cédric Traizet requested to merge missing_include_stackindicefunctor into develop

Summary

IndicesStackFunctor uses itk::VariableLengthVector but the file itkVariableLengthVector.h is not included. Depending on how otbStackIndiceFunctor.h is included this might not cause a compilation error, but in my case when I try to build the tests I get the following error:

[ 50%] Building CXX object Modules/Radiometry/Indices/CMakeFiles/OTBIndicesHeaderTest1.dir/test/OTBIndicesHeaderTest1.cxx.o
In file included from /Datas/Code/OTB/SuperBuild/OTB/build/Modules/Radiometry/Indices/test/OTBIndicesHeaderTest1.cxx:23:0:
/Datas/Code/OTB/otb/Modules/Radiometry/Indices/include/otbIndicesStackFunctor.h:54:22: error: ‘itk’ does not name a type
   using OutputType = itk::VariableLengthVector<typename IndiceType::OutputType>;
                      ^
/Datas/Code/OTB/otb/Modules/Radiometry/Indices/include/otbIndicesStackFunctor.h:75:19: error: ‘OutputType’ has not been declared
   void operator()(OutputType& out, const PixelType& in) const
                   ^
/Datas/Code/OTB/otb/Modules/Radiometry/Indices/include/otbIndicesStackFunctor.h: In member function ‘void otb::Functor::IndicesStackFunctor<TIndice>::operator()(int&, const PixelType&) const’:
/Datas/Code/OTB/otb/Modules/Radiometry/Indices/include/otbIndicesStackFunctor.h:80:14: error: invalid types ‘int[size_t {aka long unsigned int}]’ for array subscript
       out[idx] = (*indice)(in);
              ^

This MR adds the missing header

Copyright

The copyright owner is CNES and has signed the ORFEO ToolBox Contributor License Agreement.


Check before merging:

  • All discussions are resolved
  • At least 2 👍 votes from core developers, no 👎 vote.
  • The feature branch is (reasonably) up-to-date with the base branch
  • Dashboard is green
  • Copyright owner has signed the ORFEO ToolBox Contributor License Agreement
  • Optionally, run git diff develop... -U0 --no-color | clang-format-diff.py -p1 -i on latest changes and commit
Edited by Cédric Traizet

Merge request reports

Loading