Usefulness of e.g. `L1Phase2MuDTExtPhContainer`?

[makortel]

While working on #50802 I noticed the following pattern

cmssw/DataFormats/L1DTTrackFinder/interface/L1Phase2MuDTExtPhContainer.h

Lines 34 to 48 in 88702e7

	class L1Phase2MuDTExtPhContainer {
	public:
	typedef std::vector<L1Phase2MuDTExtPhDigi> Segment_Container;
	typedef Segment_Container::const_iterator Segment_iterator;
	// Constructor
	L1Phase2MuDTExtPhContainer();
	void setContainer(const Segment_Container& inputSegments);
	Segment_Container const* getContainer() const;
	private:
	Segment_Container m_segments;
	};

This pattern does not seem to add any functionality to just std::vector<L1Phase2MuDTExtPhDigi>, so I'd suggest to explore replacing L1Phase2MuDTExtPhContainer with std::vector<L1Phase2MuDTExtPhDigi> (also for other *Container types in the package).

[makortel]

assign DataFormats/L1DTTrackFinder

[cmsbuild]

New categories assigned: l1

@BenjaminRS,@quinnanm you have been requested to review this Pull request/Issue and eventually sign? Thanks

[cmsbuild]

cms-bot internal usage

[cmsbuild]

A new Issue was created by @makortel.

@Dr15Jones, @ftenchini, @makortel, @mandrenguyen, @sextonkennedy, @smuzaffar can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here