1. 26 Jun, 2020 1 commit
  2. 23 Jun, 2020 1 commit
  3. 22 Jun, 2020 2 commits
  4. 16 Jun, 2020 1 commit
  5. 10 Jun, 2020 1 commit
  6. 08 Jun, 2020 4 commits
  7. 05 Jun, 2020 1 commit
  8. 02 Jun, 2020 2 commits
    • Mark Olesen's avatar
      ENH: unify use of dictionary method names · 3e43edf0
      Mark Olesen authored
      - previously introduced `getOrDefault` as a dictionary _get_ method,
        now complete the transition and use it everywhere instead of
        `lookupOrDefault`. This avoids mixed usage of the two methods that
        are identical in behaviour, makes for shorter names, and promotes
        the distinction between "lookup" access (ie, return a token stream,
        locate and return an entry) and "get" access (ie, the above with
        conversion to concrete types such as scalar, label etc).
      3e43edf0
    • Mark Olesen's avatar
      ENH: support predicate checks for argList (similar to dictionary methods) · 31b17221
      Mark Olesen authored
      - Favour use of argList methods that are more similar to dictionary
        method names with the aim of reducing the cognitive load.
      
        * Silently deprecate two-parameter get() method in favour of the
          more familiar getOrDefault.
        * Silently deprecate opt() method in favour of get()
      
        These may be verbosely deprecated in future versions.
      31b17221
  9. 29 May, 2020 1 commit
  10. 23 May, 2020 3 commits
  11. 14 May, 2020 1 commit
  12. 11 May, 2020 1 commit
  13. 04 May, 2020 1 commit
  14. 01 May, 2020 1 commit
  15. 28 Apr, 2020 2 commits
  16. 27 Apr, 2020 1 commit
  17. 15 Apr, 2020 1 commit
  18. 01 Apr, 2020 2 commits
    • Mark Olesen's avatar
      ENH: adjustments to error streams · 617b6027
      Mark Olesen authored
      - refactor common exit/abort code
      
      - support single-parameter output. Eg,
      
          FatalError("some-executable")
              << "Cannot load something\n"
              << exit(FatalError);
      617b6027
    • Mark Olesen's avatar
      ENH: add a Pstream::shutdown() method (#1660) · 01f65054
      Mark Olesen authored
      - previously used a Pstream::exit() invoked from the argList
        destructor to handle all MPI shutdown, but this has the unfortunate
        side-effect of using a fixed return value for the program exit.
      
        Instead use the Pstream::shutdown() method in the destructor and allow
        the normal program exit codes as usual. This means that the
        following code now works as expected.
      
        ```
        argList args(...);
      
        if (...)
        {
            InfoErr<< "some error\n";
            return 1;
        }
        ```
      01f65054
  19. 16 Mar, 2020 1 commit
  20. 11 Mar, 2020 3 commits
  21. 21 Feb, 2020 2 commits
    • Mark Olesen's avatar
      ENH: handle entry alternatives outside of string expansion · 083181ca
      Mark Olesen authored
      - string expansions have supported "${var:-default}" syntax for
        several versions, but this did not apply plain dictionary expansions.
      
        Eg, the following did not parse
      
           massFlow  ${entry1:-100};
      
      ENH: remove content and length restriction on '${..}' quoted variables
      
      - allows this type of content:
      
           velocity2  ${velocity1:- ( 0 -100 10) };
      
      - accept empty parameter strings for entries. This allows the
        following expansion to work as expected:
      
            hex (n1 n2..)  ${inletBlock:-} (10 10 10) simpleGrading (1 1 1)
      
        ie, optionally define the cellZone name for a given block
      
      ENH: add single parameter dictionary writeEntry method.
      
      - the dictionary knows its own name (dictName), which can be used
        when writing content
      083181ca
    • Mark Olesen's avatar
  22. 19 Feb, 2020 3 commits
  23. 18 Feb, 2020 4 commits
    • Mark Olesen's avatar
      ENH: support use of IOstreamOption for IFstream/OFstream · e3f681fa
      Mark Olesen authored
      - can be convenient to bundle IO options as a single parameter
      e3f681fa
    • Mark Olesen's avatar
      STYLE: use unique_ptr for Fstream resource management · f3106ec1
      Mark Olesen authored
      STYLE: change return type of NewOFstream from Ostream to OSstream
      f3106ec1
    • Mark Olesen's avatar
      ENH: support IOstreamOption in abstract stream types · 3135dcf2
      Mark Olesen authored
      - expose the write IOstreamOption in Time
      3135dcf2
    • Mark Olesen's avatar
      ENH: improvements to IOstreamOption · 33f9ae50
      Mark Olesen authored
      * Support default values for format/compress enum lookups.
      
        - Avoids situations where the preferred default format is not ASCII.
          For example, with dictionary input:
      
              format binar;
      
          The typing mistake would previously have caused formatEnum to
          default to ASCII. We can now properly control its behaviour.
      
              IOstream::formatEnum
              (
                  dict.get<word>("format"), IOstream::BINARY
              );
      
          Allowing us to switch ascii/binary, using BINARY by default even in
          the case of spelling mistakes. The mistakes are flagged, but the
          return value can be non-ASCII.
      
      * The format/compression lookup behave as pass-through if the lookup
        string is empty.
      
        - Allows the following to work without complaint
      
            IOstream::formatEnum
            (
                dict.getOrDefault("format", word::null), IOstream::BINARY
            );
      
        - Or use constructor-like failsafe method
      
            IOstream::formatEnum("format", dict, IOstream::BINARY);
      
        - Apply the same behaviour with setting stream format/compression
          from a word.
      
             is.format("binar");
      
          will emit a warning, but leave the stream format UNCHANGED
      
      * Rationalize versionNumber construction
      
        - constexpr constructors where possible.
          Default construct is the "currentVersion"
      
        - Construct from token to shift the burden to versionNumber.
          Support token as argument to version().
      
          Now:
      
              is.version(headerDict.get<token>("version"));
      
          or failsafe constructor method
      
              is.version
              (
                  IOstreamOption::versionNumber("version", headerDict)
              );
      
          Before (controlled input):
      
              is.version
              (
                  IOstreamOption::versionNumber
                  (
                      headerDict.get<float>("version")
                  )
              );
      
          Old, uncontrolled input - has been removed:
      
              is.version(headerDict.lookup("version"));
      
      * improve consistency, default behaviour for IOstreamOption construct
      
        - constexpr constructors where possible
      
        - add copy construct with change of format.
      
        - construct IOstreamOption from streamFormat is now non-explicit.
          This is a commonly expected result with no ill-effects
      33f9ae50