Skip to content

Commit 28ed086

Browse files
authored
Merge pull request #49811 from bsunanda/Phase2-hgx365C
Phase2-hgx365C Try to avoid the complation issue as reported by Akrit Behera for HGCalTypes in UBSCAN platform
2 parents 49be58d + 32651a9 commit 28ed086

File tree

2 files changed

+20
-18
lines changed

2 files changed

+20
-18
lines changed

Geometry/HGCalCommonData/interface/HGCalTypes.h

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -156,20 +156,6 @@ class HGCalTypes {
156156
static constexpr int32_t faccell_ = 100;
157157
static constexpr int32_t faccelltype_ = 10000;
158158
static constexpr int32_t faccell6_ = 1000;
159-
static constexpr int32_t layerType_[7] = {HGCalTypes::WaferCenter,
160-
HGCalTypes::WaferCenterB,
161-
HGCalTypes::WaferCenterR,
162-
HGCalTypes::CornerCenterYp,
163-
HGCalTypes::CornerCenterYm,
164-
HGCalTypes::CornerCenterXp,
165-
HGCalTypes::CornerCenterXm};
166-
static constexpr std::string layerTypes_[7] = {
167-
"Center", "CenterB", "CenterYp", "CenterYm", "CenterR", "CenterXp", "CenterXm"};
168-
static constexpr std::string waferType_[4] = {"HD120", "LD200", "LD300", "HD200"};
169-
static constexpr std::string waferTypeX_[27] = {
170-
"Full", "Five", "ChopTwo", "ChopTwoM", "Half", "Semi", "Semi2", "Three", "Half2",
171-
"Five2", "Unknown10", "LDTop", "LDBottom", "LDLeft", "LDRight", "LDFive", "LDThree", "Unknown17",
172-
"Unknown18", "Unknown19", "Unknown20", "HDTop", "HDBottom", "HDLeft", "HDRight", "HDFive", "Out"};
173159
};
174160

175161
#endif

Geometry/HGCalCommonData/src/HGCalTypes.cc

Lines changed: 20 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -40,15 +40,31 @@ int32_t HGCalTypes::getUnpackedCellType6(int id) { return (id / faccell6_); }
4040
int32_t HGCalTypes::getUnpackedCell6(int id) { return (id % faccell6_); }
4141

4242
int32_t HGCalTypes::layerType(int type) {
43-
return ((type >= 0) && (type < 7)) ? HGCalTypes::layerType_[type] : HGCalTypes::WaferCenter;
43+
static const int32_t layerTypeX[7] = {HGCalTypes::WaferCenter,
44+
HGCalTypes::WaferCenterB,
45+
HGCalTypes::WaferCenterR,
46+
HGCalTypes::CornerCenterYp,
47+
HGCalTypes::CornerCenterYm,
48+
HGCalTypes::CornerCenterXp,
49+
HGCalTypes::CornerCenterXm};
50+
return ((type >= 0) && (type < 7)) ? layerTypeX[type] : HGCalTypes::WaferCenter;
4451
}
4552

46-
std::string HGCalTypes::layerTypeX(int32_t type) { return layerTypes_[HGCalTypes::layerType(type)]; }
53+
std::string HGCalTypes::layerTypeX(int32_t type) {
54+
static const std::string layerTypes[7] = {
55+
"Center", "CenterB", "CenterYp", "CenterYm", "CenterR", "CenterXp", "CenterXm"};
56+
return layerTypes[layerType(type)];
57+
}
4758

4859
std::string HGCalTypes::waferType(int32_t type) {
49-
return (((type >= 0) && (type < 4)) ? HGCalTypes::waferType_[type] : "Undefined");
60+
static const std::string waferType[4] = {"HD120", "LD200", "LD300", "HD200"};
61+
return (((type >= 0) && (type < 4)) ? waferType[type] : "Undefined");
5062
}
5163

5264
std::string HGCalTypes::waferTypeX(int32_t type) {
53-
return (((type >= 0) && (type < 27)) ? HGCalTypes::waferTypeX_[type] : "UnknownXX");
65+
static const std::string waferTypeX[27] = {
66+
"Full", "Five", "ChopTwo", "ChopTwoM", "Half", "Semi", "Semi2", "Three", "Half2",
67+
"Five2", "Unknown10", "LDTop", "LDBottom", "LDLeft", "LDRight", "LDFive", "LDThree", "Unknown17",
68+
"Unknown18", "Unknown19", "Unknown20", "HDTop", "HDBottom", "HDLeft", "HDRight", "HDFive", "Out"};
69+
return (((type >= 0) && (type < 27)) ? waferTypeX[type] : "UnknownXX");
5470
}

0 commit comments

Comments
 (0)