By far the most common type of multidomain function you want to create is one where each domain has the same functional form and has a domain index equal to it's place in the function. This adds a method to cover this case to function factory.

Giovanni Di Siena authored 5 years ago and Gigg, Martyn Anthony committed 5 years ago

In places other substitutions have been made, e.g
Clang does not yet specialize std::shared_ptr for T[]. Vector
has been used instead. The operator[] methods were incorrectly
marked const but returning a non-const reference - this has been fixed.
Refs #25842

Runs clang-tidy pass by ref across the codebase to fix-up various
warnings we are seeing in cppcheck

Also updated class_maker.py

re #27319

Update copyright headers in all files

Squashed commit of the following:

* First version of a script to manage copyright statements

re #23468

* Neatened up script, added comments

re #23488

* move script to tools directory

re #23488

* small script changes and a couple of manual file changes

re #23488

* Minor chnage to whitespace detection in regex

re #23488

* Add an excluded directory

re #23488

* remove a repeasted copyright statement in a file

re #23488

* Don't comsume the comment end if it is on the same line

re #23488

* fix error in new copright parsing

re #23488

* remove double copyrifght entry

re #23488

* Improve handling of old copyrights at the start of comments

re #23488

* remove empty comments re #23488

* exclude gsoapgenerated directories

re #23488

* Sort out greedy line matching re #23488

* improve empty comment removal

re #23488

* improve false positives

re #23488

* impressive speedup by limiting regex matching length

re #23488

* remove evil invisible non ascii character

Also upadte the copyright at the same time

re #23488

* resolve multiple copyrights in a single file

re #23488

* resolve an issue with new statement detection

re #23488

* another unprintable unicode character

re #23488

* pep updates and cmake the new copyright fit clang format

re #23488

* update already done new format headers

re #23488

* wrong type of bracket

re #23488

* Update class_maker and friends

re #23488

* Update all copyright statements

re #23488

* clang format re #23488

* flake8 warnings re #23488

* Flake8 warnings re #23488

* Exclude .cmake.in and rb.in files

re #23488

* replace missing line re #23488

* exclude .py.in files as they are flasely recognized as C++

re #23488

* another setp.py.in re #23488

* another .py.in correction re #23488

* Hopefully the last of the .py.in files re #23488

* resolve utf-8 encoding of python files and changed ABINS checksum

re #23488

* updates to unit tests that reference line numbers

re #23488

* remaining unit test files and other fixes

re #23488

Update to clang-format-5 and reformat the codebase 

The createComposite function is recursive. If a child function has a
"parent attribute" (one that starts with $, like $domains) then it is
passed up to its parent and applied there.

re #15518

(cherry picked from commit 8f8de3c94876962103626c4e51d4b901588394ac)

The createComposite function is recursive. If a child function has a
"parent attribute" (one that starts with $, like $domains) then it is
passed up to its parent and applied there.

In this case we had:
"composite=MultiDomainFunction,NumDeriv=true;(name=FunctionFactoryTest_"
        "FunctA,a0=0,a1=0.5;name=FunctionFactoryTest_FunctB,b0=0.1,b1=0.2,ties="
        "(b1=0.2),$domains=i);(name=FunctionFactoryTest_FunctA,a0=0,a1=0.5;"
        "name=FunctionFactoryTest_FunctB,b0=0.1,b1=0.2,$domains=i);ties=(f1.f1."
        "b1=f0.f1.b1)"
so the domains attribute was passed up to the parent CompositeFunction, but that
function doesn't have that attribute - it needs to be passed up again to the
grandparent MultiDomainFunction!

Now the new test passes.

re #16942