Skip to content
Snippets Groups Projects
  1. Dec 21, 2018
  2. Jul 18, 2018
    • Victor Poughon's avatar
      REFAC: remove unnecessary calls to c_str · ee124eea
      Victor Poughon authored
      This was done with this script:
      
          #!/bin/bash
      
          set -eou pipefail
      
          # Use C++11 sstream constructor from string
          # Exclude 'ThirdParty' because it uses ossimFilename type
          find Modules/ Examples/ -not -iwholename "*ThirdParty*" -type f -print0 | xargs -0 sed -i -E "s/std::ifstream (.*)\((.*)\.c_str\(\)/std::ifstream \1\(\2/"
          find Modules/ Examples/ -not -iwholename "*ThirdParty*" -type f -print0 | xargs -0 sed -i -E "s/std::ofstream (.*)\((.*)\.c_str\(\)/std::ofstream \1\(\2/"
      
          # Use C++11 ::open(const std::string&) of ifstream, ofstream, etc.
          # Exclude ThirdParty because of ossimFilename type
          find Modules/ Examples/ -not -iwholename "*ThirdParty*" -type f -print0 | xargs -0 sed -i -E "s/\.open\((.*).c_str\(\)/\.open\(\1/"
      
          # No need for c_str when outputing to std::cout and such
          find Modules/ Examples/ -type f -print0 | xargs -0 sed -i -E "s/\.c_str\(\) ?<</ <</"
      
          # Use itk::ExceptionObject::SetDescription and SetLocation string versions
          find Modules/ Examples/ -type f -print0 | xargs -0 sed -i -E "s/e\.SetDescription\((.*)\.c_str\(\)\)/e\.SetDescription\(\1\)/"
          find Modules/ Examples/ -type f -print0 | xargs -0 sed -i -E "s/e\.SetLocation\((.*)\.c_str\(\)\)/e\.SetLocation\(\1\)/"
      
          # Use itksys::SystemTool:: string versions
          find Modules/ Examples/ -type f -print0 | xargs -0 sed -i -E "s/itksys::SystemTools::(.*)\((.*)\.c_str\(\)/itksys::SystemTools::\1\(\2/"
          # Exclude otbLogger because it uses GetCurrentDateTime which does not have a string version
          git checkout --  Modules/Core/Common/src/otbLogger.cxx
          # Exclude test/otbOGRLayerStreamStitchingFilter.cxx because there's a double use which is not compatible with the regex
          git checkout -- Modules/Segmentation/OGRProcessing/test/otbOGRLayerStreamStitchingFilter.cxx
      ee124eea
  3. Apr 11, 2018
  4. Apr 04, 2018
  5. Apr 03, 2018
    • Rashad Kanavath's avatar
      ENH: '|' character in description break otb-qgis · 83b74db5
      Rashad Kanavath authored
      Add a rule to check if parameter key  and description are valid for QGIS
      too. ApplicationEngine must enforce this rule and TestApplication must
      include an additional test case. This can be done as part of refactoring
      efforts in ApplicationEngine
      83b74db5
  6. Mar 12, 2018
  7. Feb 19, 2018
  8. Feb 12, 2018
  9. Dec 14, 2017
  10. Oct 17, 2017
  11. Sep 04, 2017
  12. May 03, 2017
  13. May 02, 2017
  14. Mar 08, 2017
  15. Jan 27, 2017
  16. Jan 23, 2017
  17. Oct 13, 2016
  18. Sep 23, 2016
  19. Sep 15, 2016
  20. Jul 20, 2016
  21. Jun 16, 2016
    • Manuel Grizonnet's avatar
      ENH: Explicitly recognize virtual functions in applications · 29522e41
      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.
      
              #Run clang modernize check using utility script which allow to process the code in parallel (only on Applications directory)
              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/Modules/Applications/.*/include/.* -j8 -fix -p ~/projets/otb/bin/release/OTB-clang3.8/ ~/projets/otb/src/OTB/Modules/Applications/*/app/*.cxx  > ~/temporary/run-clang-tidy-app-log.txt
      
              #Replace override by ITK macro (to maintain compat with c++98) in Modules/Applications
              find . -type f -print -name "*.cxx" |xargs perl -pi -e 's/\ override/\ ITK_OVERRIDE/g'
      29522e41
  22. Jan 04, 2016
    • 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
  23. Aug 03, 2015
  24. Jun 25, 2015
  25. Apr 28, 2015
  26. Feb 20, 2015
  27. Feb 18, 2015
  28. Dec 19, 2014
  29. Nov 05, 2014
  30. Oct 28, 2014
  31. Sep 10, 2014
  32. Sep 01, 2014
  33. Aug 29, 2014
  34. Aug 28, 2014
  35. Aug 26, 2014
  36. Aug 22, 2014
Loading