Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1397)

Issue 157147: code review 157147: cgo translates empty function arguments into void inst... (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
15 years, 3 months ago by eden.li
Modified:
15 years, 3 months ago
Reviewers:
rsc
Visibility:
Public.

Description

cgo translates empty function arguments into void instead of dying with 'unexpected type: ...'. Fixes issue 162.

Patch Set 1 #

Patch Set 2 : code review 157147: cgo translates empty function arguments into void inst... #

Unified diffs Side-by-side diffs Delta from patch set Stats (+9 lines, -0 lines) Patch
M src/cmd/cgo/gcc.go View 1 chunk +9 lines, -0 lines 0 comments Download

Messages

Total messages: 3
eden.li
Hello rsc, I'd like you to review the following change.
15 years, 3 months ago (2009-11-24 04:48:47 UTC) #1
rsc
LGTM
15 years, 3 months ago (2009-11-24 05:54:07 UTC) #2
rsc
15 years, 3 months ago (2009-11-24 06:02:17 UTC) #3
*** Submitted as http://code.google.com/p/go/source/detail?r=cb2c748233c8 ***

	cgo translates empty function arguments into void instead of dying with
'unexpected type: ...'.
  Fixes issue 162.

R=rsc
http://codereview.appspot.com/157147

Committer: Russ Cox <rsc@golang.org>
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b