Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 20 Oct 2024 09:59:36 +0000
From:      bugzilla-noreply@freebsd.org
To:        multimedia@FreeBSD.org
Subject:   [Bug 281887] multimedia/mjpegtools: fix build with clang 19
Message-ID:  <bug-281887-12827-AlB2Pp2EmM@https.bugs.freebsd.org/bugzilla/>
In-Reply-To: <bug-281887-12827@https.bugs.freebsd.org/bugzilla/>
References:  <bug-281887-12827@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=3D281887

--- Comment #2 from commit-hook@FreeBSD.org ---
A commit in branch main references this bug:

URL:
https://cgit.FreeBSD.org/ports/commit/?id=3D8cf728289caf3a06b225866707277d5=
ff170dff0

commit 8cf728289caf3a06b225866707277d5ff170dff0
Author:     Dimitry Andric <dim@FreeBSD.org>
AuthorDate: 2024-10-06 09:50:53 +0000
Commit:     Dimitry Andric <dim@FreeBSD.org>
CommitDate: 2024-10-20 09:58:58 +0000

    multimedia/mjpegtools: fix build with clang 19

    Clang 19 has become more strict about errors in member functions, which
    results in errors building multimedia/mjpegtools:

      ./Region2D.hh:439:13: error: no member named 'DoesContainPoint' in
'Region2D<INDEX, SIZE>'
        439 |                         || this->DoesContainPoint (rHere.m_tn=
Y,
tnX)))
            |                            ~~~~  ^
      ./Region2D.hh:446:22: error: no member named 'Begin' in 'Region2D<IND=
EX,
SIZE>'
        446 |         for (itHere =3D this->Begin(); itHere !=3D this->End(=
);
++itHere)
            |                       ~~~~  ^
      ./Region2D.hh:446:47: error: no member named 'End' in 'Region2D<INDEX,
SIZE>'
        446 |         for (itHere =3D this->Begin(); itHere !=3D this->End(=
);
++itHere)
            |                                                ~~~~  ^
      ./Region2D.hh:549:14: error: no member named 'DoesContainPoint' in
'Region2D<INDEX, SIZE>'
        549 |                         && !this->DoesContainPoint (rHere.m_t=
nY,
tnX))
            |                             ~~~~  ^
      ./Region2D.hh:556:22: error: no member named 'Begin' in 'Region2D<IND=
EX,
SIZE>'
        556 |         for (itHere =3D this->Begin(); itHere !=3D this->End(=
);
++itHere)
            |                       ~~~~  ^
      ./Region2D.hh:556:47: error: no member named 'End' in 'Region2D<INDEX,
SIZE>'
        556 |         for (itHere =3D this->Begin(); itHere !=3D this->End(=
);
++itHere)
            |                                                ~~~~  ^
      ./Region2D.hh:635:16: error: no member named 'DoesContainPoint' in
'Region2D<INDEX, SIZE>'
        635 |                         if (!(this->DoesContainPoint
(rHere.m_tnY, tnX)
            |                               ~~~~  ^
      ./Region2D.hh:643:22: error: no member named 'Begin' in 'Region2D<IND=
EX,
SIZE>'
        643 |         for (itHere =3D this->Begin(); itHere !=3D this->End(=
);
++itHere)
            |                       ~~~~  ^
      ./Region2D.hh:643:47: error: no member named 'End' in 'Region2D<INDEX,
SIZE>'
        643 |         for (itHere =3D this->Begin(); itHere !=3D this->End(=
);
++itHere)
            |                                                ~~~~  ^

    The member functions `DoesContainPoint`, `Begin` and `End` are all
    commented out in Region2D.hh, probably because the code was never
    finished up properly. The member functions referencing them are also
    never used, so comment those out too, using `#if 0`, `#endif` pairs.

    PR:             281887
    Approved by:    maintainer timeout (2 weeks)
    MFH:            2024Q4

 .../files/patch-y4mdenoise_Region2D.hh (new)       | 34 ++++++++++++++++++=
++++
 1 file changed, 34 insertions(+)

--=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-281887-12827-AlB2Pp2EmM>