NullablePrimitive
@Nullable should not be used for primitive types since they cannot be null

Severity
WARNING
Tags
Style
Has Fix?
REQUIRES_HUMAN_ATTENTION

The problem

Primitives can never be null.

Suppression

Suppress false positives by adding the suppression annotation @SuppressWarnings("NullablePrimitive") to the enclosing element.


Positive examples

NullablePrimitivePositiveCases.java

/*
 * Copyright 2015 The Error Prone Authors.
 *
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 *
 *     http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */

package com.google.errorprone.bugpatterns.testdata;

import javax.annotation.Nullable;

/**
 * @author sebastian.h.monte@gmail.com (Sebastian Monte)
 */
public class NullablePrimitivePositiveCases {

  // BUG: Diagnostic contains: remove
  @Nullable
  int a;

  public void method(
      // BUG: Diagnostic contains: remove
      @Nullable
      int a) {
  }

  // BUG: Diagnostic contains: remove
  @Nullable
  public int method() {
    return 0;
  }
}

Negative examples

NullablePrimitiveNegativeCases.java

/*
 * Copyright 2015 The Error Prone Authors.
 *
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 *
 *     http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */

package com.google.errorprone.bugpatterns.testdata;

import javax.annotation.Nullable;

/** @author sebastian.h.monte@gmail.com (Sebastian Monte) */
public class NullablePrimitiveNegativeCases {
  @Nullable Integer a;

  public void method(@Nullable Integer a) {}

  @Nullable
  public Integer method() {
    return Integer.valueOf(0);
  }
}