Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add missing includes and remove override identifier for non-virtual functions for CondFormats #29040

Merged
merged 2 commits into from Mar 3, 2020
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
2 changes: 2 additions & 0 deletions CondFormats/EcalObjects/interface/EcalCondObjectContainer.h
Expand Up @@ -7,6 +7,8 @@
#include "DataFormats/EcalDetId/interface/EBDetId.h"
#include "DataFormats/EcalDetId/interface/EEDetId.h"

#include <vector>

template <typename T>
class EcalCondObjectContainer {
public:
Expand Down
4 changes: 4 additions & 0 deletions CondFormats/EcalObjects/interface/EcalTPGCrystalStatus.h
Expand Up @@ -9,6 +9,10 @@
#include "CondFormats/EcalObjects/interface/EcalCondObjectContainer.h"
#include "CondFormats/EcalObjects/interface/EcalTPGCrystalStatusCode.h"

#include "DataFormats/EcalDetId/interface/EcalContainer.h"

#include <vector>

typedef EcalCondObjectContainer<EcalTPGCrystalStatusCode> EcalTPGCrystalStatusMap;

typedef EcalTPGCrystalStatusMap::const_iterator EcalTPGCrystalStatusMapIterator;
Expand Down
Expand Up @@ -5,7 +5,9 @@

#include "CondFormats/EgammaObjects/interface/ElectronLikelihoodCategoryData.h"
#include "CondFormats/PhysicsToolsObjects/interface/Histogram.h"

#include <vector>
#include <atomic>

struct ElectronLikelihoodCalibration {
struct Entry {
Expand Down
4 changes: 3 additions & 1 deletion CondFormats/HcalObjects/interface/HcalCondObjectContainer.h
Expand Up @@ -4,13 +4,15 @@
#include "CondFormats/Serialization/interface/Serializable.h"
#include "CondFormats/HcalObjects/interface/HcalDetIdRelationship.h"

#include <vector>
#include "DataFormats/HcalDetId/interface/HcalDetId.h"
#include "DataFormats/HcalDetId/interface/HcalOtherDetId.h"
#include "DataFormats/HcalDetId/interface/HcalCastorDetId.h"
#include "DataFormats/HcalDetId/interface/HcalZDCDetId.h"
#include "FWCore/Utilities/interface/Exception.h"

#include <vector>
#include <string>

class HcalTopology;

//#define HCAL_COND_SUPPRESS_DEFAULT
Expand Down
7 changes: 4 additions & 3 deletions CondFormats/HcalObjects/interface/HcalIndexLookup.h
@@ -1,16 +1,17 @@
#ifndef CondFormats_HcalObjects_HcalIndexLookup_h
#define CondFormats_HcalObjects_HcalIndexLookup_h

#include <cstdint>
#include <climits>

#include "FWCore/Utilities/interface/Exception.h"

#include "boost/serialization/access.hpp"
#include "boost/serialization/version.hpp"
#include "boost/serialization/vector.hpp"
#include "boost/serialization/utility.hpp"

#include <cstdint>
#include <climits>
#include <vector>

//
// Storable lookup of unsigned index values by unsigned key
// (raw detId, ieta, etc)
Expand Down
7 changes: 4 additions & 3 deletions CondFormats/HcalObjects/interface/HcalItemArrayColl.h
@@ -1,9 +1,6 @@
#ifndef CondFormats_HcalObjects_HcalItemArrayColl_h
#define CondFormats_HcalObjects_HcalItemArrayColl_h

#include <memory>
#include <array>

#include "boost/array.hpp"
#include "boost/serialization/access.hpp"
#include "boost/serialization/version.hpp"
Expand All @@ -16,6 +13,10 @@
#include "boost/serialization/boost_array.hpp"
#endif

#include <memory>
#include <array>
#include <vector>

//
// This collection manages arrays of pointers and references.
// In particular, it can be used for storing objects in
Expand Down
5 changes: 3 additions & 2 deletions CondFormats/HcalObjects/interface/HcalItemColl.h
@@ -1,13 +1,14 @@
#ifndef CondFormats_HcalObjects_HcalItemColl_h
#define CondFormats_HcalObjects_HcalItemColl_h

#include <memory>

#include "boost/serialization/access.hpp"
#include "boost/serialization/version.hpp"
#include "boost/serialization/shared_ptr.hpp"
#include "boost/serialization/vector.hpp"

#include <memory>
#include <vector>

//
// This collection manages only pointers and references.
// In particular, it can be used for storing objects in
Expand Down
3 changes: 3 additions & 0 deletions CondFormats/HcalObjects/interface/HcalLUTCorr.h
Expand Up @@ -3,6 +3,9 @@

#include "CondFormats/Serialization/interface/Serializable.h"

#include <vector>
#include <cstdint>

/*
\class HcalLUTCorr
\author Radek Ofierzynski
Expand Down
5 changes: 4 additions & 1 deletion CondFormats/HcalObjects/interface/HcalLUTCorrs.h
Expand Up @@ -12,6 +12,9 @@ POOL object to store LUT Corrections
#include "CondFormats/HcalObjects/interface/HcalCondObjectContainer.h"
#include "CondFormats/HcalObjects/interface/HcalLUTCorr.h"

#include <string>
#include <cstdint>

//typedef HcalCondObjectContainer<HcalLUTCorr> HcalLUTCorrs;

class HcalLUTCorrs : public HcalCondObjectContainer<HcalLUTCorr> {
Expand All @@ -21,7 +24,7 @@ class HcalLUTCorrs : public HcalCondObjectContainer<HcalLUTCorr> {
#endif
HcalLUTCorrs(const HcalTopology* topo) : HcalCondObjectContainer<HcalLUTCorr>(topo) {}

std::string myname() const override { return (std::string) "HcalLUTCorrs"; }
std::string myname() const { return (std::string) "HcalLUTCorrs"; }

private:
COND_SERIALIZABLE;
Expand Down
3 changes: 3 additions & 0 deletions CondFormats/HcalObjects/interface/HcalLogicalMap.h
Expand Up @@ -3,7 +3,10 @@

#include "CondFormats/HcalObjects/interface/HcalMappingEntry.h"
#include "CondFormats/HcalObjects/interface/HcalElectronicsMap.h"

#include <vector>
#include <memory>

class HcalTopology;

class HcalLogicalMap {
Expand Down
5 changes: 4 additions & 1 deletion CondFormats/HcalObjects/interface/HcalPedestals.h
Expand Up @@ -12,6 +12,9 @@ POOL container to store Pedestal values 4xCapId, using template
#include "CondFormats/HcalObjects/interface/HcalPedestal.h"
#include "CondFormats/HcalObjects/interface/HcalCondObjectContainer.h"

#include <vector>
#include <string>

//typedef HcalCondObjectContainer<HcalPedestal> HcalPedestals;

class HcalPedestals : public HcalCondObjectContainer<HcalPedestal> {
Expand All @@ -28,7 +31,7 @@ class HcalPedestals : public HcalCondObjectContainer<HcalPedestal> {
// set unit boolean
void setUnitADC(bool isADC) { unitIsADC = isADC; }

std::string myname() const override { return (std::string) "HcalPedestals"; }
std::string myname() const { return (std::string) "HcalPedestals"; }

private:
bool unitIsADC;
Expand Down
Expand Up @@ -6,6 +6,8 @@
#include "boost/serialization/vector.hpp"
#include "boost/serialization/version.hpp"

#include <vector>

class PiecewiseScalingPolynomial {
public:
inline PiecewiseScalingPolynomial() {}
Expand Down
6 changes: 4 additions & 2 deletions CondFormats/HcalObjects/interface/StorableDoubleMap.h
@@ -1,12 +1,14 @@
#ifndef CondFormats_HcalObjects_StorableDoubleMap_h
#define CondFormats_HcalObjects_StorableDoubleMap_h

#include <string>
#include <memory>
#include "FWCore/Utilities/interface/Exception.h"

#include "boost/serialization/map.hpp"

#include <string>
#include <memory>
#include <map>

template <typename T>
class StorableDoubleMap {
public:
Expand Down
5 changes: 3 additions & 2 deletions CondFormats/SiPhase2TrackerObjects/interface/DTCELinkId.h
Expand Up @@ -19,11 +19,12 @@ Description: DTCELinkId identifies a specific eLink in the interface of a specif
//
//

#include "CondFormats/Serialization/interface/Serializable.h"

#include <cstdint>
#include <functional>
#include <limits>

#include "CondFormats/Serialization/interface/Serializable.h"

class DTCELinkId {
public:
DTCELinkId() noexcept;
Expand Down
8 changes: 5 additions & 3 deletions CondFormats/SiPixelObjects/interface/SiPixelFedCablingTree.h
@@ -1,12 +1,14 @@
#ifndef SiPixelFedCablingTree_H
#define SiPixelFedCablingTree_H

#include "CondFormats/SiPixelObjects/interface/CablingPathToDetUnit.h"
#include "CondFormats/SiPixelObjects/interface/SiPixelFedCabling.h"
#include "CondFormats/SiPixelObjects/interface/PixelFEDCabling.h"

#include <vector>
#include <unordered_map>
#include <string>

#include "CondFormats/SiPixelObjects/interface/SiPixelFedCabling.h"
#include "CondFormats/SiPixelObjects/interface/PixelFEDCabling.h"
#include <map>

class SiPixelFedCablingTree final : public SiPixelFedCabling {
public:
Expand Down
2 changes: 2 additions & 0 deletions CondFormats/SiPixelObjects/interface/SiPixelFrameConverter.h
Expand Up @@ -7,6 +7,8 @@
#include "CondFormats/SiPixelObjects/interface/SiPixelFedCablingTree.h"
#include "CondFormats/SiPixelObjects/interface/PixelFEDCabling.h"
#include "CondFormats/SiPixelObjects/interface/PixelROC.h"
#include "CondFormats/SiPixelObjects/interface/LocalPixel.h"
#include "CondFormats/SiPixelObjects/interface/GlobalPixel.h"

#include "FWCore/Utilities/interface/GCC11Compatibility.h"
#include <cstdint>
Expand Down