Improve error reporting of ImmutableMap

[gissuebot]

Original issue created by Ash2kk on 2012-05-02 at 11:07 AM

---

It can be easier to debug a little if Guava provided key of the value in the error message in ImmutableMap.entryOf() (and other such places).

java.lang.NullPointerException: null value
        at com.google.common.base.Preconditions.checkNotNull(Preconditions.java:208)
        at com.google.common.collect.ImmutableMap.entryOf(ImmutableMap.java:144)
        at com.google.common.collect.ImmutableMap.copyOf(ImmutableMap.java:292)

[gissuebot]

Original comment posted by wasserman.louis on 2012-05-02 at 02:08 PM

---

That particular change is easy enough, I think -- we just add a message to checkNotNull.

---

Labels: Type-Defect, Package-Collect

[gissuebot]

Original comment posted by gak@google.com on 2012-05-10 at 06:44 PM

---

(No comment entered for this change.)

---

Status: Fixed
Labels: Milestone-Release13

[gissuebot]

Original comment posted by Ash2kk on 2012-07-18 at 08:00 AM

---

public static <K, V> ImmutableMap<K, V> of(K k1, V v1) was not fixed.

[gissuebot]

Original comment posted by cpovirk@google.com on 2012-07-19 at 08:03 PM

---

(No comment entered for this change.)

---

Status: Accepted

[gissuebot]

Original comment posted by cpovirk@google.com on 2012-07-24 at 12:27 AM

---

Thanks for catching our oversight.

cc36cde

---

Status: Fixed
Labels: Milestone-Release14