Page MenuHomeFreeBSD
Differential D17154

rtld-elf: fix more warnings to allow compiling with WARNS=6
ClosedPublic
Actions

Authored by arichardson on Sep 13 2018, 2:52 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, May 10, 8:25 AM
Unknown Object (File)
Tue, May 5, 6:32 PM
Unknown Object (File)
Mon, May 4, 10:42 AM
Unknown Object (File)
Sun, May 3, 7:15 PM
Unknown Object (File)
Sat, May 2, 11:24 PM
Unknown Object (File)
Mon, Apr 27, 4:33 PM
Unknown Object (File)
Mon, Apr 27, 3:18 PM
Unknown Object (File)
Mon, Apr 27, 7:13 AM
View All Files
Subscribers
imp

Details

Reviewers
kib
jhb
brooks
Commits
rS339879: rtld-elf: fix more warnings to allow compiling with WARNS=6

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

arichardson created this revision.Sep 13 2018, 2:52 PM
Harbormaster completed remote builds in B19568: Diff 48004.Sep 13 2018, 2:52 PM
kib accepted this revision.Sep 13 2018, 3:48 PM
This revision is now accepted and ready to land.Sep 13 2018, 3:48 PM
brooks accepted this revision.Sep 13 2018, 4:17 PM

LGTM

arichardson updated this revision to Diff 49606.Oct 25 2018, 11:21 AM
  • Fixed all issues found by universe build with WARNS=6
  • Silenced a false positive GCC warning
This revision now requires review to proceed.Oct 25 2018, 11:21 AM
Harbormaster completed remote builds in B20418: Diff 49606.Oct 25 2018, 11:21 AM
arichardson edited the summary of this revision. (Show Details)Oct 25 2018, 11:21 AM
kib added a comment.Oct 25 2018, 11:49 AM

Please commit whatever you have now. phab makes it impossible to cleanly see the diff between your patches when there are additional changes in the code from upstream repo, I do not want to re-read all the changes one more time.

brooks accepted this revision.Oct 29 2018, 8:50 PM
This revision is now accepted and ready to land.Oct 29 2018, 8:50 PM
Closed by commit rS339879: rtld-elf: fix more warnings to allow compiling with WARNS=6 (authored by arichardson). · Explain WhyOct 29 2018, 9:08 PM
This revision was automatically updated to reflect the committed changes.
Herald added a subscriber: imp. · View Herald TranscriptOct 29 2018, 9:08 PM

Revision Contents
Changeset List

PathSize
head/
libexec/
rtld-elf/
9 lines
2 lines
23 lines
1 line
riscv/
2 lines
15 lines
2 lines
2 lines

Diff 49769

View Options

head/libexec/rtld-elf/Makefile

Loading...
View Options

head/libexec/rtld-elf/libmap.c

Loading...
View Options

head/libexec/rtld-elf/malloc.c

Loading...
View Options

head/libexec/rtld-elf/map_object.c

Loading...
View Options

head/libexec/rtld-elf/riscv/reloc.c

Loading...
View Options

head/libexec/rtld-elf/rtld.c

Loading...
View Options

head/libexec/rtld-elf/rtld_lock.c

Loading...
View Options

head/libexec/rtld-elf/rtld_printf.c

Loading...