use final field and fixed a precondition check

This commit is contained in:
Sebastian Sdorra
2014-01-09 19:27:09 +01:00
parent aa9f8f6592
commit c8db162bfa
5 changed files with 11 additions and 11 deletions

View File

@@ -199,5 +199,5 @@ public abstract class FileBasedStore<T> implements StoreBase<T>
protected File directory;
/** Field description */
private String suffix;
private final String suffix;
}

View File

@@ -105,11 +105,11 @@ public class FileBasedStoreFactory
//~--- fields ---------------------------------------------------------------
/** Field description */
private SCMContextProvider context;
private final SCMContextProvider context;
/** Field description */
private File dataDirectory;
/** Field description */
private String dataDirectoryName;
private final String dataDirectoryName;
}

View File

@@ -46,7 +46,7 @@ import java.io.OutputStream;
*
* @author Sebastian Sdorra
*/
public class FileBlob implements Blob
public final class FileBlob implements Blob
{
/**
@@ -122,8 +122,8 @@ public class FileBlob implements Blob
//~--- fields ---------------------------------------------------------------
/** Field description */
private File file;
private final File file;
/** Field description */
private String id;
private final String id;
}

View File

@@ -142,7 +142,7 @@ public class FileBlobStore extends FileBasedStore<Blob> implements BlobStore
@Override
public void remove(Blob blob)
{
Preconditions.checkNotNull("blob argument is required");
Preconditions.checkNotNull(blob, "blob argument is required");
remove(blob.getId());
}
@@ -196,6 +196,6 @@ public class FileBlobStore extends FileBasedStore<Blob> implements BlobStore
//~--- fields ---------------------------------------------------------------
/** Field description */
private KeyGenerator keyGenerator;
/** key generator */
private final KeyGenerator keyGenerator;
}

View File

@@ -98,6 +98,6 @@ public class FileBlobStoreFactory extends FileBasedStoreFactory
//~--- fields ---------------------------------------------------------------
/** Field description */
private KeyGenerator keyGenerator;
/** key generator */
private final KeyGenerator keyGenerator;
}