code review 13700045: cmd/cgo: fix build (missing file from earlier CL) (issue 13700045)

30 views
Skip to first unread message

r...@golang.org

unread,
Sep 16, 2013, 2:21:52 PM9/16/13
to golan...@googlegroups.com, golan...@googlegroups.com, re...@codereview-hr.appspotmail.com
Reviewers: golang-dev1,

Message:
Hello golang-dev (cc: golan...@googlegroups.com),

I'd like you to review this change to
https://code.google.com/p/go/


Description:
cmd/cgo: fix build (missing file from earlier CL)

Please review this at https://codereview.appspot.com/13700045/

Affected files (+23, -0 lines):
A misc/cgo/test/issue6390.go


Index: misc/cgo/test/issue6390.go
===================================================================
new file mode 100644
--- /dev/null
+++ b/misc/cgo/test/issue6390.go
@@ -0,0 +1,23 @@
+// Copyright 2013 The Go Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style
+// license that can be found in the LICENSE file.
+
+package cgotest
+
+// #include <stdlib.h>
+import "C"
+
+import "testing"
+
+func test6390(t *testing.T) {
+ p1 := C.malloc(1024)
+ if p1 == nil {
+ t.Fatalf("C.malloc(1024) returned nil")
+ }
+ p2 := C.malloc(0)
+ if p2 == nil {
+ t.Fatalf("C.malloc(0) returned nil")
+ }
+ C.free(p1)
+ C.free(p2)
+}


r...@golang.org

unread,
Sep 16, 2013, 2:21:57 PM9/16/13
to r...@golang.org, golan...@googlegroups.com, re...@codereview-hr.appspotmail.com
*** Submitted as
https://code.google.com/p/go/source/detail?r=10155bed0490 ***

cmd/cgo: fix build (missing file from earlier CL)

TBR=golang-dev
CC=golang-dev
https://codereview.appspot.com/13700045


https://codereview.appspot.com/13700045/

r...@golang.org

unread,
Sep 16, 2013, 5:39:47 PM9/16/13
to r...@golang.org, golan...@googlegroups.com, re...@codereview-hr.appspotmail.com
Reply all
Reply to author
Forward
0 new messages