0027900: Coding rules - drop redundant Name parameter from V3d_Viewer constructor
[occt.git] / src / V3d / V3d_View_2.cxx
index 4a6ae4d..597c97a 100644 (file)
 //=============================================================================
 void V3d_View::SetLightOn (const Handle(V3d_Light)& theLight)
 {
-  if (!MyActiveLights.Contains (theLight))
+  if (!myActiveLights.Contains (theLight))
   {
-    V3d_BadValue_Raise_if (MyActiveLights.Extent() >= LightLimit(),
+    V3d_BadValue_Raise_if (myActiveLights.Extent() >= LightLimit(),
                            "V3d_View::SetLightOn, "
                            "too many lights");
-    MyActiveLights.Append (theLight);
+    myActiveLights.Append (theLight);
     UpdateLights();
   }
 }
@@ -71,7 +71,7 @@ void V3d_View::SetLightOff (const Handle(V3d_Light)& theLight)
   Standard_TypeMismatch_Raise_if (MyViewer->IsGlobalLight (theLight),
                                   "V3d_View::SetLightOff, "
                                   "the light is global");
-  MyActiveLights.Remove (theLight);
+  myActiveLights.Remove (theLight);
   UpdateLights();
 }
 
@@ -81,11 +81,8 @@ void V3d_View::SetLightOff (const Handle(V3d_Light)& theLight)
 //=============================================================================
 Standard_Boolean V3d_View::IsActiveLight (const Handle(V3d_Light)& theLight) const
 {
-  if (theLight.IsNull())
-  {
-    return Standard_False;
-  }
-  return MyActiveLights.Contains(theLight);
+  return !theLight.IsNull()
+       && myActiveLights.Contains (theLight);
 }
 
 //=============================================================================
@@ -94,11 +91,11 @@ Standard_Boolean V3d_View::IsActiveLight (const Handle(V3d_Light)& theLight) con
 //=============================================================================
 void V3d_View::SetLightOn()
 {
-  for (MyViewer->InitDefinedLights(); MyViewer->MoreDefinedLights(); MyViewer->NextDefinedLights())
+  for (V3d_ListOfLightIterator aDefLightIter (MyViewer->DefinedLightIterator()); aDefLightIter.More(); aDefLightIter.Next())
   {
-    if (!MyActiveLights.Contains (MyViewer->DefinedLight()))
+    if (!myActiveLights.Contains (aDefLightIter.Value()))
     {
-      MyActiveLights.Append (MyViewer->DefinedLight());
+      myActiveLights.Append (aDefLightIter.Value());
     }
   }
   UpdateLights();
@@ -110,64 +107,27 @@ void V3d_View::SetLightOn()
 //=============================================================================
 void V3d_View::SetLightOff()
 {
-  InitActiveLights();
-  while(MoreActiveLights())
+  for (V3d_ListOfLight::Iterator anActiveLightIter (myActiveLights); anActiveLightIter.More();)
   {
-    if (!MyViewer->IsGlobalLight (ActiveLight()))
+    if (!MyViewer->IsGlobalLight (anActiveLightIter.Value()))
     {
-      MyActiveLights.Remove (ActiveLight());
+      myActiveLights.Remove (anActiveLightIter);
     }
     else
     {
-      NextActiveLights();
+      anActiveLightIter.Next();
     }
   }
   UpdateLights();
 }
 
 //=============================================================================
-//function : InitActiveLights
-//purpose  :
-//=============================================================================
-void V3d_View::InitActiveLights()
-{
-  myActiveLightsIterator.Initialize(MyActiveLights);
-}
-
-//=============================================================================
-//function : MoreActiveLights
-//purpose  :
-//=============================================================================
-Standard_Boolean V3d_View::MoreActiveLights() const
-{
-  return myActiveLightsIterator.More();
-}
-
-//=============================================================================
-//function : NextActiveLights
-//purpose  :
-//=============================================================================
-void V3d_View::NextActiveLights()
-{
-  myActiveLightsIterator.Next();
-}
-
-//=============================================================================
-//function : ActiveLight
-//purpose  :
-//=============================================================================
-Handle(V3d_Light) V3d_View::ActiveLight() const
-{
-  return (Handle(V3d_Light)&)(myActiveLightsIterator.Value());
-}
-
-//=============================================================================
 //function : IfMoreLights
 //purpose  :
 //=============================================================================
 Standard_Boolean V3d_View::IfMoreLights() const
 {
-  return MyActiveLights.Extent() < LightLimit();
+  return myActiveLights.Extent() < LightLimit();
 }
 
 //=======================================================================