Minor changes to BRIGHT_BOX class.

pcbnew/tools/bright_box.cpp

@@ -30,44 +30,31 @@ using namespace KIGFX;
 
 BRIGHT_BOX::BRIGHT_BOX( BOARD_ITEM* aItem ) :
     EDA_ITEM( NOT_USED ),    // this item is never added to a BOARD so it needs no type
-    item( aItem )
+    m_item( aItem )
 {
 }
 
 
-const BOX2I BRIGHT_BOX::ViewBBox() const
-{
-    return item->ViewBBox();
-}
-
-
-void BRIGHT_BOX::ViewGetLayers( int aLayers[], int& aCount ) const
-{
-    aLayers[0] = BrightBoxLayer;
-    aCount = 1;
-}
-
-
 void BRIGHT_BOX::ViewDraw( int aLayer, GAL* aGal ) const
 {
     aGal->SetIsStroke( true );
     aGal->SetIsFill( false );
-    aGal->SetLineWidth( LineWidth );
-    aGal->SetStrokeColor( BrightColor );
+    aGal->SetLineWidth( LINE_WIDTH );
+    aGal->SetStrokeColor( BOX_COLOR );
 
-    if( item->Type() == PCB_TRACE_T )
+    if( m_item->Type() == PCB_TRACE_T )
     {
-        const TRACK* track = static_cast<const TRACK*>( item );
+        const TRACK* track = static_cast<const TRACK*>( m_item );
 
         aGal->DrawSegment( track->GetStart(), track->GetEnd(), track->GetWidth() );
     }
     else
     {
-        BOX2I box = item->ViewBBox();
+        BOX2I box = m_item->ViewBBox();
 
         aGal->DrawRectangle( box.GetOrigin(), box.GetOrigin() + box.GetSize() );
     }
 }
 
 
-const COLOR4D BRIGHT_BOX::BrightColor = KIGFX::COLOR4D( 0.0, 1.0, 0.0, 1.0 );
+const COLOR4D BRIGHT_BOX::BOX_COLOR = KIGFX::COLOR4D( 0.0, 1.0, 0.0, 1.0 );

pcbnew/tools/bright_box.h

@@ -42,21 +42,28 @@ public:
     BRIGHT_BOX( BOARD_ITEM* aItem );
     ~BRIGHT_BOX() {};
 
-    virtual const BOX2I ViewBBox() const;
+    virtual const BOX2I ViewBBox() const
+    {
+        return m_item->ViewBBox();
+    }
 
     void ViewDraw( int aLayer, KIGFX::GAL* aGal ) const;
-    void ViewGetLayers( int aLayers[], int& aCount ) const;
+
+    void ViewGetLayers( int aLayers[], int& aCount ) const
+    {
+        aLayers[0] = ITEM_GAL_LAYER( GP_OVERLAY );
+        aCount = 1;
+    }
 
     void Show( int x, std::ostream& st ) const
     {
     }
 
 private:
-    static const int BrightBoxLayer = ITEM_GAL_LAYER( GP_OVERLAY );
-    static const KIGFX::COLOR4D BrightColor;
-    static const double LineWidth = 100000.0;
+    static const KIGFX::COLOR4D BOX_COLOR;
+    static const double LINE_WIDTH = 100000.0;
 
-    BOARD_ITEM* item;
+    BOARD_ITEM* m_item;
 };
 
 #endif