• Update list of authors/contributors

Cleanup commented code

In D131#3576, @abudinen wrote:

@tlatham, @jback, @kreps can we land this?

• Update History.md

The config file is created in commit 000e3be, which is in the master branch.
In the v3-patches branch, the formatting check is enabled in the CI in 05702ff and the formatting is applied to the source files in 8ab96ea. This has been tagged in v3r6.
The application of the formatting in master will happen after the timedep-branch is merged.

Commit 8ab96ea5c5ed in branch v3-patches closes this revision

Many thanks indeed for this @abudinen!

Thanks, this improves exception safety and makes it less likely to introduce accidental deadlocks with future changes that make early returns etc.

Thanks @jback for fixing these quickly. Good point about the R02-02-00-patches version needing fixing. Apologies for not spotting this when reviewing D119. I still haven't made the release from that branch, so we can include the fix before I do so. I think you can just cherry-pick the relevant commits from here onto that branch.

Thanks for this @jback! I've just spotted a couple of small things.

Thanks @abudinen - just a minor wording suggestion for the History file.
Since these are my changes, I'll let John and/or Michal do the actual reviewing!

• Minor fix to JSON

• Update History.md

Looks good to me but then they are my changes ;)
So probably best to have them checked by @jback and/or @kreps before merging.

Looks good, many thanks!

In the context of the thread-safety work, Fernando has been able to make EvtDalitzReso::evaluate a const function. If we were to cherry-pick those changes here we could avoid the copy of each of the EvtDalitzReso instances in EvtD0ToKspipi::calcTotAmp. What do you think?