Dr. Christian Tacke

DO-NOT-MERGE: Enable more clang-tidy warnings.

1

--- Checklist: * [ ] Followed the [Contributing Guidelines](https://github.com/FairRootGroup/FairRoot/blob/dev/CONTRIBUTING.md) ## Summary by CodeRabbit - **Chores** - Updated build configurations in the CI pipeline: - Removed macOS and various Linux build...

Ownership semantics of `FairRootManager::GetObject`

7

- [ ] Analyze usage in FairRunAnaProof, FairRunOnline of GetObject - #1369 - #1396 https://github.com/FairRootGroup/FairRoot/blob/12f519f8054d50d0d0edc345878f93a8b76a9699/fairroot/online/steer/FairRunOnline.cxx#L221 - [ ] Cleanup FairRootManager and GetObject - #1353 - #1354 - #1402 - [...

(Copied from https://github.com/FairRootGroup/FairRoot/pull/1028#issuecomment-916839687 by @dennisklein) Individual parts of this list can be handled in dedicated PRs or Issues. I suggest to follow up on this by: * [ ] Rebase...

**Is your feature request related to a problem? Please describe.** Our CMake related documentation is really not perfect. **Describe the solution you'd like** Provide some documentation. Maybe in doxygen? `cmake/cmake.dox`...

ROOT 6.26 finally got a notice that TGeoTrack got deprecated. * https://root.cern.ch/doc/v626/classTGeoTrack.html * https://root.cern/doc/v626/release-notes.html#class-reference-guide We should investigate migration options!

**Is your feature request related to a problem? Please describe.** It looks like `examples/advanced/MbsTutorial/` is the only part of the code that tests / is an example for `FairRunOnline`. As...

We deprecated and deactivated MbsAPI for 19.0. If nobody complains, we should remove it for 19.2. According to a group meeting on 2021-07-26, CBM was using MbsAPI and might be...

FR: Read /etc/docker/config.json in addition to ~/.docker/config.json

6

As a sysadmin, I would like to set some options globally for all users. So it would be good, if the logic reading `~/.docker/config.json` would also read `/etc/docker/config.json` and merge...

[core] Pass TString by value to TNamed ctor

1

# This Pull request: Passing by const-reference can mean two constructions (once on the caller, once in the ctor using the TString copy ctor). Instead passing by value and using...

Consider passing TString by value to TNamed ctor

6

### Explain what you would like to see improved and how. The [TNamed ctor](https://root.cern/doc/master/classTNamed.html#a92b15a72a05ddf8f23c67023a29ee6a9) currently takes TString by const reference: ```cpp TNamed::TNamed(const TString& name, const TString& title) ``` Reading it...