0024171: Eliminate CLang compiler warning -Wreorder
[occt.git] / src / AIS / AIS_Selection.cxx
index bf909e6..91c72ae 100755 (executable)
@@ -1,14 +1,31 @@
+// Copyright (c) 1998-1999 Matra Datavision
+// Copyright (c) 1999-2012 OPEN CASCADE SAS
+//
+// The content of this file is subject to the Open CASCADE Technology Public
+// License Version 6.5 (the "License"). You may not use the content of this file
+// except in compliance with the License. Please obtain a copy of the License
+// at http://www.opencascade.org and read it completely before using this file.
+//
+// The Initial Developer of the Original Code is Open CASCADE S.A.S., having its
+// main offices at: 1, place des Freres Montgolfier, 78280 Guyancourt, France.
+//
+// The Original Code and all software distributed under the License is
+// distributed on an "AS IS" basis, without warranty of any kind, and the
+// Initial Developer hereby disclaims all such warranties, including without
+// limitation, any warranties of merchantability, fitness for a particular
+// purpose or non-infringement. Please see the License for the specific terms
+// and conditions governing the rights and limitations under the License.
+
 #include <AIS_Selection.ixx>
 #include <TCollection_AsciiString.hxx>
 #include <TColStd_SequenceOfTransient.hxx>
 
 #define BUC60953   // SAV_050701 : the array of selected objects has always the same length
-                   // independently of number of objects selected. Thus, if there were selected
-                   // more than MaxSizeOfResult objects we have got an exception.
-                   // 
-                   // Moreover, Select method was optimized a little bit.
-                   // Now it checks the state of incoming owner. If the state is 0
-                   // there is no searching for object in <myresult> array.
+// independently of number of objects selected. Thus, if there were selected
+// more than MaxSizeOfResult objects we have got an exception.
+// Moreover, Select method was optimized a little bit.
+// Now it checks the state of incoming owner. If the state is 0
+// there is no searching for object in <myresult> array.
 
 #define OCC138
 
@@ -17,7 +34,7 @@
 #define OCC1039    //SAV: 25/11/02 clearing selected objects if any on the AIS_Selection remove.
 
 #define USE_MAP    //san : 18/04/03 USE_MAP - additional datamap is used to speed up access 
-                   //to certain owners in <myresult> list
+//to certain owners in <myresult> list
 
 #ifdef BUC60953
 #include <SelectMgr_EntityOwner.hxx>
@@ -33,7 +50,6 @@
 #define MaxSizeOfResult 100000
 #endif
 
-// beurk mais moins qu''avant.... rob
 static void AIS_Sel_CurrentSelection (Handle(AIS_Selection)& InputSel)     
 {
   static Handle(AIS_Selection) theCurrentSelection;
@@ -120,7 +136,7 @@ Standard_Boolean AIS_Selection::Find(const Standard_CString aName)
 //=======================================================================
 Standard_Boolean AIS_Selection::SetCurrentSelection (const Standard_CString aName) 
 {  
-  AIS_Selection::CreateSelection(aName); // juste pour voir... ?????????????
+  AIS_Selection::CreateSelection(aName); 
 
 
   Handle(AIS_Selection) anAISSelection = AIS_Selection::Selection(aName) ;
@@ -193,7 +209,7 @@ AIS_SelectStatus AIS_Selection::Select(const Handle(Standard_Transient)& anObjec
     if(arr(i)==anObject)
       Found=i;
   }
-  // Il n''est pas dedans, on le rajoute....
+  // If it is not inside, it is added...
 
 #ifdef BUC60953
   if ( !selected || Found == -1 ) {
@@ -202,7 +218,6 @@ AIS_SelectStatus AIS_Selection::Select(const Handle(Standard_Transient)& anObjec
 #endif  
     if((S->myNb)+1>arr.Length()){
       Handle(TColStd_HArray1OfTransient) NiouTab = new TColStd_HArray1OfTransient(1,arr.Length()+MaxSizeOfResult);
-      // oui bon, mais...
       for(i=arr.Lower();i<=S->myNb;i++){
        const Handle(Standard_Transient)& T = S->myresult->Value(i);
        NiouTab->SetValue(i,T);
@@ -215,7 +230,7 @@ AIS_SelectStatus AIS_Selection::Select(const Handle(Standard_Transient)& anObjec
     S->myresult->SetValue(S->myNb,anObject);
     return AIS_SS_Added;
   }
-  // il etait dedans on l''enleve...
+  // it was inside and it is removed...
   for(i=Found;i<=S->myNb;i++)
     arr(i)=arr(i+1);
   S->myNb--;
@@ -233,11 +248,12 @@ AIS_SelectStatus AIS_Selection::Select(const Handle(Standard_Transient)& anObjec
   if ( S->myResultMap.IsBound( anObject ) ){
     AIS_NListTransient::Iterator aListIter = S->myResultMap.Find( anObject );
 //skt-----------------------------------------------------------------
-    if( S->myIterator == aListIter )
-       if( S->myIterator.More() )
-           S->myIterator.Next();
-       else
-           S->myIterator = AIS_NListTransient::Iterator();
+    if( S->myIterator == aListIter ) {
+      if( S->myIterator.More() )
+          S->myIterator.Next();
+      else
+          S->myIterator = AIS_NListTransient::Iterator();
+    }
 //--------------------------------------------------------------------
 #ifdef BUC60953
     // In the mode of advanced mesh selection only one owner is created
@@ -268,9 +284,7 @@ AIS_SelectStatus AIS_Selection::Select(const Handle(Standard_Transient)& anObjec
   S->myresult.Append( anObject, aListIter );
   S->myResultMap.Bind( anObject, aListIter );
   return AIS_SS_Added;
-#endif //USE_MAP
-  return AIS_SS_Removed;
-  
+#endif //USE_MAP  
 }
 
 //=======================================================================
@@ -288,7 +302,6 @@ AIS_SelectStatus AIS_Selection::AddSelect(const Handle(Standard_Transient)& anOb
   Standard_Integer i;
   if((S->myNb)+1>arr.Length()){
       Handle(TColStd_HArray1OfTransient) NiouTab = new TColStd_HArray1OfTransient(1,arr.Length()+MaxSizeOfResult);
-      // oui bon, mais...
       for(i=arr.Lower();i<=S->myNb;i++){
        const Handle(Standard_Transient)& T = S->myresult->Value(i);
        NiouTab->SetValue(i,T);