From cdb8880cb3284869895e8e4ef14e912b6dee3b96 Mon Sep 17 00:00:00 2001 From: andy Date: Fri, 15 Jan 2010 15:57:44 +0000 Subject: [PATCH] Code clean-up --- .../HashTables/HashTable/HashTable.H | 25 +- .../StaticHashTable/StaticHashTable.C | 4 +- .../StaticHashTable/StaticHashTable.H | 16 +- .../StaticHashTable/StaticHashTableI.H | 4 +- .../Lists/BiIndirectList/BiIndirectList.H | 4 +- .../Lists/BiIndirectList/BiIndirectListI.H | 4 +- .../Lists/DynamicList/DynamicList.H | 148 +++++----- .../Lists/DynamicList/DynamicListI.H | 2 +- .../Lists/IndirectList/IndirectList.H | 6 +- .../Lists/IndirectList/IndirectListI.H | 4 +- src/OpenFOAM/containers/Lists/List/List.C | 2 +- src/OpenFOAM/containers/Lists/List/List.H | 10 +- src/OpenFOAM/containers/Lists/List/ListI.H | 2 +- .../containers/Lists/PackedList/PackedList.H | 171 ++++++------ .../containers/Lists/PackedList/PackedListI.H | 11 +- .../containers/Lists/PtrList/PtrList.H | 9 +- .../containers/Lists/PtrList/PtrListI.H | 27 +- .../containers/Lists/UPtrList/UPtrList.H | 4 +- src/OpenFOAM/db/IOstreams/Tstreams/ITstream.H | 2 +- .../primitiveEntry/primitiveEntry.C | 2 +- .../primitiveEntry/primitiveEntry.H | 2 +- src/edgeMesh/edgeFormats/edgeFormatsCore.H | 5 +- .../edgeFormats/emesh/EMESHedgeFormat.C | 4 +- .../edgeFormats/emesh/EMESHedgeFormat.H | 1 + src/edgeMesh/edgeFormats/obj/OBJedgeFormat.C | 4 +- .../edgeFormats/starcd/STARCDedgeFormat.C | 6 +- .../edgeFormats/starcd/STARCDedgeFormat.H | 21 +- src/edgeMesh/edgeFormats/vtk/VTKedgeFormat.C | 6 +- src/edgeMesh/edgeFormats/vtk/VTKedgeFormat.H | 1 + src/edgeMesh/edgeMesh.C | 11 +- src/edgeMesh/edgeMesh.H | 22 +- src/edgeMesh/edgeMeshNew.C | 2 +- .../featureEdgeMesh/featureEdgeMesh.C | 4 +- src/surfMesh/MeshedSurface/MeshedSurface.C | 42 ++- src/surfMesh/MeshedSurface/MeshedSurface.H | 261 +++++++++--------- .../MeshedSurface/MeshedSurfaceZones.C | 4 +- .../MeshedSurfaceIOAllocator.C | 25 +- .../MeshedSurfaceIOAllocator.H | 29 +- .../UnsortedMeshedSurface.C | 30 +- .../UnsortedMeshedSurface.H | 29 +- src/surfMesh/surfMesh/surfMesh.C | 17 +- src/surfMesh/surfMesh/surfMesh.H | 10 +- 42 files changed, 506 insertions(+), 487 deletions(-) diff --git a/src/OpenFOAM/containers/HashTables/HashTable/HashTable.H b/src/OpenFOAM/containers/HashTables/HashTable/HashTable.H index 492d6a9e0f..67b6944e13 100644 --- a/src/OpenFOAM/containers/HashTables/HashTable/HashTable.H +++ b/src/OpenFOAM/containers/HashTables/HashTable/HashTable.H @@ -109,7 +109,6 @@ struct HashTableCore { return iteratorEnd(); } - }; @@ -139,6 +138,7 @@ class HashTable //- Construct from key, next pointer and object inline hashedEntry(const Key&, hashedEntry* next, const T&); + private: //- Disallow default bitwise copy construct hashedEntry(const hashedEntry&); @@ -209,9 +209,8 @@ public: HashTable(const Xfer >&); - // Destructor - - ~HashTable(); + //- Destructor + ~HashTable(); // Member Functions @@ -247,6 +246,7 @@ public: //- Print information Ostream& printInfo(Ostream&) const; + // Edit //- Insert a new hashedEntry @@ -291,7 +291,7 @@ public: void transfer(HashTable&); //- Transfer contents to the Xfer container - inline Xfer< HashTable > xfer(); + inline Xfer > xfer(); // Member Operators @@ -356,9 +356,8 @@ public: //- Current hash index label hashIndex_; - protected: - // Protected Member Functions + protected: // Constructors @@ -380,6 +379,8 @@ public: ); + // Protected Member Functions + //- Increment to the next position inline void increment(); @@ -392,6 +393,7 @@ public: //- Return const access to referenced object inline const T& cobject() const; + public: // Member operators @@ -434,6 +436,7 @@ public: const label hashIndex ); + public: // Constructors @@ -444,13 +447,12 @@ public: //- Construct end iterator inline iterator(const iteratorEnd& unused); + // Member operators //- Conversion to a const_iterator inline operator const_iterator() const; - // Access - //- Return referenced hash value inline T& operator*(); inline T& operator()(); @@ -492,6 +494,7 @@ public: const label hashIndex ); + public: // Constructors @@ -502,9 +505,8 @@ public: //- Construct end iterator inline const_iterator(const iteratorEnd& unused); - // Member operators - // Access + // Member operators //- Return referenced hash value inline const T& operator*() const; @@ -536,7 +538,6 @@ public: Ostream&, const HashTable& ); - }; diff --git a/src/OpenFOAM/containers/HashTables/StaticHashTable/StaticHashTable.C b/src/OpenFOAM/containers/HashTables/StaticHashTable/StaticHashTable.C index 1d0ab2c786..9cbe3154d6 100644 --- a/src/OpenFOAM/containers/HashTables/StaticHashTable/StaticHashTable.C +++ b/src/OpenFOAM/containers/HashTables/StaticHashTable/StaticHashTable.C @@ -97,11 +97,10 @@ Foam::StaticHashTable::StaticHashTable {} - template Foam::StaticHashTable::StaticHashTable ( - const Xfer< StaticHashTable >& ht + const Xfer >& ht ) : StaticHashTableCore(), @@ -452,7 +451,6 @@ void Foam::StaticHashTable::clearStorage() } - template void Foam::StaticHashTable::transfer ( diff --git a/src/OpenFOAM/containers/HashTables/StaticHashTable/StaticHashTable.H b/src/OpenFOAM/containers/HashTables/StaticHashTable/StaticHashTable.H index 9806a138e0..3a2911432c 100644 --- a/src/OpenFOAM/containers/HashTables/StaticHashTable/StaticHashTable.H +++ b/src/OpenFOAM/containers/HashTables/StaticHashTable/StaticHashTable.H @@ -72,10 +72,6 @@ template Ostream& operator<< ); -/*---------------------------------------------------------------------------*\ - Class StaticHashTableName Declaration -\*---------------------------------------------------------------------------*/ - /*---------------------------------------------------------------------------*\ Class StaticHashTableCore Declaration \*---------------------------------------------------------------------------*/ @@ -100,7 +96,6 @@ struct StaticHashTableCore iteratorEnd() {} }; - }; @@ -135,6 +130,7 @@ class StaticHashTable //- Assign a new hashed entry to a possibly already existing key bool set(const Key&, const T& newElmt, bool protect); + public: @@ -183,11 +179,11 @@ public: StaticHashTable(const StaticHashTable&); //- Construct by transferring the parameter contents - StaticHashTable(const Xfer< StaticHashTable >&); + StaticHashTable(const Xfer >&); - // Destructor - ~StaticHashTable(); + //- Destructor + ~StaticHashTable(); // Member Functions @@ -251,7 +247,7 @@ public: void transfer(StaticHashTable&); //- Transfer contents to the Xfer container - inline Xfer< StaticHashTable > xfer(); + inline Xfer > xfer(); // Member Operators @@ -275,6 +271,7 @@ public: //- The opposite of the equality operation. bool operator!=(const StaticHashTable&) const; + // STL type definitions //- Type of values the StaticHashTable contains. @@ -317,6 +314,7 @@ public: //- Index of current element at hashIndex label elemIndex_; + public: // Constructors diff --git a/src/OpenFOAM/containers/HashTables/StaticHashTable/StaticHashTableI.H b/src/OpenFOAM/containers/HashTables/StaticHashTable/StaticHashTableI.H index bd031708e9..8d951f6c1c 100644 --- a/src/OpenFOAM/containers/HashTables/StaticHashTable/StaticHashTableI.H +++ b/src/OpenFOAM/containers/HashTables/StaticHashTable/StaticHashTableI.H @@ -79,7 +79,7 @@ inline bool Foam::StaticHashTable::set template -inline Foam::Xfer< Foam::StaticHashTable > +inline Foam::Xfer > Foam::StaticHashTable::xfer() { return xferMove(*this); @@ -417,6 +417,4 @@ Foam::StaticHashTable::end() const } -// * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * // - // ************************************************************************* // diff --git a/src/OpenFOAM/containers/Lists/BiIndirectList/BiIndirectList.H b/src/OpenFOAM/containers/Lists/BiIndirectList/BiIndirectList.H index 1f761d9caf..d52369c606 100644 --- a/src/OpenFOAM/containers/Lists/BiIndirectList/BiIndirectList.H +++ b/src/OpenFOAM/containers/Lists/BiIndirectList/BiIndirectList.H @@ -74,7 +74,7 @@ public: ( const UList& posList, const UList& negList, - const Xfer< List