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
F157180524: D15102.id41584.diff
Tue, May 19, 2:06 AM
F157180377: D15102.id41584.diff
Tue, May 19, 2:03 AM
F157149828: D15102.id41575.diff
Mon, May 18, 7:17 PM
F157149590: D15102.id41575.diff
Mon, May 18, 7:13 PM
Unknown Object (File)
Mon, May 18, 12:19 PM
Unknown Object (File)
Mon, May 18, 12:11 PM
Unknown Object (File)
Mon, May 18, 12:58 AM
Unknown Object (File)
Mon, May 18, 12:58 AM
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

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

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
head/
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
mips/
mips/
35 lines
8 lines
powerpc/
powerpc/
53 lines
53 lines
riscv/
riscv/
42 lines
10 lines
sparc64/
sparc64/
25 lines
sys/
2 lines

Diff 41584

View Options

head/ObsoleteFiles.inc

Loading...
View Options

head/share/man/man9/Makefile

Loading...
View Options

head/share/man/man9/fetch.9

Loading...
View Options

head/share/man/man9/store.9

Loading...
View Options

head/sys/amd64/amd64/support.S

Loading...
View Options

head/sys/arm/arm/fusu.S

Loading...
View Options

head/sys/arm64/arm64/support.S

Loading...
View Options

head/sys/arm64/arm64/trap.c

Loading...
View Options

head/sys/conf/files.powerpc

Loading...
View Options

head/sys/i386/i386/support.s

Loading...
View Options

head/sys/mips/mips/support.S

Loading...
View Options

head/sys/mips/mips/trap.c

Loading...
View Options

head/sys/powerpc/powerpc/fuswintr.c

Loading...
View Options

head/sys/powerpc/powerpc/suswintr.c

Loading...
View Options

head/sys/riscv/riscv/support.S

Loading...
View Options

head/sys/riscv/riscv/trap.c

Loading...
View Options

head/sys/sparc64/sparc64/support.S

Loading...
View Options

head/sys/sys/resourcevar.h

Loading...