- 03 Apr, 2018 9 commits
-
-
Mark OLESEN authored
- when constructing dimensioned fields that are to be zero-initialized, it is preferrable to use a form such as dimensionedScalar(dims, Zero) dimensionedVector(dims, Zero) rather than dimensionedScalar("0", dims, 0) dimensionedVector("zero", dims, vector::zero) This reduces clutter and also avoids any suggestion that the name of the dimensioned quantity has any influence on the field's name. An even shorter version is possible. Eg, dimensionedScalar(dims) but reduces the clarity of meaning. - NB: UniformDimensionedField is an exception to these style changes since it does use the name of the dimensioned type (instead of the regIOobject).
-
Mark OLESEN authored
Feature catalyst See merge request !195
-
Mark OLESEN authored
-
Mark OLESEN authored
-
Mark OLESEN authored
-
Mark OLESEN authored
- this can occur when OpenFOAM is run in serial mode, but connects to a program that starts/stops MPI on its own.
-
-
Mark OLESEN authored
- makes the meaning clearer
-
Mark OLESEN authored
-
- 28 Mar, 2018 2 commits
- 27 Mar, 2018 2 commits
-
-
Andrew Heather authored
ENH: reduce profiling overhead (issue #764) See merge request !194
-
Mark OLESEN authored
-
- 26 Mar, 2018 5 commits
-
-
Mark OLESEN authored
- avoid clockTime in favour of clockValue. - avoid allocations when profiling is not active. - replace hashing with manual pointer lists
-
Mark OLESEN authored
Updated/modified PtrList implementation See merge request !193
-
Mark OLESEN authored
- improve internal handling to permit deriving resizable containers (eg, PtrDynList). - include '->' iterator dereferencing - Only append/set non-const autoPtr references. This doesn't break existing code, but makes the intention more transparent.
-
Mark OLESEN authored
-
Mark OLESEN authored
- in many cases can just use lookupOrDefault("key", bool) instead of lookupOrDefault<bool> or lookupOrDefault<Switch> since reading a bool from an Istream uses the Switch(Istream&) anyhow STYLE: relocated Switch string names into file-local scope
-
- 24 Mar, 2018 3 commits
-
-
Mark OLESEN authored
- this is a continuation of commit 39d8a9e6 with simpler template logic.
-
Mark OLESEN authored
- specialize transfer and swap to ensure allocated capacity isn't forgotten.
-
Mark OLESEN authored
- can assist the compiler in producing tighter code.
-
- 22 Mar, 2018 4 commits
-
-
Mark OLESEN authored
-
Mark OLESEN authored
-
Mark OLESEN authored
-
Mark OLESEN authored
- easier to ensure that flags are consistent
-
- 21 Mar, 2018 5 commits
-
-
- occurred when the initial mesh distribution was missing cells on some processors.
-
Andrew Heather authored
The set of injectionMethods has been extended to include a new option: injectionMethod movingPoint; The position is then read as a TimeFunction1 entry, e.g. for a 'table' type: position table ( (0 (-0.009 0.0995 0)) (1e-3 (0.009 0.0995 0)) ); where the list corresponds to the tuples (time (position)), and the time is relative to the start of injection (SOI)
-
mattijs authored
-
mattijs authored
-
mattijs authored
-
- 20 Mar, 2018 2 commits
-
-
Andrew Heather authored
-
Andrew Heather authored
-
- 19 Mar, 2018 1 commit
-
-
Andrew Heather authored
-
- 16 Mar, 2018 7 commits
-
-
Mark OLESEN authored
-
Mark OLESEN authored
- also ensure fewer side-effects from inplaceReorder - provide ListOps::reorder especially for PackedList and PackedBoolList since they behave differently from regular lists.
-
Mark OLESEN authored
- this also provides a better separation of the intent (ie, inserting a single value, or inserting multiply values)
-
Mark OLESEN authored
- eliminate iterators from PackedList since they were unused, had lower performance than direct access and added unneeded complexity. - eliminate auto-vivify for the PackedList '[] operator. The set() method provides any required auto-vivification and removing this ability from the '[]' operator allows for a lower when accessing the values. Replaced the previous cascade of iterators with simpler reference class. PackedBoolList: - (temporarily) eliminate logic and addition operators since these contained partially unclear semantics. - the new test() method tests the value of a single bit position and returns a bool without any ambiguity caused by the return type (like the get() method), nor the const/non-const access (like operator[] has). The name corresponds to what std::bitset uses. - more consistent use of PackedBoolList test(), set(), unset() methods for fewer operation and clearer code. Eg, if (list.test(index)) ... | if (list[index]) ... if (!list.test(index)) ... | if (list[index] == 0u) ... list.set(index); | list[index] = 1u; list.unset(index); | list[index] = 0u; - deleted the operator=(const labelUList&) and replaced with a setMany() method for more clarity about the intended operation and to avoid any potential inadvertent behaviour.
-
Mark OLESEN authored
- consistency with other containers. Allows range-for, enables various std algorithms, and can be used with ListOp::create() with an iterator range.
-
Mark OLESEN authored
- clockValue class for managing the clock values only, with a null constructor that does not query the system clock (can defer to later). Can also be used directly for +/- operations. - refactor clockTime, cpuTime, clock to reduce storage.
-
Mark OLESEN authored
-