From f7906d265a2cad8a629684a64c4a3a1cb6f981cb Mon Sep 17 00:00:00 2001 From: Paul Barron Date: Tue, 10 Dec 2019 11:39:00 +0100 Subject: [PATCH] Readability change, make case statements easier to read No code changes --- solution/tc_project_app/POUs/MAIN.TcPOU | 22 ++++++++++++++-------- 1 file changed, 14 insertions(+), 8 deletions(-) diff --git a/solution/tc_project_app/POUs/MAIN.TcPOU b/solution/tc_project_app/POUs/MAIN.TcPOU index eac19ba..070885c 100644 --- a/solution/tc_project_app/POUs/MAIN.TcPOU +++ b/solution/tc_project_app/POUs/MAIN.TcPOU @@ -74,7 +74,9 @@ ERROR();]]> - @@ -185,7 +188,7 @@ IF bColdstartDone = FALSE THEN CheckFeedbackType: // Check if axis is incremental, opt-in/restore home and not moving at shutdown -> if so restore FOR i:=1 TO gvl_app.axisNum DO IF fbReadEncRefSys[i].Value=0 AND NOT(bMovingAtShutdown[i]) THEN - IF gvl.axes[i].config.iRestorePosition=2 THEN + IF gvl.axes[i].config.eRestorePosition=2 THEN gvl.axes[i].config.nHomeSeq:=15; gvl.axes[i].control.eCommand:=10; END_IF @@ -195,9 +198,9 @@ IF bColdstartDone = FALSE THEN ExecutePosRestore: // Execute position restore with or without bHome set FOR i:=1 TO gvl_app.axisNum DO IF fbReadEncRefSys[i].Value = 0 THEN - CASE GVL.axes[i].config.iRestorePosition OF - 1: fbRestorePosition[i].Execute:=TRUE; - 2: gvl.axes[i].control.bExecute:=TRUE; + CASE GVL.axes[i].config.eRestorePosition OF + 1: fbRestorePosition[i].Execute:=TRUE; + 2: gvl.axes[i].control.bExecute:=TRUE; END_CASE END_IF END_FOR @@ -205,12 +208,12 @@ IF bColdstartDone = FALSE THEN CheckFinished: // Check set position or direct homing is finished on axes that were opt-in FOR i:=1 TO gvl_app.axisNum DO IF fbReadEncRefSys[i].Value = 0 THEN - CASE gvl.axes[i].config.iRestorePosition OF - 1: + CASE gvl.axes[i].config.eRestorePosition OF + 1: IF NOT fbRestorePosition[i].Done THEN RETURN; END_IF - 2: + 2: IF NOT gvl.axes[i].status.bHomed THEN RETURN; END_IF @@ -236,6 +239,8 @@ END_IF]]> + + @@ -263,6 +268,7 @@ END_IF]]> +