Unconditionally check for non-closed-loop parameters in writeFloat64

epics-motor · Oct 24, 2022 · 85fe00e · 85fe00e
1 parent e48ca00
commit 85fe00e
Showing 1 changed file with 29 additions and 32 deletions.
diff --git a/pigcs2App/src/PIasynController.cpp b/pigcs2App/src/PIasynController.cpp
@@ -329,46 +329,43 @@ asynStatus PIasynController::writeFloat64(asynUser *pasynUser, epicsFloat64 valu
             break;
         }
     }
-    if (cl_iter < PIGCS2_CL_PARAM_QTT)
+    if (function == PI_SUP_TARGET)
     {
-        if (function == PI_SUP_TARGET)
-        {
-            printf("PI_SUP_TargetAO: %f for axis %d\n", value, pAxis->axisNo_);
-        }
-        else if (function == PI_SUP_PIVOT_X)
-        {
-            status = m_pGCSController->SetPivotX(value);
-        }
-        else if (function == PI_SUP_PIVOT_Y)
-        {
-            status = m_pGCSController->SetPivotY(value);
-        }
-        else if (function == PI_SUP_PIVOT_Z)
-        {
-            status = m_pGCSController->SetPivotZ(value);
-        }
+        printf("PI_SUP_TargetAO: %f for axis %d\n", value, pAxis->axisNo_);
+    }
+    else if (function == PI_SUP_PIVOT_X)
+    {
+        status = m_pGCSController->SetPivotX(value);
+    }
+    else if (function == PI_SUP_PIVOT_Y)
+    {
+        status = m_pGCSController->SetPivotY(value);
+    }
+    else if (function == PI_SUP_PIVOT_Z)
+    {
+        status = m_pGCSController->SetPivotZ(value);
+    }
     //    else if (function == motorPosition_) // Entspricht das DFH ?
     //    {
     //  //      pAxis->enc_offset = (double) value - pAxis->nextpoint.axis[0].p;
     //        asynPrint(pasynUser, ASYN_TRACE_FLOW,
     //            "%s:%s: Set axis %d to position %d",
     //            driverName, functionName, pAxis->axisNo_, value);
     //    }
-        else if (function == motorResolution_ )
-        {
-            /* Call base class call its method (if we have our parameters check this here) */
-            status = asynMotorController::writeFloat64(pasynUser, value);
-        }
-        else if (function == motorEncoderRatio_)
-        {
-            /* Call base class call its method (if we have our parameters check this here) */
-            status = asynMotorController::writeFloat64(pasynUser, value);
-        }
-        else
-        {
-            /* Call base class call its method (if we have our parameters check this here) */
-            status = asynMotorController::writeFloat64(pasynUser, value);
-        }
+    else if (function == motorResolution_ )
+    {
+        /* Call base class call its method (if we have our parameters check this here) */
+        status = asynMotorController::writeFloat64(pasynUser, value);
+    }
+    else if (function == motorEncoderRatio_)
+    {
+        /* Call base class call its method (if we have our parameters check this here) */
+        status = asynMotorController::writeFloat64(pasynUser, value);
+    }
+    else
+    {
+        /* Call base class call its method (if we have our parameters check this here) */
+        status = asynMotorController::writeFloat64(pasynUser, value);
     }
 
     /* Do callbacks so higher layers see any changes */