[[ Bug 13658 ]] Only reconsider the meaning of EOF for binary processes

livecodesebastien · livecodesebastien · commit 67f72d04ce35 · 2014-10-17T15:01:30.000+01:00
diff --git a/engine/src/cmdsf.cpp b/engine/src/cmdsf.cpp
@@ -3282,9 +3282,9 @@ Exec_stat MCRead::exec(MCExecPoint &ep)
 				else if (arg == OA_PROCESS)
                 {
 					t_is_text = MCprocesses[index] . textmode != 0;
-                    // SN-2014-10-14: [[ Bug 13658 ]] Ensure that we read all we can from the process, not
+                    // SN-2014-10-14: [[ Bug 13658 ]] Ensure that we read all we can from a binary process, not
                     //  until 0x4 (which might be read before the end, when outputting binary data)
-                    if (*sptr == 0x4)
+                    if (!t_is_text && *sptr == 0x4)
                         ep.setsvalue("");
                 }
 				else

Original file line number	Diff line number	Diff line change
`@@ -3282,9 +3282,9 @@ Exec_stat MCRead::exec(MCExecPoint &ep)`
`3282`	`3282`	`else if (arg == OA_PROCESS)`
`3283`	`3283`	`{`
`3284`	`3284`	`t_is_text = MCprocesses[index] . textmode != 0;`
`3285`		`- // SN-2014-10-14: [[ Bug 13658 ]] Ensure that we read all we can from the process, not`
	`3285`	`+ // SN-2014-10-14: [[ Bug 13658 ]] Ensure that we read all we can from a binary process, not`
`3286`	`3286`	`// until 0x4 (which might be read before the end, when outputting binary data)`
`3287`		`- if (*sptr == 0x4)`
	`3287`	`+ if (!t_is_text && *sptr == 0x4)`
`3288`	`3288`	`ep.setsvalue("");`
`3289`	`3289`	`}`
`3290`	`3290`	`else`