Syntax Question in Mutation.java

[David Medinets]

In core/src/main/java/org/apache/accumulo/core/data/Mutation.java, I
see the following code:


    private List<byte[]> values;
    private int cachedValLens = -1;

  long getValueLengths() {
    if (values == null)
      return 0;

    if (cachedValLens == -1) {
      int tmpCVL = 0;
      for (byte val[] : values)
        tmpCVL += val.length;

      cachedValLens = tmpCVL;
    }

    return cachedValLens;

  }

PMD is suggesting that the line:

      for (byte val[] : values)

should be

      for (byte[] val : values)

Is this a useful change?