Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

x/net/bpf: Extension constants should all be of type Extension #20559

Closed
breml opened this issue Jun 2, 2017 · 1 comment
Closed

x/net/bpf: Extension constants should all be of type Extension #20559

breml opened this issue Jun 2, 2017 · 1 comment

Comments

@breml
Copy link
Contributor

@breml breml commented Jun 2, 2017

What version of Go are you using (go version)?

go version go1.8 linux/amd64

What operating system and processor architecture are you using (go env)?

GOHOSTARCH="amd64"
GOHOSTOS="linux"

What did you do?

package main

import (
	"fmt"

	"golang.org/x/net/bpf"
)

func main() {
	fmt.Printf("%T, %T, %T\n", bpf.ExtLen, bpf.ExtProto, bpf.ExtRand)
}

What did you expect to see?

Expect all Extension constants to be of type Extension (and not int)
The test program should print:

bpf.Extension, bpf.Extension, bpf.Extension

What did you see instead?

bpf.Extension, int, int
@gopherbot gopherbot added this to the Unreleased milestone Jun 2, 2017
@gopherbot
Copy link

@gopherbot gopherbot commented Jun 2, 2017

CL https://golang.org/cl/44710 mentions this issue.

@golang golang locked and limited conversation to collaborators Jun 2, 2018
NET12115 added a commit to NET12115/Golang-C-NET that referenced this issue Feb 28, 2022
Fixes golang/go#20559

Change-Id: Ieb794badb56c6830e32028133f67223424323ea2
Reviewed-on: https://go-review.googlesource.com/44710
Reviewed-by: Matt Layher <mdlayher@gmail.com>
Run-TryBot: Matt Layher <mdlayher@gmail.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants