No need for extra checks of colors in TG classes

parGC and parFont has default value

Author: linev

gui/gui/src/TGButton.cxx

@@ -1915,7 +1915,7 @@ void TGTextButton::SavePrimitive(std::ostream &out, Option_t *option /*= ""*/)
    parFont.Form("%s::GetDefaultFontStruct()",IsA()->GetName());
    parGC.Form("%s::GetDefaultGC()()",IsA()->GetName());
 
-   if ((GetDefaultFontStruct() != fFontStruct) || (GetDefaultGC()() != fNormGC) || (GetOptions() != (kRaisedFrame | kDoubleBorder))) {
+   if ((GetDefaultFontStruct() != fFontStruct) || (GetDefaultGC()() != fNormGC)) {
       TGFont *ufont = gClient->GetResourcePool()->GetFontPool()->FindFont(fFontStruct);
       if (ufont) {
          ufont->SavePrimitive(out, option);

gui/gui/src/TGButtonGroup.cxx

@@ -628,8 +628,7 @@ void TGButtonGroup::SavePrimitive(std::ostream &out, Option_t *option /*= ""*/)
    // coverity[dereference]
    parGC.Form("%s::GetDefaultGC()()",IsA()->GetName());
 
-   if ((GetDefaultFontStruct() != fFontStruct) || (GetDefaultGC()() != fNormGC) ||
-       (fBackground != GetDefaultFrameBackground())) {
+   if ((GetDefaultFontStruct() != fFontStruct) || (GetDefaultGC()() != fNormGC)) {
       TGFont *ufont = gClient->GetResourcePool()->GetFontPool()->FindFont(fFontStruct);
       if (ufont) {
          ufont->SavePrimitive(out, option);
@@ -720,8 +719,7 @@ void TGHButtonGroup::SavePrimitive(std::ostream &out, Option_t *option /*= ""*/)
    parFont.Form("%s::GetDefaultFontStruct()", IsA()->GetName());
    parGC.Form("%s::GetDefaultGC()()", IsA()->GetName());
 
-   if ((GetDefaultFontStruct() != fFontStruct) || (GetDefaultGC()() != fNormGC) ||
-       (fBackground != GetDefaultFrameBackground())) {
+   if ((GetDefaultFontStruct() != fFontStruct) || (GetDefaultGC()() != fNormGC)) {
       TGFont *ufont = gClient->GetResourcePool()->GetFontPool()->FindFont(fFontStruct);
       if (ufont) {
          ufont->SavePrimitive(out, option);
@@ -802,8 +800,7 @@ void TGVButtonGroup::SavePrimitive(std::ostream &out, Option_t *option /*= ""*/)
    parFont.Form("%s::GetDefaultFontStruct()", IsA()->GetName());
    parGC.Form("%s::GetDefaultGC()()", IsA()->GetName());
 
-   if ((GetDefaultFontStruct() != fFontStruct) || (GetDefaultGC()() != fNormGC) ||
-       (fBackground != GetDefaultFrameBackground())) {
+   if ((GetDefaultFontStruct() != fFontStruct) || (GetDefaultGC()() != fNormGC)) {
       TGFont *ufont = gClient->GetResourcePool()->GetFontPool()->FindFont(fFontStruct);
       if (ufont) {
          ufont->SavePrimitive(out, option);

gui/gui/src/TGFrame.cxx

@@ -3183,8 +3183,7 @@ void TGGroupFrame::SavePrimitive(std::ostream &out, Option_t *option /*= ""*/)
    // coverity[dereference]
    parGC.Form("%s::GetDefaultGC()()", IsA()->GetName());
 
-   if ((GetDefaultFontStruct() != fFontStruct) || (GetDefaultGC()() != fNormGC) ||
-       (fBackground != GetDefaultFrameBackground())) {
+   if ((GetDefaultFontStruct() != fFontStruct) || (GetDefaultGC()() != fNormGC)) {
       TGFont *ufont = gClient->GetResourcePool()->GetFontPool()->FindFont(fFontStruct);
       if (ufont) {
          ufont->SavePrimitive(out, option);

gui/gui/src/TGTextEntry.cxx

@@ -1836,7 +1836,7 @@ void TGTextEntry::SavePrimitive(std::ostream &out, Option_t *option /*= ""*/)
    // coverity[dereference]
    parGC.Form("%s::GetDefaultGC()()", IsA()->GetName());
 
-   if ((GetDefaultFontStruct() != fFontStruct) || (GetDefaultGC()() != fNormGC.GetGC()) || (fBackground != GetWhitePixel())) {
+   if ((GetDefaultFontStruct() != fFontStruct) || (GetDefaultGC()() != fNormGC.GetGC())) {
       TGFont *ufont = gClient->GetResourcePool()->GetFontPool()->FindFont(fFontStruct);
       if (ufont) {
          ufont->SavePrimitive(out, option);