[Libreoffice-commits] core.git: comphelper/source package/qa pyuno/source test/source unotest/source
Michael Stahl
mstahl at redhat.com
Mon Mar 13 14:56:53 UTC 2017
On 13.03.2017 10:43, Miklos Vajna wrote:
> Hi,
>
> On Sat, Mar 11, 2017 at 10:09:01AM +0000, Michael Stahl <mstahl at redhat.com> wrote:
>> commit 9899ffd244dd367ba69dffe1f21f4f0222064a46
>> Author: Michael Stahl <mstahl at redhat.com>
>> Date: Sat Mar 11 00:44:21 2017 +0100
>>
>> comphelper: fix MSVC hang in ThreadPool::shutdown()
>
> The assert added here:
>
>> ThreadPool::~ThreadPool()
>> {
>> - shutdown();
>> + // note: calling shutdown from global variable dtor blocks forever on Win7
>> + // note2: there isn't enough MSVCRT left on exit to call assert() properly
>> + // so these asserts just print something to stderr but exit status is
>> + // still 0, but hopefully they will be more helpful on non-WNT platforms
>> + assert(mbTerminate);
>
> Seems to be a bit brave, see e.g.
> <https://ci.libreoffice.org/job/lo_gerrit/8283/Config=linux_clang_dbgutil_64/console>.
strange, the jenkins build was working fine.
> I'm not sure what is a good way forward -- change that to a SAL_WARN()
> or fix unopkg to shut the thread pool down?
so i guess the unopkg maintainer needs to add the necessary shutdown
code to fix the clearly broken (on Windows 7) situation.
> (I had to revert this locally to have my gcc/dbgutil build 'make check'
> pass.)
do you have any unusual configure flags?
More information about the LibreOffice
mailing list