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)
Thu, Dec 18, 11:06 AM
Unknown Object (File)
Sun, Dec 14, 5:16 PM
Unknown Object (File)
Thu, Dec 11, 3:38 PM
Unknown Object (File)
Wed, Dec 10, 11:41 PM
Unknown Object (File)
Wed, Dec 10, 9:38 PM
Unknown Object (File)
Sat, Nov 29, 9:07 PM
Unknown Object (File)
Nov 25 2025, 11:02 PM
Unknown Object (File)
Nov 19 2025, 12:00 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.