FAQ
Reviewers: golang-dev_googlegroups.com,

Message:
Hello golang-dev@googlegroups.com,

I'd like you to review this change to
https://go.googlecode.com/hg/


Description:
test: move map delete test to nil.go

No need for a separate test for this.

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

Affected files:
R test/fixedbugs/issue4535.go
M test/nil.go


Index: test/fixedbugs/issue4535.go
===================================================================
deleted file mode 100644
--- a/test/fixedbugs/issue4535.go
+++ /dev/null
@@ -1,12 +0,0 @@
-// run
-
-// Copyright 2012 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 main
-
-func main() {
- var m map[int]int
- delete(m, 0)
-}
Index: test/nil.go
===================================================================
--- a/test/nil.go
+++ b/test/nil.go
@@ -151,6 +151,8 @@
shouldPanic(func() {
m[2] = 3
})
+ // can delete (non-existent) entries
+ delete(m, 2)
}

// nil slice

Search Discussions

Related Discussions

Discussion Navigation
viewthread | post
Discussion Overview
groupgolang-dev @
categoriesgo
postedDec 13, '12 at 7:03p
activeDec 13, '12 at 8:17p
posts10
users3
websitegolang.org

3 users in discussion

Iant: 6 posts Minux Ma: 3 posts Russ Cox: 1 post

People

Translate

site design / logo © 2022 Grokbase