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