{
	"description": "go.tools/go/types: don't put an interface nil into the Objects map\nIf a method cannot type check, we end up with the interface variable\nm to hold <*Func, nil>. Don't put that in the map because it defeats\nthe usual != nil check.",
	"cc": [
		"<EMAIL>",
		"<EMAIL>"
	],
	"reviewers": [
		"<EMAIL>"
	],
	"messages": [
		{
			"sender": "<EMAIL>",
			"recipients": [
				"<EMAIL>",
				"<EMAIL>",
				"<EMAIL>",
				"<EMAIL>",
				"<EMAIL>"
			],
			"text": "LGTM",
			"disapproval": false,
			"date": "2013-08-06 06:02:22.603810",
			"approval": true
		},
		{
			"sender": "<EMAIL>",
			"recipients": [
				"<EMAIL>",
				"<EMAIL>",
				"<EMAIL>",
				"<EMAIL>",
				"<EMAIL>"
			],
			"text": "Hello <EMAIL>, <EMAIL> (cc: <EMAIL>),\n\nI'd like you to review this change to\nhttps://code.google.com/p/go.tools",
			"disapproval": false,
			"date": "2013-08-06 06:19:05.989340",
			"approval": false
		},
		{
			"sender": "<EMAIL>",
			"recipients": [
				"<EMAIL>",
				"<EMAIL>",
				"<EMAIL>",
				"<EMAIL>",
				"<EMAIL>"
			],
			"text": "*** Submitted as https://code.google.com/p/go/source/detail?r=6b618c627a85&repo=tools ***\n\ngo.tools/go/types: don't put an interface nil into the Objects map\nIf a method cannot type check, we end up with the interface variable\nm to hold <*Func, nil>. Don't put that in the map because it defeats\nthe usual != nil check.\n\nR=gri, dsymonds\nCC=golang-dev\nhttps://codereview.appspot.com/12506043",
			"disapproval": false,
			"date": "2013-08-06 06:19:18.078530",
			"approval": false
		},
		{
			"sender": "<EMAIL>",
			"recipients": [
				"<EMAIL>",
				"<EMAIL>",
				"<EMAIL>",
				"<EMAIL>",
				"<EMAIL>"
			],
			"text": "https://codereview.appspot.com/12506043/diff/7001/go/types/resolver.go\nFile go/types/resolver.go (right):\n\nhttps://codereview.appspot.com/12506043/diff/7001/go/types/resolver.go#newcode550\ngo/types/resolver.go:550: check.recordObject(ident, m)\nThis is incorrect. The API requires that all identifiers are reported. The correct fix is to record an actual (untyped) nil value.",
			"disapproval": false,
			"date": "2013-08-06 16:44:13.569730",
			"approval": false
		}
	],
	"owner_email": "<EMAIL>",
	"private": false,
	"base_url": "",
	"owner": "r",
	"subject": "code review 12506043: go.tools/go/types: don't put an interface nil into the ...",
	"created": "2013-08-06 06:00:24.223260",
	"patchsets": [
		1,
		4001,
		7001
	],
	"modified": "2013-08-06 16:44:13.807610",
	"closed": true,
	"issue": 12506043
}