Page MenuHomeFreeBSD
Differential D14652

Fix build of dspdfviewer on 11.1
ClosedPublic
Actions

Authored by adridg on Mar 11 2018, 1:01 PM.
Tags
None
Referenced Files
F137089722: D14652.id.diff
Fri, Nov 21, 3:06 AM
F137089519: D14652.id40494.diff
Fri, Nov 21, 3:05 AM
F137089043: D14652.id40163.diff
Fri, Nov 21, 3:04 AM
F137088037: D14652.diff
Fri, Nov 21, 3:02 AM
Unknown Object (File)
Thu, Nov 20, 12:27 PM
Unknown Object (File)
Sat, Nov 1, 2:42 AM
Unknown Object (File)
Sun, Oct 26, 4:17 AM
Unknown Object (File)
Fri, Oct 24, 8:15 PM
View All 72 Files
Subscribers
mat

Details

Reviewers
tcberner
rakuco
Commits
rP465086: Fix graphics/dspdfviewer build with modern Clang. The error message
Summary

Build fails (while testing update to CMake, unrelated) with:

../hyperlinkarea.cpp:35:18: error: zero as null pointer constant [-Werror,-Wzero-as-null-pointer-constant]

if ( pixmap == 0 )
               ^
               nullptr

Diff Detail

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

Event Timeline

adridg created this revision.Mar 11 2018, 1:01 PM
Herald added a subscriber: mat. · View Herald TranscriptMar 11 2018, 1:01 PM
Harbormaster completed remote builds in B15494: Diff 40163.Mar 11 2018, 1:01 PM
tcberner accepted this revision.Mar 11 2018, 1:43 PM

Looks good to me.

This revision is now accepted and ready to land.Mar 11 2018, 1:43 PM
mat added inline comments.Mar 12 2018, 10:12 AM
graphics/dspdfviewer/files/patch-hyperlinkarea.cpp
14–15 ↗(On Diff #40163)

I think it should be == NULL. The problem it tells you is that you are comparing a pointer with an integer.

Closed by commit rP465086: Fix graphics/dspdfviewer build with modern Clang. The error message (authored by adridg). · Explain WhyMar 20 2018, 1:02 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
graphics/
dspdfviewer/
files/
18 lines

Diff 40494

View Options

head/graphics/dspdfviewer/files/patch-hyperlinkarea.cpp

Loading...