Page MenuHomeFreeBSD
Differential D15102

Remove the unused fuwintr() and suiwintr() functions.
ClosedPublic
Actions

Authored by brooks on Apr 16 2018, 5:56 PM.
Tags
None
Referenced Files
F167027654: D15102.id41584.diff
Tue, Aug 18, 4:05 PM
F166967396: D15102.id41575.diff
Tue, Aug 18, 6:07 AM
Unknown Object (File)
Mon, Aug 17, 1:40 PM
Unknown Object (File)
Mon, Aug 17, 1:17 PM
Unknown Object (File)
Sun, Aug 16, 7:09 PM
Unknown Object (File)
Sun, Aug 16, 7:08 PM
Unknown Object (File)
Sun, Aug 16, 7:07 PM
Unknown Object (File)
Sat, Aug 15, 9:21 PM
View All Files
Subscribers
andrew
emaste
imp
jhibbits

Details

Reviewers
kib
jhb
andrew
Group Reviewers
manpages
Commits
rS332656: Remove the unused fuwintr() and suiwintr() functions.
Summary

Half of implementations always failed (returned (-1)) and they were
previously used in only one place.

Depends on: D15101

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 16189
Build 16141: arc lint + arc unit

Event Timeline

brooks created this revision.Apr 16 2018, 5:56 PM
Herald added subscribers: jhibbits, emaste, andrew. · View Herald TranscriptApr 16 2018, 5:56 PM
Harbormaster completed remote builds in B16171: Diff 41539.Apr 16 2018, 5:56 PM
kib accepted this revision.Apr 16 2018, 7:12 PM
This revision is now accepted and ready to land.Apr 16 2018, 7:12 PM
andrew added a comment.Apr 17 2018, 7:54 AM

Do we need to add the removed MLINKS to ObsoleteFiles.inc?

brooks updated this revision to Diff 41575.Apr 17 2018, 3:39 PM
  • Add MLINKS to OLD_FILES.
This revision now requires review to proceed.Apr 17 2018, 3:39 PM
Harbormaster completed remote builds in B16189: Diff 41575.Apr 17 2018, 3:39 PM
andrew accepted this revision.Apr 17 2018, 4:39 PM
This revision is now accepted and ready to land.Apr 17 2018, 4:39 PM
brooks edited the summary of this revision. (Show Details)Apr 17 2018, 6:03 PM
Closed by commit rS332656: Remove the unused fuwintr() and suiwintr() functions. (authored by brooks). · Explain WhyApr 17 2018, 6:04 PM
This revision was automatically updated to reflect the committed changes.
Herald added a reviewer: manpages. · View Herald TranscriptApr 17 2018, 6:04 PM
Herald added a subscriber: imp. · View Herald Transcript

Revision Contents
Changeset List

PathSize
3 lines
share/
man/
man9/
2 lines
12 lines
11 lines
sys/
amd64/
amd64/
14 lines
arm/
arm/
22 lines
arm64/
arm64/
45 lines
10 lines
conf/
2 lines
i386/
i386/
14 lines
kern/
22 lines
mips/
mips/
35 lines
8 lines
powerpc/
powerpc/
riscv/
riscv/
42 lines
10 lines
sparc64/
sparc64/
25 lines
sys/
2 lines

Diff 41575

View Options

ObsoleteFiles.inc

Loading...
View Options

share/man/man9/Makefile

Loading...
View Options

share/man/man9/fetch.9

Loading...
View Options

share/man/man9/store.9

Loading...
View Options

sys/amd64/amd64/support.S

Loading...
View Options

sys/arm/arm/fusu.S

Loading...
View Options

sys/arm64/arm64/support.S

Loading...
View Options

sys/arm64/arm64/trap.c

Loading...
View Options

sys/conf/files.powerpc

Loading...
View Options

sys/i386/i386/support.s

Loading...
View Options

sys/kern/subr_prof.c

Loading...
View Options

sys/mips/mips/support.S

Loading...
View Options

sys/mips/mips/trap.c

Loading...
View Options

sys/powerpc/powerpc/fuswintr.c

Loading...
View Options

sys/powerpc/powerpc/suswintr.c

Loading...
View Options

sys/riscv/riscv/support.S

Loading...
View Options

sys/riscv/riscv/trap.c

Loading...
View Options

sys/sparc64/sparc64/support.S

Loading...
View Options

sys/sys/resourcevar.h

Loading...