[Lazarus] Lazarus Release Candidate 1 of 1.2

[Luca Olivetti]

Al 06/11/13 20:10, En/na Mattias Gaertner ha escrit:

> Once the release is out we will have to be more selective about which fixes can be merged for further 1.2.x releases. 

I didn't test it yet (doing it now), but I suggest a simple fix that
doesn't warrant a bug report

--- C:/Documents and Settings/luca/Configuración
local/Temp/lazhelphtml.pas-revBASE.svn000.tmp.pas	Fri May 24 19:30:06 2013
+++ C:/laz_1_2/lcl/lazhelphtml.pas	Mon Sep 23 14:16:50 2013
@@ -365,10 +365,11 @@

   // run
   try
     BrowserProcess:=TProcessUTF8.Create(nil);
     try
+      BrowserProcess.InheritHandles:=false;
       BrowserProcess.CommandLine:=CommandLine;
       BrowserProcess.Execute;
     finally
       BrowserProcess.Free;
     end;



Unfortunately the default for InheritHandles is true, and while the
above has no effect until this bug in fpc is fixed
http://bugs.freepascal.org/view.php?id=22271
I don't think the spawned browser needs to inherit the handles of the
calling process.
In fact InheritHandles true caused me serious problems in the past.

Bye
-- 
Luca Olivetti
Wetron Automation Technology http://www.wetron.es
Tel. +34 935883004  Fax +34 935883007