Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 28 Sep 2024 10:02:24 +0000
From:      bugzilla-noreply@freebsd.org
To:        office@FreeBSD.org
Subject:   [Bug 281516] graphics/libemf: fix build with libc++ 19
Message-ID:  <bug-281516-25061-wJzZU4JE4L@https.bugs.freebsd.org/bugzilla/>
In-Reply-To: <bug-281516-25061@https.bugs.freebsd.org/bugzilla/>
References:  <bug-281516-25061@https.bugs.freebsd.org/bugzilla/>

next in thread | previous in thread | raw e-mail | index | archive | help
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D281516

--- Comment #3 from commit-hook@FreeBSD.org ---
A commit in branch 2024Q3 references this bug:

URL:
https://cgit.FreeBSD.org/ports/commit/?id=3D5b9dda30c763cfaf81e0abcdeee4f6b=
2ac02ff45

commit 5b9dda30c763cfaf81e0abcdeee4f6b2ac02ff45
Author:     Dimitry Andric <dim@FreeBSD.org>
AuthorDate: 2024-09-15 11:55:03 +0000
Commit:     Dimitry Andric <dim@FreeBSD.org>
CommitDate: 2024-09-28 10:01:20 +0000

    graphics/libemf: fix build with libc++ 19

    As noted in the libc++ 19 release notes [1], std::char_traits<> is now
    only provided for char, char8_t, char16_t, char32_t and wchar_t, and any
    instantiation for other types will fail.

    This causes graphics/libemf to fail to compile with clang 19 and libc++
    19, resulting in errors similar to:

      /usr/include/c++/v1/string:820:42: error: implicit instantiation of
undefined template 'std::char_traits<unsigned short>'
        820 |   static_assert(is_same<_CharT, typename
traits_type::char_type>::value,
            |                                          ^
      libemf.cpp:982:30: note: in instantiation of template class
'std::basic_string<unsigned short>' requested here
        982 |     std::basic_string<WCHAR> filename_w( filename, filename +
filename_count );
            |                              ^
      /usr/include/c++/v1/__fwd/string.h:23:29: note: template is declared =
here
         23 | struct _LIBCPP_TEMPLATE_VIS char_traits;
            |                             ^

    libemf should really use char16_t for its 16-bit character type, but the
    code is full of Windows assumptions that WCHAR is unsigned short. Since
    std::basic_string<unsigned short> is no longer available, use
    std::basic_string<char16_t> instead, and cast the result to the
    appropriate type for GetEnhMetaFileW().

    [1] https://libcxx.llvm.org/ReleaseNotes/19.html#deprecations-and-remov=
als

    PR:             281516
    Approved by:    maintainer timeout (~2 weeks)
    MFH:            2024Q3

    (cherry picked from commit 2b65317363f625eceb125cad5864fd873a66562f)

 graphics/libemf/files/patch-libemf_libemf.cpp | 14 +++++++++++++-
 1 file changed, 13 insertions(+), 1 deletion(-)

--=20
You are receiving this mail because:
You are the assignee for the bug.=



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-281516-25061-wJzZU4JE4L>