Remove ensureNoPipeIfWait, possible source of wrong report like #459

https://github.com/Basile-z/dexed/issues/459

src/u_ceproject.pas

@@ -811,7 +811,6 @@ begin
       prc.CurrentDirectory := fSymStringExpander.expand(prc.CurrentDirectory)
     else
       prc.CurrentDirectory:= chd;
-    ensureNoPipeIfWait(prc);
     prc.Execute;
     while prc.Running do
       if poUsePipes in prc.Options then

src/u_common.pas

@@ -250,11 +250,6 @@ type
    *)
   procedure killProcess(var process: TAsyncProcess);
 
-  (**
-   * Ensures that the i/o process pipes are not redirected if it waits on exit.
-   *)
-  procedure ensureNoPipeIfWait(process: TProcess);
-
   (**
    * Returns true if ExeName is already running.
    *)
@@ -1081,13 +1076,6 @@ begin
   process := nil;
 end;
 
-procedure ensureNoPipeIfWait(process: TProcess);
-begin
-  if not (poWaitonExit in process.Options) then
-    exit;
-  process.Options := process.Options - [poStderrToOutPut, poUsePipes];
-end;
-
 function getLineEndingLength(const fname: string): byte;
 var
   value: char = #0;

src/u_main.pas

@@ -3171,7 +3171,7 @@ begin
   fRunProc := TDexedProcess.Create(nil);
   if redirect then
   begin
-  	fRunProc.Options := [poStderrToOutPut, poUsePipes];
+  	fRunProc.Options := [poStderrToOutPut, poUsePipes, poWaitOnExit];
   	fRunProc.ShowWindow := swoHIDE;
   	fRunProc.OnReadData := @asyncprocOutput;
   	fRunProc.OnTerminate:= @asyncprocTerminate;

src/u_tools.pas

@@ -250,7 +250,6 @@ begin
         fProcess.Parameters.Text := arg;
     end;
   end;
-  ensureNoPipeIfWait(fProcess);
 
   if fProcess.Executable.fileExists then
   begin