Fix warning from changes in PR #6656

graf3d/eve7/inc/ROOT/REveDataCollection.hxx

@@ -27,20 +27,17 @@ class TClass;
 namespace ROOT {
 namespace Experimental {
 
-class REveDataItem;
-
-
 //==============================================================================
-// could be a nested class ???
-struct REveDataItem
+class REveDataItem
 {
+private:
    void*    fDataPtr{nullptr};
 
    Bool_t   fRnrSelf{true};
    Color_t  fColor{0};
    Bool_t   fFiltered{false};
 
-
+public:
    REveDataItem(void* d, Color_t c): fDataPtr(d), fColor(c){}
 
    Bool_t  GetRnrSelf() const { return fRnrSelf; }
@@ -48,6 +45,8 @@ struct REveDataItem
    Bool_t  GetFiltered() const { return fFiltered; }
    Bool_t  GetVisible() const { return (!fFiltered) && fRnrSelf; }
 
+   void*  GetDataPtr() { return fDataPtr; } 
+
    void SetFiltered(Bool_t i) { fFiltered = i; }
    void SetMainColor(Color_t i) { fColor = i; }
    void SetRnrSelf(Bool_t i) { fRnrSelf = i; }
@@ -140,9 +139,8 @@ public:
    void ApplyFilter();
 
    Int_t GetNItems() const { return (Int_t) fItemList->fItems.size(); }
-   void *GetDataPtr(Int_t i) const { return  fItemList->fItems[i]->fDataPtr; }
-   //   const REveDataItem& RefDataItem(Int_t i) const { return fItems[i]; }
-    const REveDataItem* GetDataItem(Int_t i) const { return  fItemList->fItems[i]; }
+   void *GetDataPtr(Int_t i) const { return  fItemList->fItems[i]->GetDataPtr(); }
+   const REveDataItem* GetDataItem(Int_t i) const { return  fItemList->fItems[i]; }
 
    void  StreamPublicMethods(nlohmann::json &cj) const;
    Int_t WriteCoreJson(nlohmann::json &cj, Int_t rnr_offset) override;

graf3d/eve7/src/REveDataCollection.cxx

@@ -48,7 +48,7 @@ REveDataItemList::REveDataItemList(const std::string& n, const std::string& t):
 
 void REveDataItemList::SetItemVisible(Int_t idx, Bool_t visible)
 {
-   fItems[idx]->fRnrSelf = visible;
+   fItems[idx]->SetRnrSelf(visible);
    ItemChanged(idx);
    StampObjProps();
 }
@@ -58,7 +58,7 @@ void REveDataItemList::SetItemVisible(Int_t idx, Bool_t visible)
 void REveDataItemList::SetItemColorRGB(Int_t idx, UChar_t r, UChar_t g, UChar_t b)
 {
    Color_t c = TColor::GetColor(r, g, b);
-   fItems[idx]->fColor = c;
+   fItems[idx]->SetMainColor(c);
    ItemChanged(idx);
    StampObjProps();
 }
@@ -110,9 +110,9 @@ Int_t REveDataItemList::WriteCoreJson(nlohmann::json &j, Int_t rnr_offset)
    for (auto & chld : fItems)
    {
       nlohmann::json i;
-      i["fFiltered"] = chld->fFiltered;
-      i["fRnrSelf"] = chld->fRnrSelf;
-      i["fColor"] = chld->fColor;
+      i["fFiltered"] = chld->GetFiltered();
+      i["fRnrSelf"] = chld->GetRnrSelf();
+      i["fColor"] = chld->GetMainColor();
       j["items"].push_back(i);
    }
 
@@ -236,7 +236,7 @@ void REveDataCollection::ApplyFilter()
    int idx = 0;
    for (auto &ii : fItemList->fItems)
    {
-      bool res = fFilterFoo(ii->fDataPtr);
+      bool res = fFilterFoo(ii->GetDataPtr());
 
       // printf("Item:%s -- filter result = %d\n", ii.fItemPtr->GetElementName(), res);
 
@@ -333,7 +333,7 @@ void REveDataCollection::SetMainColor(Color_t newv)
    REveElement::SetMainColor(newv);
    for (auto & chld : fItemList->fItems)
    {
-      chld->fColor = newv;
+      chld->SetMainColor(newv);
    }
    fItemList->StampObjProps();
    fItemList->SetMainColor(newv);