You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by ti...@apache.org on 2016/05/27 17:54:34 UTC
svn commit: r1745796 -
/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/util/SmallMap.java
Author: tilman
Date: Fri May 27 17:54:34 2016
New Revision: 1745796
URL: http://svn.apache.org/viewvc?rev=1745796&view=rev
Log:
PDFBOX-2852: make methods final that are called during construction; remove unneeded modifiers
Modified:
pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/util/SmallMap.java
Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/util/SmallMap.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/util/SmallMap.java?rev=1745796&r1=1745795&r2=1745796&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/util/SmallMap.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/util/SmallMap.java Fri May 27 17:54:34 2016
@@ -55,7 +55,7 @@ public class SmallMap<K, V> implements M
* Returns index of key within map-array or <code>-1</code>
* if key is not found (or key is <code>null</code>).
*/
- private final int findKey(Object key)
+ private int findKey(Object key)
{
if (isEmpty() || (key==null))
{
@@ -77,7 +77,7 @@ public class SmallMap<K, V> implements M
* Returns index of value within map-array or <code>-1</code>
* if value is not found (or value is <code>null</code>).
*/
- private final int findValue(Object value)
+ private int findValue(Object value)
{
if (isEmpty() || (value==null))
{
@@ -199,7 +199,7 @@ public class SmallMap<K, V> implements M
}
@Override
- public void putAll(Map<? extends K, ? extends V> otherMap)
+ public final void putAll(Map<? extends K, ? extends V> otherMap)
{
if ((mapArr == null) || (mapArr.length == 0))
{
@@ -317,7 +317,7 @@ public class SmallMap<K, V> implements M
{
private final int keyIdx;
- public SmallMapEntry(int keyInMapIdx)
+ SmallMapEntry(int keyInMapIdx)
{
keyIdx = keyInMapIdx;
}