consistent naming between header and source file

This commit is contained in:
Erik Frojdh 2019-01-21 11:11:39 +01:00
parent e1ec60c9c1
commit b74aed91dc
2 changed files with 24 additions and 24 deletions

View File

@ -727,15 +727,15 @@ void multiSlsDetector::updateOffsets() {
} }
} }
int multiSlsDetector::setOnline(int off, int detPos) { int multiSlsDetector::setOnline(int value, int detPos) {
// single // single
if (detPos >= 0) { if (detPos >= 0) {
return detectors[detPos]->setOnline(off); return detectors[detPos]->setOnline(value);
} }
// multi // multi
if (off != GET_ONLINE_FLAG) { if (value != GET_ONLINE_FLAG) {
auto r = parallelCall(&slsDetector::setOnline, off); auto r = parallelCall(&slsDetector::setOnline, value);
thisMultiDetector->onlineFlag = sls::minusOneIfDifferent(r); thisMultiDetector->onlineFlag = sls::minusOneIfDifferent(r);
} }
return thisMultiDetector->onlineFlag; return thisMultiDetector->onlineFlag;
@ -1317,7 +1317,7 @@ int multiSlsDetector::setSpeed(speedVariable index, int value, int detPos) {
return (sls::allEqualTo(r, static_cast<int>(OK)) ? OK : FAIL); return (sls::allEqualTo(r, static_cast<int>(OK)) ? OK : FAIL);
} }
int multiSlsDetector::setDynamicRange(int p, int detPos) { int multiSlsDetector::setDynamicRange(int dr, int detPos) {
// single // single
if (detPos >= 0) { if (detPos >= 0) {
FILE_LOG(logERROR) << "Dynamic Range cannot be set individually"; FILE_LOG(logERROR) << "Dynamic Range cannot be set individually";
@ -1326,7 +1326,7 @@ int multiSlsDetector::setDynamicRange(int p, int detPos) {
} }
// multi // multi
auto r = parallelCall(&slsDetector::setDynamicRange, p); auto r = parallelCall(&slsDetector::setDynamicRange, dr);
int ret = sls::minusOneIfDifferent(r); int ret = sls::minusOneIfDifferent(r);
// update shm // update shm
@ -1344,7 +1344,7 @@ int multiSlsDetector::setDynamicRange(int p, int detPos) {
// for usability // for usability
if (getDetectorTypeAsEnum() == EIGER) { if (getDetectorTypeAsEnum() == EIGER) {
switch (p) { switch (dr) {
case 32: case 32:
FILE_LOG(logINFO) << "Setting Clock to Quarter Speed to cope with " FILE_LOG(logINFO) << "Setting Clock to Quarter Speed to cope with "
"Dynamic Range of 32"; "Dynamic Range of 32";
@ -1380,15 +1380,15 @@ int multiSlsDetector::getDataBytes(int detPos) {
return sls::sum(r); return sls::sum(r);
} }
int multiSlsDetector::setDAC(int val, dacIndex idac, int mV, int detPos) { int multiSlsDetector::setDAC(int val, dacIndex index, int mV, int detPos) {
// single // single
if (detPos >= 0) { if (detPos >= 0) {
return detectors[detPos]->setDAC(val, idac, mV); return detectors[detPos]->setDAC(val, index, mV);
} }
// multi // multi
auto r = parallelCall(&slsDetector::setDAC, val, idac, mV); auto r = parallelCall(&slsDetector::setDAC, val, index, mV);
if (getDetectorTypeAsEnum() != EIGER || idac != HIGH_VOLTAGE) if (getDetectorTypeAsEnum() != EIGER || index != HIGH_VOLTAGE)
return sls::minusOneIfDifferent(r); return sls::minusOneIfDifferent(r);
// ignore slave values for hv (-999) // ignore slave values for hv (-999)
@ -1401,14 +1401,14 @@ int multiSlsDetector::setDAC(int val, dacIndex idac, int mV, int detPos) {
return firstValue; return firstValue;
} }
int multiSlsDetector::getADC(dacIndex idac, int detPos) { int multiSlsDetector::getADC(dacIndex index, int detPos) {
// single // single
if (detPos >= 0) { if (detPos >= 0) {
return detectors[detPos]->getADC(idac); return detectors[detPos]->getADC(index);
} }
// multi // multi
auto r = parallelCall(&slsDetector::getADC, idac); auto r = parallelCall(&slsDetector::getADC, index);
return sls::minusOneIfDifferent(r); return sls::minusOneIfDifferent(r);
} }
@ -2561,15 +2561,15 @@ void multiSlsDetector::printReceiverConfiguration(int detPos) {
d->printReceiverConfiguration(); d->printReceiverConfiguration();
} }
int multiSlsDetector::setReceiverOnline(int off, int detPos) { int multiSlsDetector::setReceiverOnline(int value, int detPos) {
// single // single
if (detPos >= 0) { if (detPos >= 0) {
return detectors[detPos]->setReceiverOnline(off); return detectors[detPos]->setReceiverOnline(value);
} }
// multi // multi
if (off != GET_ONLINE_FLAG) { if (value != GET_ONLINE_FLAG) {
auto r = parallelCall(&slsDetector::setReceiverOnline, off); auto r = parallelCall(&slsDetector::setReceiverOnline, value);
thisMultiDetector->receiverOnlineFlag = sls::minusOneIfDifferent(r); thisMultiDetector->receiverOnlineFlag = sls::minusOneIfDifferent(r);
} }
return thisMultiDetector->receiverOnlineFlag; return thisMultiDetector->receiverOnlineFlag;
@ -3864,7 +3864,7 @@ int multiSlsDetector::kbhit() {
bool multiSlsDetector::isDetectorIndexOutOfBounds(int detPos) { bool multiSlsDetector::isDetectorIndexOutOfBounds(int detPos) {
// position exceeds multi list size // position exceeds multi list size
if (detPos >= (int)detectors.size()) { if (detPos >= static_cast<int>(detectors.size())) {
FILE_LOG(logERROR) << "Position " << detPos FILE_LOG(logERROR) << "Position " << detPos
<< " is out of bounds with " << " is out of bounds with "
"a detector list of " "a detector list of "

View File

@ -121,7 +121,7 @@ class multiSlsDetector : public virtual slsDetectorDefs,
* one * one
* @param update true to update last user pid, date etc * @param update true to update last user pid, date etc
*/ */
multiSlsDetector(int id = 0, bool verify = true, bool update = true); explicit multiSlsDetector(int id = 0, bool verify = true, bool update = true);
/** /**
* Destructor * Destructor
@ -399,7 +399,7 @@ class multiSlsDetector : public virtual slsDetectorDefs,
* @param detPos -1 for all detectors in list or specific detector position * @param detPos -1 for all detectors in list or specific detector position
* @returns online/offline status * @returns online/offline status
*/ */
int setOnline(int const online = GET_ONLINE_FLAG, int detPos = -1); int setOnline(int value = GET_ONLINE_FLAG, int detPos = -1);
/** /**
* Checks if each of the detectors are online/offline * Checks if each of the detectors are online/offline
@ -734,7 +734,7 @@ class multiSlsDetector : public virtual slsDetectorDefs,
* @param detPos -1 for all detectors in list or specific detector position * @param detPos -1 for all detectors in list or specific detector position
* @returns value of speed set * @returns value of speed set
*/ */
int setSpeed(speedVariable sp, int value = -1, int detPos = -1); int setSpeed(speedVariable index, int value = -1, int detPos = -1);
/** /**
* Set/get dynamic range and updates the number of dataBytes * Set/get dynamic range and updates the number of dataBytes
@ -744,7 +744,7 @@ class multiSlsDetector : public virtual slsDetectorDefs,
* @param detPos -1 for all detectors in list or specific detector position * @param detPos -1 for all detectors in list or specific detector position
* @returns current dynamic range * @returns current dynamic range
*/ */
int setDynamicRange(int i = -1, int detPos = -1); int setDynamicRange(int dr = -1, int detPos = -1);
/** /**
* Recalculated number of data bytes for multi detector * Recalculated number of data bytes for multi detector
@ -1354,7 +1354,7 @@ class multiSlsDetector : public virtual slsDetectorDefs,
* @param detPos -1 for all detectors in list or specific detector position * @param detPos -1 for all detectors in list or specific detector position
* @returns online/offline flag * @returns online/offline flag
*/ */
int setReceiverOnline(int const online = GET_ONLINE_FLAG, int detPos = -1); int setReceiverOnline(int value = GET_ONLINE_FLAG, int detPos = -1);
/** /**
* Checks if the receiver is really online * Checks if the receiver is really online