[ntuple] remove GetCollectionInfo() from collection fields

Several collection fields still had a public GetCollectionInfo() method.
It was an oversight not to remove them earlier. These methods are
unneeded and potentially dangerous to expose.

(cherry picked from commit 47662b4)

Author: jblomer

tree/ntuple/inc/ROOT/RField/RFieldProxiedCollection.hxx

@@ -180,16 +180,6 @@ public:
    size_t GetValueSize() const final { return fProxy->Sizeof(); }
    size_t GetAlignment() const final { return alignof(std::max_align_t); }
    void AcceptVisitor(ROOT::Detail::RFieldVisitor &visitor) const final;
-   void
-   GetCollectionInfo(ROOT::NTupleSize_t globalIndex, RNTupleLocalIndex *collectionStart, ROOT::NTupleSize_t *size) const
-   {
-      fPrincipalColumn->GetCollectionInfo(globalIndex, collectionStart, size);
-   }
-   void
-   GetCollectionInfo(RNTupleLocalIndex localIndex, RNTupleLocalIndex *collectionStart, ROOT::NTupleSize_t *size) const
-   {
-      fPrincipalColumn->GetCollectionInfo(localIndex, collectionStart, size);
-   }
 };
 
 ////////////////////////////////////////////////////////////////////////////////

tree/ntuple/inc/ROOT/RField/RFieldSequenceContainer.hxx

@@ -154,16 +154,6 @@ public:
    size_t GetValueSize() const final;
    size_t GetAlignment() const final;
    void AcceptVisitor(ROOT::Detail::RFieldVisitor &visitor) const final;
-   void
-   GetCollectionInfo(ROOT::NTupleSize_t globalIndex, RNTupleLocalIndex *collectionStart, ROOT::NTupleSize_t *size) const
-   {
-      fPrincipalColumn->GetCollectionInfo(globalIndex, collectionStart, size);
-   }
-   void
-   GetCollectionInfo(RNTupleLocalIndex localIndex, RNTupleLocalIndex *collectionStart, ROOT::NTupleSize_t *size) const
-   {
-      fPrincipalColumn->GetCollectionInfo(localIndex, collectionStart, size);
-   }
 };
 
 template <typename ItemT>
@@ -238,16 +228,6 @@ public:
    size_t GetValueSize() const final { return sizeof(std::vector<char>); }
    size_t GetAlignment() const final { return std::alignment_of<std::vector<char>>(); }
    void AcceptVisitor(ROOT::Detail::RFieldVisitor &visitor) const final;
-   void
-   GetCollectionInfo(ROOT::NTupleSize_t globalIndex, RNTupleLocalIndex *collectionStart, ROOT::NTupleSize_t *size) const
-   {
-      fPrincipalColumn->GetCollectionInfo(globalIndex, collectionStart, size);
-   }
-   void
-   GetCollectionInfo(RNTupleLocalIndex localIndex, RNTupleLocalIndex *collectionStart, ROOT::NTupleSize_t *size) const
-   {
-      fPrincipalColumn->GetCollectionInfo(localIndex, collectionStart, size);
-   }
 };
 
 template <typename ItemT>
@@ -296,16 +276,6 @@ public:
    size_t GetValueSize() const final { return sizeof(std::vector<bool>); }
    size_t GetAlignment() const final { return std::alignment_of<std::vector<bool>>(); }
    void AcceptVisitor(ROOT::Detail::RFieldVisitor &visitor) const final;
-   void
-   GetCollectionInfo(ROOT::NTupleSize_t globalIndex, RNTupleLocalIndex *collectionStart, ROOT::NTupleSize_t *size) const
-   {
-      fPrincipalColumn->GetCollectionInfo(globalIndex, collectionStart, size);
-   }
-   void
-   GetCollectionInfo(RNTupleLocalIndex localIndex, RNTupleLocalIndex *collectionStart, ROOT::NTupleSize_t *size) const
-   {
-      fPrincipalColumn->GetCollectionInfo(localIndex, collectionStart, size);
-   }
 };
 
 ////////////////////////////////////////////////////////////////////////////////

tree/ntuple/src/RFieldSequenceContainer.cxx

@@ -341,7 +341,7 @@ std::size_t ROOT::RRVecField::ReadBulkImpl(const RBulkSpec &bulkSpec)
    // Get size of the first RVec of the bulk
    RNTupleLocalIndex firstItemIndex;
    ROOT::NTupleSize_t collectionSize;
-   this->GetCollectionInfo(bulkSpec.fFirstIndex, &firstItemIndex, &collectionSize);
+   fPrincipalColumn->GetCollectionInfo(bulkSpec.fFirstIndex, &firstItemIndex, &collectionSize);
    *beginPtr = itemValueArray;
    *sizePtr = collectionSize;
    *capacityPtr = -1;