Re: .: 14 commits - binaryurp/source comphelper/inc comphelper/Library_comphelp.mk comphelper/prj comphelper/source configmgr/source cui/Library_cui.mk cui/prj cui/source dbaccess/source desktop/Library_deploymentgui.mk desktop/Library_sofficeapp.mk desktop/prj desktop/source extensions/Library_pcr.mk extensions/prj extensions/source filter/Library_filterconfig.mk filter/Library_xsltfilter.mk filter/prj filter/source forms/Library_frm.mk forms/prj forms/source jvmfwk/plugins pyuno/prj pyuno/source salhelper/inc salhelper/Library_salhelper.mk salhelper/Package_inc.mk salhelper/source sal/inc sal/Package_inc.mk svtools/inc svtools/source ucb/source

[Stephan Bergmann]

On 02/23/2012 02:56 PM, Michael Stahl wrote:

On 23/02/12 10:48, Stephan Bergmann wrote:

+void salhelper::Thread::launch() {
+    SAL_INFO("salhelper.thread", "launch "<<  name_);
+    // Assumption is that osl::Thread::create returns normally iff it causes
+    // osl::Thread::run to start executing:

looks like this assumption which you moved there is actually wrong

Indeed.  Slightly modified your patch, pushed now.

Thanks,
Stephan
_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice