[padpainter] use base class methods in attributes setters

Reduce direct access to protected class memebers
Reduce code duplication

Author: linev

graf2d/gpad/src/TPadPainter.cxx

@@ -197,8 +197,7 @@ void TPadPainter::DrawPixels(const unsigned char * /*pixelData*/, UInt_t /*width
 
 void TPadPainter::SetAttFill(const TAttFill &att)
 {
-   if (&fAttFill != &att)
-      att.Copy(fAttFill);
+   TPadPainterBase::SetAttFill(att);
 
    gVirtualX->SetAttFill(fWinContext, att);
 }
@@ -208,8 +207,7 @@ void TPadPainter::SetAttFill(const TAttFill &att)
 
 void TPadPainter::SetAttLine(const TAttLine &att)
 {
-   if (&fAttLine != &att)
-      att.Copy(fAttLine);
+   TPadPainterBase::SetAttLine(att);
 
    gVirtualX->SetAttLine(fWinContext, att);
 }
@@ -219,8 +217,7 @@ void TPadPainter::SetAttLine(const TAttLine &att)
 
 void TPadPainter::SetAttMarker(const TAttMarker &att)
 {
-   if (&fAttMarker != &att)
-      att.Copy(fAttMarker);
+   TPadPainterBase::SetAttMarker(att);
 
    gVirtualX->SetAttMarker(fWinContext, att);
 }
@@ -230,8 +227,7 @@ void TPadPainter::SetAttMarker(const TAttMarker &att)
 
 void TPadPainter::SetAttText(const TAttText &att)
 {
-   if (&fAttText != &att)
-      att.Copy(fAttText);
+   TPadPainterBase::SetAttText(att);
 
    gVirtualX->SetAttText(fWinContext, att);
 }

graf2d/gpad/src/TPadPainterPS.cxx

@@ -65,8 +65,7 @@ void TPadPainterPS::SetOpacity(Int_t percent)
 
 void TPadPainterPS::SetAttFill(const TAttFill &att)
 {
-   if (&att != &fAttFill)
-      att.Copy(fAttFill);
+   TPadPainterBase::SetAttFill(att);
 
    fPS->SetFillColor(att.GetFillColor());
    fPS->SetFillStyle(att.GetFillStyle());
@@ -77,8 +76,7 @@ void TPadPainterPS::SetAttFill(const TAttFill &att)
 
 void TPadPainterPS::SetAttLine(const TAttLine &att)
 {
-   if (&att != &fAttLine)
-      att.Copy(fAttLine);
+   TPadPainterBase::SetAttLine(att);
 
    fPS->SetLineColor(att.GetLineColor());
    fPS->SetLineStyle(att.GetLineStyle());
@@ -90,8 +88,7 @@ void TPadPainterPS::SetAttLine(const TAttLine &att)
 
 void TPadPainterPS::SetAttMarker(const TAttMarker &att)
 {
-   if (&att != &fAttMarker)
-      att.Copy(fAttMarker);
+   TPadPainterBase::SetAttMarker(att);
 
    fPS->SetMarkerColor(att.GetMarkerColor());
    fPS->SetMarkerSize(att.GetMarkerSize());
@@ -103,8 +100,7 @@ void TPadPainterPS::SetAttMarker(const TAttMarker &att)
 
 void TPadPainterPS::SetAttText(const TAttText &att)
 {
-   if (&att != &fAttText)
-      att.Copy(fAttText);
+   TPadPainterBase::SetAttText(att);
 
    fPS->SetTextAlign(att.GetTextAlign());
    fPS->SetTextAngle(att.GetTextAngle());

graf3d/gl/src/TGLPadPainter.cxx

@@ -93,14 +93,12 @@ void TGLPadPainter::OnPad(TVirtualPad *pad)
       fWinContext = gVirtualX->GetWindowContext(pad->GetCanvasID());
 }
 
-
 ////////////////////////////////////////////////////////////////////////////////
 /// Set fill attributes
 
 void TGLPadPainter::SetAttFill(const TAttFill &att)
 {
-   if (&fAttFill != &att)
-      att.Copy(fAttFill);
+   TPadPainterBase::SetAttFill(att);
 
    // TODO: dismiss in the future, gVirtualX attributes not needed in GL
    if (fWinContext && gVirtualX)
@@ -112,8 +110,7 @@ void TGLPadPainter::SetAttFill(const TAttFill &att)
 
 void TGLPadPainter::SetAttLine(const TAttLine &att)
 {
-   if (&fAttLine != &att)
-      att.Copy(fAttLine);
+   TPadPainterBase::SetAttLine(att);
 
    // TODO: dismiss in the future, gVirtualX attributes not needed in GL
    if (fWinContext && gVirtualX)
@@ -125,8 +122,7 @@ void TGLPadPainter::SetAttLine(const TAttLine &att)
 
 void TGLPadPainter::SetAttMarker(const TAttMarker &att)
 {
-   if (&fAttMarker != &att)
-      att.Copy(fAttMarker);
+   TPadPainterBase::SetAttMarker(att);
 
    // TODO: dismiss in the future, gVirtualX attributes not needed in GL
    if (fWinContext && gVirtualX)
@@ -138,8 +134,7 @@ void TGLPadPainter::SetAttMarker(const TAttMarker &att)
 
 void TGLPadPainter::SetAttText(const TAttText &att)
 {
-   if (&fAttText != &att)
-      att.Copy(fAttText);
+   TPadPainterBase::SetAttText(att);
 
    // TODO: dismiss in the future, gVirtualX attributes not needed in GL
    if (fWinContext && gVirtualX)