1. 12 Sep, 2016 1 commit
  2. 06 Sep, 2016 1 commit
  3. 03 Sep, 2016 1 commit
  4. 27 Aug, 2016 1 commit
  5. 24 Aug, 2016 1 commit
    • Brad King's avatar
      Fortran: Fix .mod file comparison for Intel 16 format · f699323a
      Brad King authored
      The Intel 16 format starts with the 0x0A 0x00 sequence that we use to
      skip past the timestamp.  This occurrence appears to be a version
      number.  Skip the first byte to avoid matching the sequence early.
      
      Ideally we should gain a better understanding of the format and avoid
      depending on short sequences that are likely to appear early by
      coincidence, but this approach will suffice for now.
      
      Closes: #16263
      f699323a
  6. 18 Aug, 2016 1 commit
  7. 16 Aug, 2016 1 commit
  8. 28 Jun, 2016 1 commit
  9. 14 Jun, 2016 1 commit
  10. 02 Jun, 2016 1 commit
    • Daniel Pfeifer's avatar
      Simplify boolean expressions · 7f6b8d33
      Daniel Pfeifer authored and Brad King's avatar Brad King committed
      Use clang-tidy's readability-simplify-boolean-expr checker.
      After applying the fix-its, revise all changes *very* carefully.
      Be aware of false positives and invalid changes.
      7f6b8d33
  11. 25 May, 2016 1 commit
    • Daniel Pfeifer's avatar
      Use enums defined in cmOutputConverter using their fully qualified name. · 25d1ef64
      Daniel Pfeifer authored and Brad King's avatar Brad King committed
      Mostly automated:
      
      values=("RelativeRoot" "NONE" "FULL" "HOME" "START" "HOME_OUTPUT" "START_OUTPUT"
              "OutputFormat" "UNCHANGED" "MAKERULE" "SHELL" "WATCOMQUOTE" "RESPONSE"
              "FortranFormat" "FortranFormatNone" "FortranFormatFixed" "FortranFormatFree")
      for i in "${values[@]}"; do git grep -l cmLocalGenerator::$i | xargs sed -i "s|cmLocalGenerator::$i|cmOutputConverter::$i|g"; done
      25d1ef64
  12. 24 May, 2016 1 commit
  13. 16 May, 2016 1 commit
    • Kitware Robot's avatar
      Revise C++ coding style using clang-format · d9fd2f54
      Kitware Robot authored and Brad King's avatar Brad King committed
      Run the `Utilities/Scripts/clang-format.bash` script to update
      all our C++ code to a new style defined by `.clang-format`.
      Use `clang-format` version 3.8.
      
      * If you reached this commit for a line in `git blame`, re-run the blame
        operation starting at the parent of this commit to see older history
        for the content.
      
      * See the parent commit for instructions to rebase a change across this
        style transition commit.
      d9fd2f54
  14. 09 May, 2016 1 commit
    • Brad King's avatar
      Remove `//------...` horizontal separator comments · 0ac18d40
      Brad King authored
      Modern editors provide plenty of ways to visually separate functions.
      Drop the explicit comments that previously served this purpose.
      Use the following command to automate the change:
      
          $ git ls-files -z -- \
              "*.c" "*.cc" "*.cpp" "*.cxx" "*.h" "*.hh" "*.hpp" "*.hxx" |
            egrep -z -v "^Source/cmCommandArgumentLexer\." |
            egrep -z -v "^Source/cmCommandArgumentParser(\.y|\.cxx|Tokens\.h)" |
            egrep -z -v "^Source/cmDependsJavaLexer\." |
            egrep -z -v "^Source/cmDependsJavaParser(\.y|\.cxx|Tokens\.h)" |
            egrep -z -v "^Source/cmExprLexer\." |
            egrep -z -v "^Source/cmExprParser(\.y|\.cxx|Tokens\.h)" |
            egrep -z -v "^Source/cmFortranLexer\." |
            egrep -z -v "^Source/cmFortranParser(\.y|\.cxx|Tokens\.h)" |
            egrep -z -v "^Source/cmListFileLexer\." |
            egrep -z -v "^Source/cm_sha2" |
            egrep -z -v "^Source/(kwsys|CursesDialog/form)/" |
            egrep -z -v "^Utilities/(KW|cm).*/" |
            xargs -0 sed -i '/^\(\/\/---*\|\/\*---*\*\/\)$/ {d;}'
      
      This avoids modifying third-party sources and generated sources.
      0ac18d40
  15. 29 Apr, 2016 1 commit
    • Brad King's avatar
      Format include directive blocks and ordering with clang-format · e1c77472
      Brad King authored
      
      
      Sort include directives within each block (separated by a blank line) in
      lexicographic order (except to prioritize `sys/types.h` first).  First
      run `clang-format` with the config file:
      
          ---
          SortIncludes: false
          ...
      
      Commit the result temporarily.  Then run `clang-format` again with:
      
          ---
          SortIncludes: true
          IncludeCategories:
            - Regex:    'sys/types.h'
              Priority: -1
          ...
      
      Commit the result temporarily.  Start a new branch and cherry-pick the
      second commit.  Manually resolve conflicts to preserve indentation of
      re-ordered includes.  This cleans up the include ordering without
      changing any other style.
      
      Use the following command to run `clang-format`:
      
          $ git ls-files -z -- \
              '*.c' '*.cc' '*.cpp' '*.cxx' '*.h' '*.hh' '*.hpp' '*.hxx' |
            egrep -z -v '(Lexer|Parser|ParserHelper)\.' |
            egrep -z -v '^Source/cm_sha2' |
            egrep -z -v '^Source/(kwsys|CursesDialog/form)/' |
            egrep -z -v '^Utilities/(KW|cm).*/' |
            egrep -z -v '^Tests/Module/GenerateExportHeader' |
            egrep -z -v '^Tests/RunCMake/CommandLine/cmake_depends/test_UTF-16LE.h' |
            xargs -0 clang-format -i
      
      This selects source files that do not come from a third-party.
      Inspired-by: Daniel Pfeifer's avatarDaniel Pfeifer <daniel@pfeifer-mail.de>
      e1c77472
  16. 10 Oct, 2015 1 commit
  17. 29 Jul, 2015 1 commit
  18. 27 Jul, 2015 4 commits
  19. 06 Jul, 2015 1 commit
  20. 20 Apr, 2015 1 commit
  21. 11 Jan, 2015 1 commit
  22. 15 Oct, 2014 1 commit
  23. 22 Jul, 2014 2 commits
    • Brad King's avatar
      cmDepends: Refactor object file path conversion · 85cea8a7
      Brad King authored
      Delay conversion of the path to object files (on the left-hand side
      of dependencies) until just before they are written.  Also do not
      convert the format of paths written to the 'depend.internal' file.
      This is consistent with the way the right-hand side of dependencies
      are already handled.
      85cea8a7
    • Brad King's avatar
      cmLocalGenerator: Rename 'MAKEFILE' to 'MAKERULE' · 9f92a78b
      Brad King authored
      Rename the internal enumeration value for converting paths destined
      for use in Makefile rule syntax.
      9f92a78b
  24. 23 Jun, 2014 1 commit
    • Brad King's avatar
      Fortran: Add support for GNU >= 4.9 compressed modules (#14975) · d90be200
      Brad King authored
      From the GCC 4.9 release notes for Fortran:
      
       https://gcc.gnu.org/gcc-4.9/changes.html
       "Module files: The version of the module files (.mod) has been
        incremented; additionally, module files are now compressed."
      
      Teach cmDependsFortran::ModulesDiffer to look for the gzip magic numbers
      at the beginning of the module file.  If found, assume the module was
      produced by gfortran >= 4.9.  The modules do not appear to contain the
      date as earlier versions did so we can compare the content directly
      and do not actually need to decompress.
      d90be200
  25. 11 Mar, 2014 2 commits
  26. 08 Mar, 2014 1 commit
    • Ben Boeckel's avatar
      strings: Remove cmStdString references · 270eb96d
      Ben Boeckel authored and Brad King's avatar Brad King committed
      Casts from std::string -> cmStdString were high on the list of things
      taking up time. Avoid such implicit casts across function calls by just
      using std::string everywhere.
      
      The comment that the symbol name is too long is no longer relevant since
      modern debuggers alias the templates anyways and the size is a
      non-issue since the underlying methods are generated since it's
      inherited.
      270eb96d
  27. 07 Jan, 2014 1 commit
  28. 06 Nov, 2012 1 commit
    • Alexander Neundorf's avatar
      cmDepends: allow multiple dependees per depender · e74ff7c2
      Alexander Neundorf authored and Brad King's avatar Brad King committed
      
      
      This patch is heavily inspired by Michael Wild.
      
      The interfaces cmDepends::Write and cmDepends::WriteDependencies where
      extended to allow multiple dependees (sources) per depender (object).
      cmDepends::Write first collect all dependencies into a std::set before
      passing it to cmDepends::WriteDependencies.
      
      cmDependsC::WriteDependencies also first collects all explicit and
      implicit dependencies into a std::set and only then writes
      depend.{internal,make}. The implementation of cmDependsFortran simply
      loops over all sources and proceeds as before, whereas the cmDependsJava
      implementation is as trivial as before.
      
      This is for preventing exponential growth of depend.{internal,make} in
      the next commit which fixes dependency-vector erasure in
      cmDepends::CheckDependencies.
      Inspired-by: default avatarMichael Wild <themiwi@users.sourceforge.net>
      e74ff7c2
  29. 13 Aug, 2012 1 commit
    • Kitware Robot's avatar
      Remove trailing whitespace from most CMake and C/C++ code · 7bbaa428
      Kitware Robot authored and Brad King's avatar Brad King committed
      Our Git commit hooks disallow modification or addition of lines with
      trailing whitespace.  Wipe out all remnants of trailing whitespace
      everywhere except third-party code.
      
      Run the following shell code:
      
      git ls-files -z -- \
       bootstrap doxygen.config '*.readme' \
       '*.c' '*.cmake' '*.cpp' '*.cxx' \
       '*.el' '*.f' '*.f90' '*.h' '*.in' '*.in.l' '*.java' \
       '*.mm' '*.pike' '*.py' '*.txt' '*.vim' |
      egrep -z -v '^(Utilities/cm|Source/(kwsys|CursesDialog/form)/)' |
      egrep -z -v '^(Modules/CPack\..*\.in)' |
      xargs -0 sed -i 's/ \+$//'
      7bbaa428
  30. 29 Feb, 2012 1 commit
  31. 28 Jan, 2010 1 commit
  32. 04 Jan, 2010 1 commit
    • Brad King's avatar
      Fix escapes in Fortran depend.make entries · cbfbb86b
      Brad King authored
      Makefile dependencies must be escaped using cmLocalGenerator::Convert
      with the cmLocalGenerator::MAKEFILE option.  This fixes Fortran module
      dependencies with spaces in the path.  We test the fix by adding a space
      to one of the module paths in the Fortran test.
      cbfbb86b
  33. 01 Oct, 2009 1 commit
  34. 28 Sep, 2009 1 commit
    • Brad King's avatar
      Convert CMake to OSI-approved BSD License · 96afb120
      Brad King authored
      This converts the CMake license to a pure 3-clause OSI-approved BSD
      License.  We drop the previous license clause requiring modified
      versions to be plainly marked.  We also update the CMake copyright to
      cover the full development time range.
      96afb120
  35. 24 Feb, 2009 1 commit
    • Brad King's avatar
      BUG: Fix Fortran implicit dependency include path · 66f2edbe
      Brad King authored
      The previous change to Source/cmDependsFortran.cxx while refactoring
      implicit dependency scanning configuration rules completely broke
      loading of the include file search path while scanning Fortran
      dependencies.  This adds the line that should have been added during the
      previous change to load the include path correctly.
      66f2edbe