Skip to content

Commit 1c96223

Browse files
WeiminShanggopherbot
authored andcommitted
route: drops const that x/unix has
For golang/go#54035 Change-Id: I5163bb1d562f0c7f7de16995fcb42ece894d07bf Reviewed-on: https://go-review.googlesource.com/c/net/+/419180 TryBot-Result: Gopher Robot <[email protected]> Reviewed-by: Damien Neil <[email protected]> Reviewed-by: Ian Lance Taylor <[email protected]> Run-TryBot: Ian Lance Taylor <[email protected]> Run-TryBot: Meng Zhuo <[email protected]> Auto-Submit: Ian Lance Taylor <[email protected]>
1 parent 52228ea commit 1c96223

31 files changed

+215
-1168
lines changed

Diff for: route/address.go

+21-17
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,11 @@
77

88
package route
99

10-
import "runtime"
10+
import (
11+
"runtime"
12+
13+
"golang.org/x/sys/unix"
14+
)
1115

1216
// An Addr represents an address associated with packet routing.
1317
type Addr interface {
@@ -23,7 +27,7 @@ type LinkAddr struct {
2327
}
2428

2529
// Family implements the Family method of Addr interface.
26-
func (a *LinkAddr) Family() int { return sysAF_LINK }
30+
func (a *LinkAddr) Family() int { return unix.AF_LINK }
2731

2832
func (a *LinkAddr) lenAndSpace() (int, int) {
2933
l := 8 + len(a.Name) + len(a.Addr)
@@ -40,7 +44,7 @@ func (a *LinkAddr) marshal(b []byte) (int, error) {
4044
return 0, errInvalidAddr
4145
}
4246
b[0] = byte(l)
43-
b[1] = sysAF_LINK
47+
b[1] = unix.AF_LINK
4448
if a.Index > 0 {
4549
nativeEndian.PutUint16(b[2:4], uint16(a.Index))
4650
}
@@ -62,7 +66,7 @@ func parseLinkAddr(b []byte) (Addr, error) {
6266
if len(b) < 8 {
6367
return nil, errInvalidAddr
6468
}
65-
_, a, err := parseKernelLinkAddr(sysAF_LINK, b[4:])
69+
_, a, err := parseKernelLinkAddr(unix.AF_LINK, b[4:])
6670
if err != nil {
6771
return nil, err
6872
}
@@ -122,7 +126,7 @@ type Inet4Addr struct {
122126
}
123127

124128
// Family implements the Family method of Addr interface.
125-
func (a *Inet4Addr) Family() int { return sysAF_INET }
129+
func (a *Inet4Addr) Family() int { return unix.AF_INET }
126130

127131
func (a *Inet4Addr) lenAndSpace() (int, int) {
128132
return sizeofSockaddrInet, roundup(sizeofSockaddrInet)
@@ -134,7 +138,7 @@ func (a *Inet4Addr) marshal(b []byte) (int, error) {
134138
return 0, errShortBuffer
135139
}
136140
b[0] = byte(l)
137-
b[1] = sysAF_INET
141+
b[1] = unix.AF_INET
138142
copy(b[4:8], a.IP[:])
139143
return ll, nil
140144
}
@@ -146,7 +150,7 @@ type Inet6Addr struct {
146150
}
147151

148152
// Family implements the Family method of Addr interface.
149-
func (a *Inet6Addr) Family() int { return sysAF_INET6 }
153+
func (a *Inet6Addr) Family() int { return unix.AF_INET6 }
150154

151155
func (a *Inet6Addr) lenAndSpace() (int, int) {
152156
return sizeofSockaddrInet6, roundup(sizeofSockaddrInet6)
@@ -158,7 +162,7 @@ func (a *Inet6Addr) marshal(b []byte) (int, error) {
158162
return 0, errShortBuffer
159163
}
160164
b[0] = byte(l)
161-
b[1] = sysAF_INET6
165+
b[1] = unix.AF_INET6
162166
copy(b[8:24], a.IP[:])
163167
if a.ZoneID > 0 {
164168
nativeEndian.PutUint32(b[24:28], uint32(a.ZoneID))
@@ -169,14 +173,14 @@ func (a *Inet6Addr) marshal(b []byte) (int, error) {
169173
// parseInetAddr parses b as an internet address for IPv4 or IPv6.
170174
func parseInetAddr(af int, b []byte) (Addr, error) {
171175
switch af {
172-
case sysAF_INET:
176+
case unix.AF_INET:
173177
if len(b) < sizeofSockaddrInet {
174178
return nil, errInvalidAddr
175179
}
176180
a := &Inet4Addr{}
177181
copy(a.IP[:], b[4:8])
178182
return a, nil
179-
case sysAF_INET6:
183+
case unix.AF_INET6:
180184
if len(b) < sizeofSockaddrInet6 {
181185
return nil, errInvalidAddr
182186
}
@@ -245,7 +249,7 @@ func parseKernelInetAddr(af int, b []byte) (int, Addr, error) {
245249
a := &Inet6Addr{}
246250
copy(a.IP[:], b[off6:off6+16])
247251
return int(b[0]), a, nil
248-
case af == sysAF_INET6:
252+
case af == unix.AF_INET6:
249253
a := &Inet6Addr{}
250254
if l-1 < off6 {
251255
copy(a.IP[:], b[1:l])
@@ -365,15 +369,15 @@ func marshalAddrs(b []byte, as []Addr) (uint, error) {
365369
}
366370

367371
func parseAddrs(attrs uint, fn func(int, []byte) (int, Addr, error), b []byte) ([]Addr, error) {
368-
var as [sysRTAX_MAX]Addr
369-
af := int(sysAF_UNSPEC)
370-
for i := uint(0); i < sysRTAX_MAX && len(b) >= roundup(0); i++ {
372+
var as [unix.RTAX_MAX]Addr
373+
af := int(unix.AF_UNSPEC)
374+
for i := uint(0); i < unix.RTAX_MAX && len(b) >= roundup(0); i++ {
371375
if attrs&(1<<i) == 0 {
372376
continue
373377
}
374-
if i <= sysRTAX_BRD {
378+
if i <= unix.RTAX_BRD {
375379
switch b[1] {
376-
case sysAF_LINK:
380+
case unix.AF_LINK:
377381
a, err := parseLinkAddr(b)
378382
if err != nil {
379383
return nil, err
@@ -384,7 +388,7 @@ func parseAddrs(attrs uint, fn func(int, []byte) (int, Addr, error), b []byte) (
384388
return nil, errMessageTooShort
385389
}
386390
b = b[l:]
387-
case sysAF_INET, sysAF_INET6:
391+
case unix.AF_INET, unix.AF_INET6:
388392
af = int(b[1])
389393
a, err := parseInetAddr(af, b)
390394
if err != nil {

Diff for: route/address_darwin_test.go

+3-1
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,8 @@ package route
77
import (
88
"reflect"
99
"testing"
10+
11+
"golang.org/x/sys/unix"
1012
)
1113

1214
type parseAddrsOnDarwinTest struct {
@@ -18,7 +20,7 @@ type parseAddrsOnDarwinTest struct {
1820

1921
var parseAddrsOnDarwinLittleEndianTests = []parseAddrsOnDarwinTest{
2022
{
21-
sysRTA_DST | sysRTA_GATEWAY | sysRTA_NETMASK,
23+
unix.RTA_DST | unix.RTA_GATEWAY | unix.RTA_NETMASK,
2224
parseKernelInetAddr,
2325
[]byte{
2426
0x10, 0x2, 0x0, 0x0, 0xc0, 0xa8, 0x56, 0x0,

Diff for: route/address_test.go

+4-2
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,8 @@ package route
1010
import (
1111
"reflect"
1212
"testing"
13+
14+
"golang.org/x/sys/unix"
1315
)
1416

1517
type parseAddrsTest struct {
@@ -21,7 +23,7 @@ type parseAddrsTest struct {
2123

2224
var parseAddrsLittleEndianTests = []parseAddrsTest{
2325
{
24-
sysRTA_DST | sysRTA_GATEWAY | sysRTA_NETMASK | sysRTA_BRD,
26+
unix.RTA_DST | unix.RTA_GATEWAY | unix.RTA_NETMASK | unix.RTA_BRD,
2527
parseKernelInetAddr,
2628
[]byte{
2729
0x38, 0x12, 0x0, 0x0, 0xff, 0xff, 0xff, 0x0,
@@ -58,7 +60,7 @@ var parseAddrsLittleEndianTests = []parseAddrsTest{
5860
},
5961
},
6062
{
61-
sysRTA_NETMASK | sysRTA_IFP | sysRTA_IFA,
63+
unix.RTA_NETMASK | unix.RTA_IFP | unix.RTA_IFA,
6264
parseKernelInetAddr,
6365
[]byte{
6466
0x7, 0x0, 0x0, 0x0, 0xff, 0xff, 0xff, 0x0,

Diff for: route/defs_darwin.go

-77
Original file line numberDiff line numberDiff line change
@@ -19,83 +19,6 @@ package route
1919
*/
2020
import "C"
2121

22-
const (
23-
sysAF_UNSPEC = C.AF_UNSPEC
24-
sysAF_INET = C.AF_INET
25-
sysAF_ROUTE = C.AF_ROUTE
26-
sysAF_LINK = C.AF_LINK
27-
sysAF_INET6 = C.AF_INET6
28-
29-
sysSOCK_RAW = C.SOCK_RAW
30-
31-
sysNET_RT_DUMP = C.NET_RT_DUMP
32-
sysNET_RT_FLAGS = C.NET_RT_FLAGS
33-
sysNET_RT_IFLIST = C.NET_RT_IFLIST
34-
sysNET_RT_STAT = C.NET_RT_STAT
35-
sysNET_RT_TRASH = C.NET_RT_TRASH
36-
sysNET_RT_IFLIST2 = C.NET_RT_IFLIST2
37-
sysNET_RT_DUMP2 = C.NET_RT_DUMP2
38-
sysNET_RT_MAXID = C.NET_RT_MAXID
39-
)
40-
41-
const (
42-
sysCTL_MAXNAME = C.CTL_MAXNAME
43-
44-
sysCTL_UNSPEC = C.CTL_UNSPEC
45-
sysCTL_KERN = C.CTL_KERN
46-
sysCTL_VM = C.CTL_VM
47-
sysCTL_VFS = C.CTL_VFS
48-
sysCTL_NET = C.CTL_NET
49-
sysCTL_DEBUG = C.CTL_DEBUG
50-
sysCTL_HW = C.CTL_HW
51-
sysCTL_MACHDEP = C.CTL_MACHDEP
52-
sysCTL_USER = C.CTL_USER
53-
sysCTL_MAXID = C.CTL_MAXID
54-
)
55-
56-
const (
57-
sysRTM_VERSION = C.RTM_VERSION
58-
59-
sysRTM_ADD = C.RTM_ADD
60-
sysRTM_DELETE = C.RTM_DELETE
61-
sysRTM_CHANGE = C.RTM_CHANGE
62-
sysRTM_GET = C.RTM_GET
63-
sysRTM_LOSING = C.RTM_LOSING
64-
sysRTM_REDIRECT = C.RTM_REDIRECT
65-
sysRTM_MISS = C.RTM_MISS
66-
sysRTM_LOCK = C.RTM_LOCK
67-
sysRTM_OLDADD = C.RTM_OLDADD
68-
sysRTM_OLDDEL = C.RTM_OLDDEL
69-
sysRTM_RESOLVE = C.RTM_RESOLVE
70-
sysRTM_NEWADDR = C.RTM_NEWADDR
71-
sysRTM_DELADDR = C.RTM_DELADDR
72-
sysRTM_IFINFO = C.RTM_IFINFO
73-
sysRTM_NEWMADDR = C.RTM_NEWMADDR
74-
sysRTM_DELMADDR = C.RTM_DELMADDR
75-
sysRTM_IFINFO2 = C.RTM_IFINFO2
76-
sysRTM_NEWMADDR2 = C.RTM_NEWMADDR2
77-
sysRTM_GET2 = C.RTM_GET2
78-
79-
sysRTA_DST = C.RTA_DST
80-
sysRTA_GATEWAY = C.RTA_GATEWAY
81-
sysRTA_NETMASK = C.RTA_NETMASK
82-
sysRTA_GENMASK = C.RTA_GENMASK
83-
sysRTA_IFP = C.RTA_IFP
84-
sysRTA_IFA = C.RTA_IFA
85-
sysRTA_AUTHOR = C.RTA_AUTHOR
86-
sysRTA_BRD = C.RTA_BRD
87-
88-
sysRTAX_DST = C.RTAX_DST
89-
sysRTAX_GATEWAY = C.RTAX_GATEWAY
90-
sysRTAX_NETMASK = C.RTAX_NETMASK
91-
sysRTAX_GENMASK = C.RTAX_GENMASK
92-
sysRTAX_IFP = C.RTAX_IFP
93-
sysRTAX_IFA = C.RTAX_IFA
94-
sysRTAX_AUTHOR = C.RTAX_AUTHOR
95-
sysRTAX_BRD = C.RTAX_BRD
96-
sysRTAX_MAX = C.RTAX_MAX
97-
)
98-
9922
const (
10023
sizeofIfMsghdrDarwin15 = C.sizeof_struct_if_msghdr
10124
sizeofIfaMsghdrDarwin15 = C.sizeof_struct_ifa_msghdr

Diff for: route/defs_dragonfly.go

-78
Original file line numberDiff line numberDiff line change
@@ -40,84 +40,6 @@ struct ifa_msghdr_dfly58 {
4040
*/
4141
import "C"
4242

43-
const (
44-
sysAF_UNSPEC = C.AF_UNSPEC
45-
sysAF_INET = C.AF_INET
46-
sysAF_ROUTE = C.AF_ROUTE
47-
sysAF_LINK = C.AF_LINK
48-
sysAF_INET6 = C.AF_INET6
49-
50-
sysSOCK_RAW = C.SOCK_RAW
51-
52-
sysNET_RT_DUMP = C.NET_RT_DUMP
53-
sysNET_RT_FLAGS = C.NET_RT_FLAGS
54-
sysNET_RT_IFLIST = C.NET_RT_IFLIST
55-
sysNET_RT_MAXID = C.NET_RT_MAXID
56-
)
57-
58-
const (
59-
sysCTL_MAXNAME = C.CTL_MAXNAME
60-
61-
sysCTL_UNSPEC = C.CTL_UNSPEC
62-
sysCTL_KERN = C.CTL_KERN
63-
sysCTL_VM = C.CTL_VM
64-
sysCTL_VFS = C.CTL_VFS
65-
sysCTL_NET = C.CTL_NET
66-
sysCTL_DEBUG = C.CTL_DEBUG
67-
sysCTL_HW = C.CTL_HW
68-
sysCTL_MACHDEP = C.CTL_MACHDEP
69-
sysCTL_USER = C.CTL_USER
70-
sysCTL_P1003_1B = C.CTL_P1003_1B
71-
sysCTL_LWKT = C.CTL_LWKT
72-
sysCTL_MAXID = C.CTL_MAXID
73-
)
74-
75-
const (
76-
sysRTM_VERSION = C.RTM_VERSION
77-
78-
sysRTM_ADD = C.RTM_ADD
79-
sysRTM_DELETE = C.RTM_DELETE
80-
sysRTM_CHANGE = C.RTM_CHANGE
81-
sysRTM_GET = C.RTM_GET
82-
sysRTM_LOSING = C.RTM_LOSING
83-
sysRTM_REDIRECT = C.RTM_REDIRECT
84-
sysRTM_MISS = C.RTM_MISS
85-
sysRTM_LOCK = C.RTM_LOCK
86-
sysRTM_RESOLVE = C.RTM_RESOLVE
87-
sysRTM_NEWADDR = C.RTM_NEWADDR
88-
sysRTM_DELADDR = C.RTM_DELADDR
89-
sysRTM_IFINFO = C.RTM_IFINFO
90-
sysRTM_NEWMADDR = C.RTM_NEWMADDR
91-
sysRTM_DELMADDR = C.RTM_DELMADDR
92-
sysRTM_IFANNOUNCE = C.RTM_IFANNOUNCE
93-
sysRTM_IEEE80211 = C.RTM_IEEE80211
94-
95-
sysRTA_DST = C.RTA_DST
96-
sysRTA_GATEWAY = C.RTA_GATEWAY
97-
sysRTA_NETMASK = C.RTA_NETMASK
98-
sysRTA_GENMASK = C.RTA_GENMASK
99-
sysRTA_IFP = C.RTA_IFP
100-
sysRTA_IFA = C.RTA_IFA
101-
sysRTA_AUTHOR = C.RTA_AUTHOR
102-
sysRTA_BRD = C.RTA_BRD
103-
sysRTA_MPLS1 = C.RTA_MPLS1
104-
sysRTA_MPLS2 = C.RTA_MPLS2
105-
sysRTA_MPLS3 = C.RTA_MPLS3
106-
107-
sysRTAX_DST = C.RTAX_DST
108-
sysRTAX_GATEWAY = C.RTAX_GATEWAY
109-
sysRTAX_NETMASK = C.RTAX_NETMASK
110-
sysRTAX_GENMASK = C.RTAX_GENMASK
111-
sysRTAX_IFP = C.RTAX_IFP
112-
sysRTAX_IFA = C.RTAX_IFA
113-
sysRTAX_AUTHOR = C.RTAX_AUTHOR
114-
sysRTAX_BRD = C.RTAX_BRD
115-
sysRTAX_MPLS1 = C.RTAX_MPLS1
116-
sysRTAX_MPLS2 = C.RTAX_MPLS2
117-
sysRTAX_MPLS3 = C.RTAX_MPLS3
118-
sysRTAX_MAX = C.RTAX_MAX
119-
)
120-
12143
const (
12244
sizeofIfMsghdrDragonFlyBSD4 = C.sizeof_struct_if_msghdr
12345
sizeofIfaMsghdrDragonFlyBSD4 = C.sizeof_struct_ifa_msghdr_dfly4

0 commit comments

Comments
 (0)