「view this page in B3 βῆτα server」

Revisions №60975

branch: master 「№60975」
Commited by: Rocco Moretti
GitHub commit link: 「9f9005b67cd2faed」 「№4281」
Difference from previous tested commit:  code diff
Commit date: 2019-10-10 16:21:03
linux.clang linux.gcc linux.srlz mac.clang
PyRosetta4.notebook linux.clang.cxx11thread.serialization.python37.PyRosetta4.unit linux.gcc.python36.PyRosetta4.unit mac.PyRosetta.unit build.clean.debug cppcheck mysql postgres linux.clang.python36.build.debug linux.zeromq.debug mpi mpi.serialization linux.icc.build.debug OpenCL mac.clang.python36.build.debug build.header build.levels ninja graphics static linux.ui mac.ui build.xcode beautification code_quality.clang_analysis code_quality.clang_tidy serialization integration.mpi integration.release_debug integration.tensorflow integration.thread integration.ubsan integration performance profile release.source scientific.fast_relax.debug scientific.protein_data_bank_diagnostic.full linux.clang.score linux.gcc.score mac.clang.score linux.scripts.pyrosetta scripts.rosetta.parse scripts.rosetta.validate scripts.rosetta.verify linux.clang.unit.release linux.gcc.unit.release util.apps

Merge pull request #4281 from RosettaCommons/roccomoretti/clang_tidy_test_fixes Fixes for clang-tidy issues This merge both adds a new test server test which runs clang-tidy on the Rosetta code base, as well as fixing a number of issues which the default clang-tidy settings identify. (Mostly using virtual functions in constructors and destructors, but a smattering of other issues.) There are a couple of places where the current (C++ standard-mandated) behavior probably isn't what is intended, but I've added comments to those, so hopefully people looking at it in the future can figure out if they want/need to change things.

Test: ubuntu.gcc.integration.ubsan

Failed sub-tests (click for more details):
carbohydrates glycomutagenesis rollmover
Test: mac.clang.integration

Failed sub-tests (click for more details):
carbohydrates glycomutagenesis rollmover