1. 14 Oct, 2016 1 commit
  2. 16 Sep, 2016 1 commit
  3. 15 Sep, 2016 1 commit
  4. 29 Aug, 2016 2 commits
  5. 25 Aug, 2016 2 commits
  6. 04 Aug, 2016 1 commit
  7. 02 Aug, 2016 2 commits
  8. 27 Jul, 2016 1 commit
  9. 26 Jul, 2016 1 commit
  10. 19 Jul, 2016 1 commit
  11. 16 Jun, 2016 1 commit
    • Manuel Grizonnet's avatar
      STYLE: Use ITK_NULLPTR to utilize c++11 features · 0f26e689
      Manuel Grizonnet authored
          Where available, use c++11 features to identify
          potential nullptr usage errors.
      
          python run-clang-tidy.py -clang-tidy-binary ~/software/clang+llvm-3.8.0-x86_64-linux-gnu-ubuntu-14.04/bin/clang-tidy -checks=-*,modernize-use-nullptr -header-filter=/home/grizonnetm/projets/otb/src/OTB/Modules/.*/.*/include/.* -j8 -fix -p ~/projets/otb/bin/release/OTB-clang3.8/ ~/projets/otb/src/OTB/Modules/*/*/src/*.cxx ~/projets/otb/src/OTB/Modules/*/*/app/*.cxx ~/projets/otb/src/OTB/Examples/*/*.cxx ~/projets/otb/src/OTB/Modules/*/*/test/*.cxx > ~/temporary/run-clang-tidy-nullptr-log.txt
      
          find . -type f -print -name "*.h" -o -name "*.txx" -o -name "*.cxx" |xargs perl -pi -e 's/nullptr/ITK_NULLPTR/g'
      0f26e689
  12. 13 Jun, 2016 1 commit
    • Manuel Grizonnet's avatar
      ENH: Remove double underscore in header guards (.txx & __otb*_h form) · 82842d15
      Manuel Grizonnet authored
      Used the following command:
      
      find . \( -iname *.txx -and ! -path *ThirdParty* \) -exec perl -pi -w -e 's/__otb(.*)_txx/otb$1_txx/g;' {} \;
      find . \( -iname *.h -and ! -path *ThirdParty* \) -exec perl -pi -w -e 's/__otb(.*)_h/otb$1_h/g;' {} \;
      
      Fixes many, but not all, clang -Wreserved-id-macro warnings.
      82842d15
  13. 10 Jun, 2016 1 commit
    • Manuel Grizonnet's avatar
      ENH: Explicitly recognize virtual functions · 3de2b346
      Manuel Grizonnet authored
          clang-tidy tool can insert missing override keyword as a macro so that missing [optional] virtual identifiers can be easily added.
      
          build otb with clang 3.8 and use cmake option CMAKE_EXPORT_COMPILE_COMMANDS=ON. Most modules and third parties have been activated.
      
          #Save override occurences in otb code
          cd src/Modules ; grep -nR "override" * > ~/temporary/override_otb.txt
      
          #Run clang modernize check using utility script which allow to process the code in parallel
          python run-clang-tidy.py -clang-tidy-binary ~/software/clang+llvm-3.8.0-x86_64-linux-gnu-ubuntu-14.04/bin/clang-tidy -checks=-*,modernize-use-override -header-filter=/home/grizonnetm/projets/otb/src/OTB/Modules/.*/.*/include/.* -j8 -fix -p ~/projets/otb/bin/release/OTB-clang3.8/ ~/projets/otb/src/OTB/Modules/*/*/src/*.cxx ~/projets/otb/src/OTB/Examples/*/*.cxx ~/projets/otb/src/OTB/Modules/*/*/test/*.cxx > ~/temporary/run-clang-tidy-log.txt
      
          #Replace override by ITK macro (to maintain compat with c++98)
          find . -type f -print -name "*.h" -o -name "*.txx" |xargs perl -pi -e 's/\ override/\ ITK_OVERRIDE/g'
      3de2b346
  14. 26 May, 2016 1 commit
  15. 29 Mar, 2016 1 commit
  16. 15 Feb, 2016 2 commits
  17. 12 Feb, 2016 2 commits
  18. 28 Jan, 2016 1 commit
    • Victor Poughon's avatar
      ENH: Improve MaskedIteratorDecorator tests · 2411ffd7
      Victor Poughon authored
      Factorize forward and reverse test cases to enable more underlying iterator
      types to be tested.
      Remove the wrapped Set() method to support const iterators. Set can still be
      reached with GetImageIterator().
      Also add a missing const in otbSubsampledImageRegionIterator that the new test
      revealed.
      2411ffd7
  19. 04 Jan, 2016 1 commit
    • Rashad Kanavath's avatar
      ENH: apply spelling.patch from DebianGIS · 8cafd175
      Rashad Kanavath authored
      This is a big one without much harm. It fixes incorrect spelling
      and grammer when packaging OTB 5.0.0 and 5.2.0 for Debian.
      Most of these changes are done by developers at DebianGIS.
      For more info, I include the original patch header below.
      Description: Fix some spelling errors to silence lintian
      enabling  to concentrate on the real problem.
      Author: Andreas Tille <tille@debian.org>
      Author: Rashad Kanavath <rashad.kanavath@c-s.fr>
      Author: Bas Couwenberg <sebastic@xs4all.nl>
      Last-Update: 2015-12-22
      8cafd175
  20. 16 Nov, 2015 1 commit
  21. 10 Nov, 2015 1 commit
  22. 30 Oct, 2015 1 commit
    • Manuel Grizonnet's avatar
      BUG: fix several problems in detection of GDAL version · 5999005e
      Manuel Grizonnet authored
      - Move declaration of GDAL_CONFIG_CHECKING in otb-module-init.cmake to ensure gdal configuration in all cases (if statement)
      - OTBCommon module needs to depend on OTBGDAL as GDAL module has to set OTB_USE_GDAL_20 before configuring otbConfigure.h
      - Set mode to STATUS for GDAL version >= to 2.0
      
      All these issues were found compiling gdal trunk (2.1.0dev) with clang 3.5.
      5999005e
  23. 02 Oct, 2015 1 commit
  24. 30 Sep, 2015 1 commit
  25. 24 Sep, 2015 2 commits
  26. 23 Sep, 2015 2 commits
  27. 21 Sep, 2015 4 commits
    • Julien Michel's avatar
      c500e839
    • Julien Michel's avatar
      BUG: Fixing bug #983 (Split map was not correctly estimated when ImageRegion... · b09ad20f
      Julien Michel authored
      BUG: Fixing bug #983 (Split map was not correctly estimated when ImageRegion did not start at [0,0]). Added two tests for this case as well.
      b09ad20f
    • Julien Michel's avatar
      BUG: Fixing bug #983 (Split map was not correctly estimated when ImageRegion... · 3f1903fa
      Julien Michel authored
      BUG: Fixing bug #983 (Split map was not correctly estimated when ImageRegion did not start at [0,0]). Added two tests for this case as well.
      3f1903fa
    • Manuel Grizonnet's avatar
      "BUG: fix missing ITCopyright.txt" · c9a072d0
      Manuel Grizonnet authored
      Following remarks from debian packagers, the ITCopyright.txt was mentioned in
      several files but not available in the Copyright directory.  After contacting
      the contributor Gregoire Mercier, he has suggested to harmonize the copyright for his
      contributions.  Files with GET or IT copyright referenced now the
      IMTCopyright.txt file (Institut Mines Telecom) which was added to the Copyright directory.
      
      Moreover, Gregoire Mercier confirmed that he is not the author of 7 files which
      are mentioning the GET or IT copyright:
      
      Modules/Filtering/Projection/include/otbROIdataConversion.h
      Modules/Filtering/Projection/include/otbROIdataConversion.txx
      Modules/Segmentation/Conversion/test/otbVectorDataRasterizeFilter.cxx
      Modules/Segmentation/Conversion/test/otbVectorDataToLabelImageFilterWithoutReader.cxx
      Modules/Segmentation/Conversion/test/otbPolygonizationRasterizationTest.cxx
      Modules/Segmentation/Conversion/test/otbVectorDataToLabelImageFilter.cxx
      Modules/Segmentation/Conversion/test/otbOGRDataSourceToLabelImageFilter.cxx
      
      Thus, the copyright was also modified for these 7 files (CNES Copyright).
      c9a072d0
  28. 16 Jul, 2015 1 commit
  29. 05 Jun, 2015 2 commits