[12gevmoller_sim] [JeffersonLab/remoll] 6aa12a: - Split out build type determination from CMakeLis...

Wouter Deconinck noreply at github.com
Mon Feb 18 13:19:51 EST 2019


  Branch: refs/heads/develop
  Home:   https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FJeffersonLab%2Fremoll&data=02%7C01%7C12gevmoller_sim%40jlab.org%7C534b58ca68154be4660208d695cdad02%7Cb4d7ee1f4fb34f0690372b5b522042ab%7C1%7C1%7C636861107932960715&sdata=pU5HljhJ0B5xjkiNTziMW%2Ff1aryeOfQ%2FnX%2F8jwMJkpA%3D&reserved=0
  Commit: 6aa12a39e37aec8c61d7058130072867a9151c1b
      https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FJeffersonLab%2Fremoll%2Fcommit%2F6aa12a39e37aec8c61d7058130072867a9151c1b&data=02%7C01%7C12gevmoller_sim%40jlab.org%7C534b58ca68154be4660208d695cdad02%7Cb4d7ee1f4fb34f0690372b5b522042ab%7C1%7C1%7C636861107932960715&sdata=1qq6mbvpLppfx959x60q%2FXK1hNuh1v95YEXALAi0CNk%3D&reserved=0
  Author: Wouter Deconinck <wdconinc at gmail.com>
  Date:   2019-02-18 (Mon, 18 Feb 2019)

  Changed paths:
    M CMakeLists.txt
    A cmake/modules/BuildType.cmake
    M cmake/modules/CMakeEnv.cmake
    A cmake/modules/PrefixDir.cmake

  Log Message:
  -----------
  - Split out build type determination from CMakeLists.txt

- Split out prefix dir determination
- modify REMOLL_DIAG_FLAGS to include -Wno-shadow, but not through
CMakeEnv


  Commit: 07215c0fd1e2d7a326e71917d72df6a2d6ae74c4
      https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FJeffersonLab%2Fremoll%2Fcommit%2F07215c0fd1e2d7a326e71917d72df6a2d6ae74c4&data=02%7C01%7C12gevmoller_sim%40jlab.org%7C534b58ca68154be4660208d695cdad02%7Cb4d7ee1f4fb34f0690372b5b522042ab%7C1%7C1%7C636861107932960715&sdata=Mikv4CMPBZ5QKqLJ3Q2W8yHHu9BCLEWTAcRThKfrM%2B8%3D&reserved=0
  Author: Wouter Deconinck <wdconinc at gmail.com>
  Date:   2019-02-18 (Mon, 18 Feb 2019)

  Changed paths:
    M src/remollGenericDetectorSum.cc

  Log Message:
  -----------
  Fix pedantic compiler warning

Initializing individual struct fields to zero is not within the c++
spirit of hiding internals and therefore superseded by setting all
fields to zero with `= { 0 }`


Compare: https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FJeffersonLab%2Fremoll%2Fcompare%2F95cbd12b0996...07215c0fd1e2&data=02%7C01%7C12gevmoller_sim%40jlab.org%7C534b58ca68154be4660208d695cdad02%7Cb4d7ee1f4fb34f0690372b5b522042ab%7C1%7C1%7C636861107932960715&sdata=FiP7I%2BlExCTDDmBVotdbgyMVe%2Fv91Fq8QTGb5VewYfE%3D&reserved=0


More information about the 12gevmoller_sim mailing list