Page MenuHomeFreeBSD

devel/qjson: Improve clang 16 workaround for usage of the 'register' keyword
ClosedPublic

Authored by rakuco on Jul 8 2023, 9:16 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Jun 24, 10:28 PM
Unknown Object (File)
Mon, Jun 24, 8:55 PM
Unknown Object (File)
Sat, Jun 22, 9:59 PM
Unknown Object (File)
Tue, Jun 18, 10:10 PM
Unknown Object (File)
Mon, Jun 17, 7:48 AM
Unknown Object (File)
Sat, Jun 15, 1:51 AM
Unknown Object (File)
Fri, Jun 14, 6:45 PM
Unknown Object (File)
Apr 28 2024, 2:29 AM
Subscribers

Details

Summary

Commit 3237ce1557467a3f89a3d1fa7a9308161665bf0c fixed the build on CURRENT with a heavy hammer that made register a macro. This change passes -Wno-register to the compiler instead, which is cleaner.

For the record: the current change restricts the change to __FreeBSD_version while this one checks for a compiler version instead -- the former could work if one's building the ports tree with GCC even though the FreeBSD release matters less than the GCC version.

Test Plan

The port built fine on amd64-CURRENT.

Diff Detail

Repository
R11 FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

rakuco requested review of this revision.Jul 8 2023, 9:16 AM
This revision was not accepted when it landed; it landed in state Needs Review.Jul 8 2023, 9:51 AM
This revision was automatically updated to reflect the committed changes.