From bfbd8df61e2c4a9b5c8b1973c25768c54cc72b50 Mon Sep 17 00:00:00 2001
From: Mark Pictor <mpictor@gmail.com>
Date: Fri, 26 Dec 2014 12:53:29 -0500
Subject: [PATCH] minor cleanup

---
 src/clstepcore/ExpDict.cc                 | 1 -
 src/clstepcore/STEPcomplex.cc             | 3 ++-
 src/clstepcore/STEPcomplex.h              | 6 +-----
 test/cpp/schema_specific/inverse_attr3.cc | 1 -
 4 files changed, 3 insertions(+), 8 deletions(-)

diff --git a/src/clstepcore/ExpDict.cc b/src/clstepcore/ExpDict.cc
index d0a9e537f..a490df8fd 100644
--- a/src/clstepcore/ExpDict.cc
+++ b/src/clstepcore/ExpDict.cc
@@ -21,7 +21,6 @@
 #include <Registry.h>
 #include "sc_memmgr.h"
 
-
 Explicit_item_id__set::Explicit_item_id__set( int defaultSize ) {
     _bufsize = defaultSize;
     _buf = new Explicit_item_id_ptr[_bufsize];
diff --git a/src/clstepcore/STEPcomplex.cc b/src/clstepcore/STEPcomplex.cc
index 9cfbf2d99..08e6b333c 100644
--- a/src/clstepcore/STEPcomplex.cc
+++ b/src/clstepcore/STEPcomplex.cc
@@ -228,9 +228,10 @@ STEPcomplex * STEPcomplex::EntityPart( const char * name, const char * currSch )
             if( scomp->eDesc->CurrName( name, currSch ) ) {
                 return scomp;
             }
-        } else
+        } else {
             cout << "Bug in STEPcomplex::EntityPart(): entity part has "
                  << "no EntityDescriptor\n";
+        }
         scomp = scomp->sc;
     }
     return 0;
diff --git a/src/clstepcore/STEPcomplex.h b/src/clstepcore/STEPcomplex.h
index 883ba6e1b..8c8a9fdfa 100644
--- a/src/clstepcore/STEPcomplex.h
+++ b/src/clstepcore/STEPcomplex.h
@@ -34,7 +34,7 @@ class SC_CORE_EXPORT STEPcomplex : public SDAI_Application_instance {
         STEPcomplex * head;
         Registry * _registry;
         int visited; ///< used when reading (or as you wish?)
-        STEPcomplex_attr_data_list _attr_data_list;
+        STEPcomplex_attr_data_list _attr_data_list; //< what is this for?
     public:
         STEPcomplex( Registry * registry, int fileid );
         STEPcomplex( Registry * registry, const std::string ** names, int fileid,
@@ -43,10 +43,6 @@ class SC_CORE_EXPORT STEPcomplex : public SDAI_Application_instance {
                      const char * schnm = 0 );
         virtual ~STEPcomplex();
 
-        virtual bool IsComplex() const {
-            return true;
-        }
-
         int EntityExists( const char * name, const char * currSch = 0 );
         STEPcomplex * EntityPart( const char * name, const char * currSch = 0 );
 
diff --git a/test/cpp/schema_specific/inverse_attr3.cc b/test/cpp/schema_specific/inverse_attr3.cc
index 495464839..b35f5280d 100644
--- a/test/cpp/schema_specific/inverse_attr3.cc
+++ b/test/cpp/schema_specific/inverse_attr3.cc
@@ -57,7 +57,6 @@ int main( int argc, char * argv[] ) {
         }
 
     EntityAggregate * aggr = instance->isdefinedby_(); //should be filled in when the file is loaded? not sure how to do it using STEPfile...
-    //FIXME FIXME FIXME fails because isdefinedby_ uses old data member rather than map lookup
     if( attr.a != aggr ) {
         cout << "Error! got different EntityAggregate's when using map vs method" << endl;
         exitStatus = EXIT_FAILURE;