https://bugs.documentfoundation.org/show_bug.cgi?id=92483

--- Comment #5 from DavidO <d.ostrov...@gmx.de> ---
(In reply to Michael Stahl from comment #2)
> assuming that this bug only affects 64 bit builds?
> 
> there is some code to load the MSVCR100.dll in
> jvmfwk/plugins/sunmajor/pluginlib/sunjavaplugin.cxx
> 
> perhaps that needs some adapting for 64bit?

Indeed:

    {
        static struct { char const * name; wchar_t const * wname; } msvcrts[] =
        {
            { "msvcr71.dll" , L"msvcr71.dll"  },
            { "msvcr100.dll", L"msvcr100.dll" },
        };
        char const* importName = (char *) dos_hdr + imports->Name + VAtoPhys;
        for (size_t i = 0; i < SAL_N_ELEMENTS(msvcrts); ++i)
        {
            if (0 == strnicmp(importName,
                // Intentional strlen() + 1 here to include terminating zero
                        msvcrts[i].name, strlen(msvcrts[i].name) + 1))
            {
                load_msvcr(reinterpret_cast<LPCWSTR>(Module->buffer),
                        msvcrts[i].wname);
                free(dos_hdr);
                return;
            }
        }
        imports++;
    }

So why is "msvcr71.dll" there as well? And why are we hard code the
names and not induce them from _MSC_VER?

@Stephan?

-- 
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

Reply via email to