minor cleanup

mpictor · mpictor · commit bfbd8df61e2c · 2014-12-26T13:34:38.000-05:00
diff --git 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
@@ -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
@@ -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
@@ -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;

Original file line number	Diff line number	Diff line change
`@@ -228,9 +228,10 @@ STEPcomplex * STEPcomplex::EntityPart( const char * name, const char * currSch )`
`228`	`228`	`if( scomp->eDesc->CurrName( name, currSch ) ) {`
`229`	`229`	`return scomp;`
`230`	`230`	`}`
`231`		`- } else`
	`231`	`+ } else {`
`232`	`232`	`cout << "Bug in STEPcomplex::EntityPart(): entity part has "`
`233`	`233`	`<< "no EntityDescriptor\n";`
	`234`	`+ }`
`234`	`235`	`scomp = scomp->sc;`
`235`	`236`	`}`
`236`	`237`	`return 0;`
Original file line number	Diff line number	Diff line change
`@@ -57,7 +57,6 @@ int main( int argc, char * argv[] ) {`
`57`	`57`	`}`
`58`	`58`
`59`	`59`	`EntityAggregate * aggr = instance->isdefinedby_(); //should be filled in when the file is loaded? not sure how to do it using STEPfile...`
`60`		`- //FIXME FIXME FIXME fails because isdefinedby_ uses old data member rather than map lookup`
`61`	`60`	`if( attr.a != aggr ) {`
`62`	`61`	`cout << "Error! got different EntityAggregate's when using map vs method" << endl;`
`63`	`62`	`exitStatus = EXIT_FAILURE;`