#426 Remove redundant null checks from bimap put

Author: alicederyn

src/main/java/org/inferred/freebuilder/processor/property/BiMapProperty.java

@@ -200,12 +200,6 @@ private void addPut(SourceBuilder code) {
         putMethod(property),
         unboxedKeyType.orElse(keyType),
         unboxedValueType.orElse(valueType));
-    if (!unboxedKeyType.isPresent()) {
-      code.addLine("  %s.requireNonNull(key);", Objects.class);
-    }
-    if (!unboxedValueType.isPresent()) {
-      code.addLine("  %s.requireNonNull(value);", Objects.class);
-    }
     code.addLine("  %s.checkArgument(", Preconditions.class)
         .addLine("      !%s.containsValue(value), \"value already present: %%s\", value);",
             property.getField())

src/test/java/org/inferred/freebuilder/processor/property/BiMapSourceTest.java

@@ -90,7 +90,6 @@ public void test_guava() {
         "   * @throws NullPointerException if {@code value} is null",
         "   */",
         "  public Person.Builder putName(int key, String value) {",
-        "    Objects.requireNonNull(value);",
         "    Preconditions.checkArgument(!name.containsValue(value), \"value already present: %s\", value);",
         "    forcePutName(key, value);",
         "    return (Person.Builder) this;",