.: 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 sbergman at redhat.com
Thu Feb 23 23:24:16 PST 2012

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.


More information about the LibreOffice mailing list