diff --git a/src/turboPmacAxis.cpp b/src/turboPmacAxis.cpp index 627576d..56ac49d 100644 --- a/src/turboPmacAxis.cpp +++ b/src/turboPmacAxis.cpp @@ -553,11 +553,6 @@ asynStatus turboPmacAxis::handleError(int error, char *userMessage, int sizeUserMessage) { asynStatus status = asynError; - // Create the unique callsite identifier manually so it can be used later in - // the shouldBePrinted calls. - msgPrintControlKey keyError = msgPrintControlKey( - pC_->portName, axisNo_, __PRETTY_FUNCTION__, __LINE__); - switch (error) { case 0: status = asynSuccess; @@ -566,8 +561,9 @@ asynStatus turboPmacAxis::handleError(int error, char *userMessage, case 1: // EPICS should already prevent this issue in the first place, // since it contains the user limits - if (pC_->getMsgPrintControl().shouldBePrinted(keyError, true, - pC_->pasynUser())) { + if (pC_->getMsgPrintControl().shouldBePrinted( + pC_->portName, axisNo_, __PRETTY_FUNCTION__, __LINE__, true, + pC_->pasynUser())) { asynPrint(pC_->pasynUser(), ASYN_TRACE_ERROR, "Controller \"%s\", axis %d => %s, line %d\nTarget " "position would exceed user limits.%s\n", @@ -579,8 +575,9 @@ asynStatus turboPmacAxis::handleError(int error, char *userMessage, break; case 5: // Command not possible - if (pC_->getMsgPrintControl().shouldBePrinted(keyError, true, - pC_->pasynUser())) { + if (pC_->getMsgPrintControl().shouldBePrinted( + pC_->portName, axisNo_, __PRETTY_FUNCTION__, __LINE__, true, + pC_->pasynUser())) { asynPrint( pC_->pasynUser(), ASYN_TRACE_ERROR, "Controller \"%s\", axis %d => %s, line %d\nAxis is " @@ -594,8 +591,9 @@ asynStatus turboPmacAxis::handleError(int error, char *userMessage, "moving. Please call the support."); break; case 8: - if (pC_->getMsgPrintControl().shouldBePrinted(keyError, true, - pC_->pasynUser())) { + if (pC_->getMsgPrintControl().shouldBePrinted( + pC_->portName, axisNo_, __PRETTY_FUNCTION__, __LINE__, true, + pC_->pasynUser())) { asynPrint(pC_->pasynUser(), ASYN_TRACE_ERROR, "Controller \"%s\", axis %d => %s, line %d\nAir cushion " "feedback stopped during movement (P%2.2d01 = %d).%s\n", @@ -609,8 +607,9 @@ asynStatus turboPmacAxis::handleError(int error, char *userMessage, setAxisParamChecked(this, motorMessageText, userMessage); break; case 9: - if (pC_->getMsgPrintControl().shouldBePrinted(keyError, true, - pC_->pasynUser())) { + if (pC_->getMsgPrintControl().shouldBePrinted( + pC_->portName, axisNo_, __PRETTY_FUNCTION__, __LINE__, true, + pC_->pasynUser())) { asynPrint( pC_->pasynUser(), ASYN_TRACE_ERROR, "Controller \"%s\", axis %d => %s, line %d\nNo air cushion " @@ -633,8 +632,9 @@ asynStatus turboPmacAxis::handleError(int error, char *userMessage, which is not properly homed or if a bug occured. */ - if (pC_->getMsgPrintControl().shouldBePrinted(keyError, true, - pC_->pasynUser())) { + if (pC_->getMsgPrintControl().shouldBePrinted( + pC_->portName, axisNo_, __PRETTY_FUNCTION__, __LINE__, true, + pC_->pasynUser())) { asynPrint( pC_->pasynUser(), ASYN_TRACE_ERROR, "Controller \"%s\", axis %d => %s, line %d\nAxis hit the " @@ -653,8 +653,9 @@ asynStatus turboPmacAxis::handleError(int error, char *userMessage, case 11: // Following error - if (pC_->getMsgPrintControl().shouldBePrinted(keyError, true, - pC_->pasynUser())) { + if (pC_->getMsgPrintControl().shouldBePrinted( + pC_->portName, axisNo_, __PRETTY_FUNCTION__, __LINE__, true, + pC_->pasynUser())) { asynPrint( pC_->pasynUser(), ASYN_TRACE_ERROR, "Controller \"%s\", axis %d => %s, line %d\nMaximum allowed " @@ -671,8 +672,9 @@ asynStatus turboPmacAxis::handleError(int error, char *userMessage, break; case 12: - if (pC_->getMsgPrintControl().shouldBePrinted(keyError, true, - pC_->pasynUser())) { + if (pC_->getMsgPrintControl().shouldBePrinted( + pC_->portName, axisNo_, __PRETTY_FUNCTION__, __LINE__, true, + pC_->pasynUser())) { asynPrint(pC_->pasynUser(), ASYN_TRACE_ERROR, "Controller \"%s\", axis %d => %s, line %d\nSecurity " "input is triggered (P%2.2d01 = %d).%s\n", @@ -690,8 +692,9 @@ asynStatus turboPmacAxis::handleError(int error, char *userMessage, case 13: // Driver hardware error triggered - if (pC_->getMsgPrintControl().shouldBePrinted(keyError, true, - pC_->pasynUser())) { + if (pC_->getMsgPrintControl().shouldBePrinted( + pC_->portName, axisNo_, __PRETTY_FUNCTION__, __LINE__, true, + pC_->pasynUser())) { asynPrint(pC_->pasynUser(), ASYN_TRACE_ERROR, "Controller \"%s\", axis %d => %s, line %d\nDriver " "hardware error triggered.%s\n", @@ -707,8 +710,9 @@ asynStatus turboPmacAxis::handleError(int error, char *userMessage, case 14: // EPICS should already prevent this issue in the first place, // since it contains the user limits - if (pC_->getMsgPrintControl().shouldBePrinted(keyError, true, - pC_->pasynUser())) { + if (pC_->getMsgPrintControl().shouldBePrinted( + pC_->portName, axisNo_, __PRETTY_FUNCTION__, __LINE__, true, + pC_->pasynUser())) { asynPrint(pC_->pasynUser(), ASYN_TRACE_ERROR, "Controller \"%s\", axis %d => %s, line %d\nMove " "command exceeds hardware limits (P%2.2d01 = %d).%s\n", @@ -723,8 +727,9 @@ asynStatus turboPmacAxis::handleError(int error, char *userMessage, break; default: - if (pC_->getMsgPrintControl().shouldBePrinted(keyError, true, - pC_->pasynUser())) { + if (pC_->getMsgPrintControl().shouldBePrinted( + pC_->portName, axisNo_, __PRETTY_FUNCTION__, __LINE__, true, + pC_->pasynUser())) { asynPrint( pC_->pasynUser(), ASYN_TRACE_ERROR, "Controller \"%s\", axis %d => %s, line %d\nUnknown error " @@ -735,10 +740,6 @@ asynStatus turboPmacAxis::handleError(int error, char *userMessage, setAxisParamChecked(this, motorMessageText, userMessage); break; } - - if (status == asynSuccess) { - pC_->getMsgPrintControl().resetCount(keyError, pC_->pasynUser()); - } return status; }