tree 4ef750f56d8aa47eaa762b20240436cc9929274a
parent 1795eb16a4a234d23b536d9423f9c244690569a7
author Joe Tsai <joetsai@digital-static.net> 1541637589 -0800
committer Joe Tsai <thebrokentoaster@gmail.com> 1541644444 +0000

reflect/protoreflect: clarify Clear and Remove operation on missing entries

Follow the precedence of Go maps where deletion on a key without an entry in
the map is a noop. Similarly, document that the following methods are safe
to call with entries that do not exist:
* Map.Clear
* KnownFields.Clear
* ExtensionFieldTypes.Remove

Change the implementation for each of these to match the documented behavior.

Change-Id: Ifccff9b7b03baaeffdc366d05f6286ba60e14934
Reviewed-on: https://go-review.googlesource.com/c/148317
Reviewed-by: Herbie Ong <herbie@google.com>