Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • openfoam openfoam
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 426
    • Issues 426
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 9
    • Merge requests 9
  • Deployments
    • Deployments
    • Releases
  • Wiki
    • Wiki
  • Activity
  • Graph
  • Create a new issue
  • Commits
  • Issue Boards
Collapse sidebar
  • Development
  • openfoamopenfoam
  • Merge requests
  • !83

ENH: improve objectRegistry functionality (issue #322)

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Mark OLESEN requested to merge feature-objectRegistry into develop Dec 01, 2016
  • Overview 0
  • Commits 3
  • Changes 6
  • Recursive searching for objects within a registry is now optional (previous it was always done).

    A recursive search effectively blocks the construction of sub-sub-registries if their names are 'masked' by some parent level sub-registry with the same name! (BUG)

  • Recursive search is now turned OFF by default, which makes it consistent with dictionary and probably causes the least number of surprises.


Various new convenience methods added:

lookupObjectRef()

  • returns a non-const reference. For example,

    volScalarField& U = mesh().lookupObjectRef<volScalarField>("U");

    Instead of

    volScalarField& U = const_cast<volScalarField&>
    (
        mesh().lookupObject<volScalarField>("U")
    );

-- lookupObjectPtr()

  • returns a const pointer, and nullptr on failure. For example,

    const volScalarField* Uptr = mesh().lookupObjectPtr<volScalarField>("U");
    if (Uptr)
    {
        const volScalarField& U = *Uptr;
        ...
    }

    Instead of

    if (mesh().foundObject<volScalarField>("U"))
    {
        const volScalarField& U = mesh().lookupObject<volScalarField>("U");
        ...
    }

-- lookupObjectRefPtr()

  • returns a non-const pointer, and nullptr on failure. For example,

    volScalarField* Uptr = mesh().lookupObjectRefPtr<volScalarField>("U");
    if (Uptr)
    {
        volScalarField& U = *Uptr;  // use as reference
        (*Uptr) = ...;              // or use directly
    }

    Instead of

    if (mesh().foundObject<volScalarField>("U"))
    {
        volScalarField& U = const_cast<volScalarField&>
        (
            mesh().lookupObject<volScalarField>("U")
        );
    }

-- sortedNames()

  • now works with template parameters and with regular expression matching as well. For example,

    wordList names  = mesh().sortedNames();
    wordList fields = mesh().sortedName<volScalarField>();

    Instead of

    wordList names  = mesh().sortedNames();
    wordList fields = mesh().names<volScalarField>();
    Foam::sort(fields);

--

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: feature-objectRegistry