Skip to content

Commit 25899f8

Browse files
committed
Ignore go vet "possible misuse of unsafe.Pointer" in c-callbacks.go
1 parent 6f3f4ed commit 25899f8

File tree

2 files changed

+16
-1
lines changed

2 files changed

+16
-1
lines changed

build/test.sh

Lines changed: 15 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,21 @@ else
2222
fi
2323

2424
echo "******** Testing: go vet ********"
25-
go vet ./...
25+
set +e
26+
go_vet_result=$(go 2>&1 vet ./...)
27+
go_vet_rc=$?
28+
set -e
29+
30+
echo "$go_vet_result"
31+
go_vet_result_lines=$(echo "$go_vet_result" | wc -l)
32+
if [ $go_vet_rc -ne 0 ]; then
33+
if [[ $go_vet_result_lines == 2 && $go_vet_result == *objectbox/c-callbacks.go*possible\ misuse\ of\ unsafe.Pointer* ]]; then
34+
echo "Ignoring known false positive of go vet"
35+
else
36+
echo "go vet failed ($go_vet_rc)"
37+
exit $go_vet_rc
38+
fi
39+
fi
2640

2741
echo "******** Testing: go test ********"
2842
go test "$@" ./...

objectbox/c-callbacks.go

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -124,6 +124,7 @@ var cCallbackMap = make(map[cCallbackId]cCallable)
124124

125125
// The result is actually not a memory pointer, just a number. That's also how it's used in cCallbackLookup().
126126
func (cbId cCallbackId) cPtr() unsafe.Pointer {
127+
//goland:noinspection GoVetUnsafePointer
127128
return unsafe.Pointer(uintptr(cbId))
128129
}
129130

0 commit comments

Comments
 (0)