diff options
| author | Alex Brainman <alex.brainman@gmail.com> | 2021-04-26 17:56:10 +1000 |
|---|---|---|
| committer | Alex Brainman <alex.brainman@gmail.com> | 2021-04-27 08:36:54 +0000 |
| commit | ca8e8317be5b288163c3623f936fb234723484cc (patch) | |
| tree | a0fc4a9190f8e86119d4586de0375ff35e8b7f30 /src/runtime/testdata | |
| parent | cb34026a95f0637a0f9eadb5e1d7aa301635f943 (diff) | |
| download | go-ca8e8317be5b288163c3623f936fb234723484cc.tar.gz go-ca8e8317be5b288163c3623f936fb234723484cc.zip | |
runtime: add missing import "C" in TestLibraryCtrlHandler
CL 211139 added TestLibraryCtrlHandler. But the CL left out import "C"
line in the test file that is supposed to be build with Cgo.
While debugging issue #45638, I discovered that the DLL built during
TestLibraryCtrlHandler does not have Dummy function. Adding import "C"
makes Dummy function appear in DLL function list.
TestLibraryCtrlHandler does not actually calls Dummy function. So I
don't see how this change affects issue #45638, but still let's make
this code correct.
Updates #45638
Change-Id: Ibab8fed29ef2ae446d0815842cf0bd040a5fb943
Reviewed-on: https://go-review.googlesource.com/c/go/+/313350
Trust: Alex Brainman <alex.brainman@gmail.com>
Run-TryBot: Alex Brainman <alex.brainman@gmail.com>
TryBot-Result: Go Bot <gobot@golang.org>
Reviewed-by: Michael Knyszek <mknyszek@google.com>
Diffstat (limited to 'src/runtime/testdata')
| -rw-r--r-- | src/runtime/testdata/testwinlibsignal/dummy.go | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/runtime/testdata/testwinlibsignal/dummy.go b/src/runtime/testdata/testwinlibsignal/dummy.go index 82dfd91c93..e610f15d06 100644 --- a/src/runtime/testdata/testwinlibsignal/dummy.go +++ b/src/runtime/testdata/testwinlibsignal/dummy.go @@ -1,7 +1,10 @@ +//go:build windows // +build windows package main +import "C" + //export Dummy func Dummy() int { return 42 |