Tidy-up of Collection library

Merged Jacques Xing requested to merge CFD-Xing/nektar:tidy-collection into master

Issue/feature addressed

This is a tidy-up of the Collection library.

Proposed solution


  • Use default keyword for destructors
  • Move factory at the beginning of the file
  • Use pCollExp[0]->GetShapeDimension() when appropriate
  • Consistently use pCollExp[0]->GetTotPoints()
  • Remove unused code


Suggested reviewers

Please suggest any people who would be appropriate to review your code.


Please add any other information that could be useful for reviewers.


  • Functions and classes, or changes to them, are documented.
  • [ ] User guide/documentation is updated.
  • Changelog is updated.
  • [ ] Suitable tests added for new functionality.
  • Contributed code is correctly formatted. (See the contributing guidelines).
  • [ ] License added to any new files.
  • No extraneous files have been added (e.g. compiler output or test data files).
Edited by Jacques Xing

Merge request reports