Skip to content

Commit

Permalink
DataSource: implement missing GetLabel method (#682)
Browse files Browse the repository at this point in the history
* GenericParameters: update comment

* DataSource add missing GetLabel override

* Make comment more generic

Co-authored-by: Juan Miguel Carceller <22276694+jmcarcell@users.noreply.github.com>
---------

Co-authored-by: Juan Miguel Carceller <22276694+jmcarcell@users.noreply.github.com>
  • Loading branch information
Zehvogel and jmcarcell committed Sep 20, 2024
1 parent 3e8f634 commit c42a4ea
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
4 changes: 4 additions & 0 deletions include/podio/DataSource.h
Original file line number Diff line number Diff line change
Expand Up @@ -85,6 +85,10 @@ class DataSource : public ROOT::RDF::RDataSource {
///
std::string GetTypeName(std::string_view columnName) const override;

std::string GetLabel() override {
return "PODIO Datasource";
};

protected:
///
/// @brief Type-erased vector of pointers to pointers to column
Expand Down
2 changes: 1 addition & 1 deletion src/GenericParameters.cc
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ GenericParameters::GenericParameters(const GenericParameters& other) :
m_stringMtx(std::make_unique<std::mutex>()),
m_doubleMtx(std::make_unique<std::mutex>()) {
{
// acquire all three locks at once to make sure all three internal maps are
// acquire all locks at once to make sure all internal maps are
// copied at the same "state" of the GenericParameters
auto& intMtx = other.getMutex<int>();
auto& floatMtx = other.getMutex<float>();
Expand Down

0 comments on commit c42a4ea

Please sign in to comment.