fix: fix tellTarget handling on old AS version

Author: jindrapetrik

libsrc/ffdec_lib/src/com/jpexs/decompiler/flash/action/ActionGraph.java

@@ -355,8 +355,8 @@ protected void finalProcess(GraphTargetItem parent, List<GraphTargetItem> list,
                     SetTargetActionItem st = (SetTargetActionItem) it;
                     if (st.target.isEmpty()) {
                         if (targetStart > -1) {
-                            targetEnd = t;
-                            break;
+                        targetEnd = t;
+                        break;
                         }
                     } else {
                         target = new DirectValueActionItem(null, null, 0, st.target, new ArrayList<>());
@@ -378,8 +378,8 @@ protected void finalProcess(GraphTargetItem parent, List<GraphTargetItem> list,
                     }
                     if ((st.target instanceof DirectValueActionItem) && st.target.getResult().equals("")) {
                         if (targetStart > -1) {
-                            targetEnd = t;
-                            break;
+                        targetEnd = t;
+                        break;
                         }
                     } else {
                         targetStart = t;
@@ -433,6 +433,8 @@ protected void finalProcess(GraphTargetItem parent, List<GraphTargetItem> list,
                         targetStartItem = st;
                         target = new DirectValueActionItem(null, null, 0, st.target, new ArrayList<>());
                     }
+                } else if (targetStart > -1) {
+                    targetEnd = t;
                 }
             }
             if (it instanceof SetTarget2ActionItem) {
@@ -445,8 +447,14 @@ protected void finalProcess(GraphTargetItem parent, List<GraphTargetItem> list,
                         targetStartItem = st;
                         target = st.target;
                     }
+                } else if (targetStart > -1) {
+                    targetEnd = t;
                 }
             }
+            
+            if (it instanceof TellTargetActionItem && targetStart > -1) {
+                targetEnd = t;
+            }
 
             if (targetStart > -1 && targetEnd > -1) {
                 List<GraphTargetItem> newlist = new ArrayList<>();
@@ -459,15 +467,15 @@ protected void finalProcess(GraphTargetItem parent, List<GraphTargetItem> list,
                 }
                 newlist.add(new TellTargetActionItem(targetStartItem.getSrc(), targetStartItem.getLineStartItem(), target, tellist));
                 //TODO: maybe set nested flag
-                for (int i = targetEnd + 1; i < list.size(); i++) {
+                for (int i = targetEnd + (it instanceof TellTargetActionItem ? 0 : 1); i < list.size(); i++) {
                     newlist.add(list.get(i));
                 }
                 list.clear();
                 list.addAll(newlist);
                 targetStart = -1;
                 targetEnd = -1;
                 target = null;
-                t = 0;
+                t = -1;
             }
         }
         for (int t = 1/*not first*/; t < list.size(); t++) {

libsrc/ffdec_lib/test/com/jpexs/decompiler/flash/ActionScript2AssemblerTest.java

@@ -627,4 +627,23 @@ public void testVariableCompoundDup() {
         assertEquals(res, "c += 1;");
     }
 
+    @Test
+    public void testTellTarget() {
+        String res = decompilePcode("SetTarget \"\"\n"
+                + "GotoFrame 1\n"
+                + "SetTarget \"../test\"\n"
+                + "GoToLabel \"MyLabel\"\n"
+                + "Play");
+        res = cleanPCode(res);
+        assertEquals(res, "tellTarget(\"\")\n"
+                + "{\n"
+                + "gotoAndStop(2);\n"
+                + "}\n"
+                + "tellTarget(\"../test\")\n"
+                + "{\n"
+                + "gotoAndStop(\"MyLabel\");\n"
+                + "play();\n"
+                + "}");
+    }
+
 }