Page MenuHomeFreeBSD
Differential D14131

Correct check for nullptr to fix build with clang 6.
ClosedPublic
Actions

Authored by jhb on Jan 30 2018, 4:26 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 18, 5:11 AM
Unknown Object (File)
Thu, Nov 13, 1:10 AM
Unknown Object (File)
Thu, Nov 6, 2:36 AM
Unknown Object (File)
Wed, Nov 5, 2:05 PM
Unknown Object (File)
Tue, Oct 28, 8:27 AM
Unknown Object (File)
Mon, Oct 27, 12:09 PM
Unknown Object (File)
Oct 18 2025, 6:37 AM
Unknown Object (File)
Oct 15 2025, 2:52 AM
View All Files
Subscribers
mat

Details

Reviewers
sbruno
bdrewery
Commits
rP460499: Fix nullptr definition buildfailure on current with clang6
Summary

Check __cplusplus version to determine presence of nullptr rather than
assuming nullptr is a cpp macro.

This should probably be posted upstream as well.

Test Plan
  • build port with recent head

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 14675
Build 14803: arc lint + arc unit

Event Timeline

jhb created this revision.Jan 30 2018, 4:26 PM
Harbormaster completed remote builds in B14675: Diff 38676.Jan 30 2018, 4:26 PM
Herald added a subscriber: mat. · View Herald TranscriptJan 30 2018, 4:26 PM
sbruno added a comment.Jan 30 2018, 6:25 PM

I'll run this through my local meat grinder and approve/commit after. THANKS

sbruno accepted this revision.Jan 31 2018, 4:33 PM
This revision is now accepted and ready to land.Jan 31 2018, 4:33 PM
Closed by commit rP460499: Fix nullptr definition buildfailure on current with clang6 (authored by sbruno). · Explain WhyJan 31 2018, 4:38 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
sysutils/
synergy/
files/
11 lines

Diff 38676

View Options

sysutils/synergy/files/patch-src_lib_common_common.h

Loading...