1. 18 Jul, 2018 1 commit
    • 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
  2. 11 Apr, 2018 1 commit
  3. 04 Apr, 2018 1 commit
  4. 03 Apr, 2018 1 commit
    • 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
  5. 12 Mar, 2018 1 commit
  6. 19 Feb, 2018 1 commit
  7. 12 Feb, 2018 1 commit
  8. 14 Dec, 2017 1 commit
  9. 09 Nov, 2017 1 commit
  10. 17 Oct, 2017 1 commit
  11. 04 Sep, 2017 1 commit
  12. 03 May, 2017 1 commit
  13. 02 May, 2017 1 commit
  14. 08 Mar, 2017 2 commits
  15. 27 Jan, 2017 1 commit
  16. 23 Jan, 2017 1 commit
  17. 13 Oct, 2016 2 commits
  18. 23 Sep, 2016 1 commit
  19. 15 Sep, 2016 1 commit
  20. 20 Jul, 2016 1 commit
  21. 12 Jul, 2016 1 commit
  22. 16 Jun, 2016 1 commit
    • 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
  23. 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
  24. 03 Aug, 2015 1 commit
  25. 25 Jun, 2015 1 commit
  26. 28 Apr, 2015 1 commit
  27. 24 Feb, 2015 1 commit
  28. 20 Feb, 2015 1 commit
  29. 18 Feb, 2015 2 commits