Author: sebb
Date: Tue Nov 23 12:43:09 2010
New Revision: 1038086
URL: http://svn.apache.org/viewvc?rev=1038086&view=rev
Log:
Rename private file field to avoid potential confusion with file fields in
subclasses
Modified:
commons/proper/vfs/trunk/core/src/main/java/org/apache/commons/vfs2/provider/DefaultFileContent.java
Modified:
commons/proper/vfs/trunk/core/src/main/java/org/apache/commons/vfs2/provider/DefaultFileContent.java
URL:
http://svn.apache.org/viewvc/commons/proper/vfs/trunk/core/src/main/java/org/apache/commons/vfs2/provider/DefaultFileContent.java?rev=1038086&r1=1038085&r2=1038086&view=diff
==============================================================================
---
commons/proper/vfs/trunk/core/src/main/java/org/apache/commons/vfs2/provider/DefaultFileContent.java
(original)
+++
commons/proper/vfs/trunk/core/src/main/java/org/apache/commons/vfs2/provider/DefaultFileContent.java
Tue Nov 23 12:43:09 2010
@@ -53,7 +53,7 @@ public final class DefaultFileContent im
static final int STATE_CLOSED = 0;
static final int STATE_OPENED = 1;
- private final AbstractFileObject file;
+ private final AbstractFileObject fileObject;
private Map<String, Object> attrs;
private Map<String, Object> roAttrs;
private FileContentInfo fileContentInfo;
@@ -69,7 +69,7 @@ public final class DefaultFileContent im
public DefaultFileContent(final AbstractFileObject file, final
FileContentInfoFactory fileContentInfoFactory)
{
- this.file = file;
+ this.fileObject = file;
this.fileContentInfoFactory = fileContentInfoFactory;
}
@@ -90,7 +90,7 @@ public final class DefaultFileContent im
{
openStreams++;
}
- ((AbstractFileSystem) file.getFileSystem()).streamOpened();
+ ((AbstractFileSystem) fileObject.getFileSystem()).streamOpened();
}
void streamClosed()
@@ -102,11 +102,11 @@ public final class DefaultFileContent im
openStreams--;
if (openStreams < 1)
{
- file.notifyAllStreamsClosed();
+ fileObject.notifyAllStreamsClosed();
}
}
}
- ((AbstractFileSystem) file.getFileSystem()).streamClosed();
+ ((AbstractFileSystem) fileObject.getFileSystem()).streamClosed();
}
/**
@@ -115,7 +115,7 @@ public final class DefaultFileContent im
*/
public FileObject getFile()
{
- return file;
+ return fileObject;
}
/**
@@ -126,9 +126,9 @@ public final class DefaultFileContent im
public long getSize() throws FileSystemException
{
// Do some checking
- if (!file.getType().hasContent())
+ if (!fileObject.getType().hasContent())
{
- throw new
FileSystemException("vfs.provider/get-size-not-file.error", file);
+ throw new
FileSystemException("vfs.provider/get-size-not-file.error", fileObject);
}
/*
if (getThreadData().getState() == STATE_WRITING ||
getThreadData().getState() == STATE_RANDOM_ACCESS)
@@ -140,11 +140,11 @@ public final class DefaultFileContent im
try
{
// Get the size
- return file.doGetContentSize();
+ return fileObject.doGetContentSize();
}
catch (final Exception exc)
{
- throw new FileSystemException("vfs.provider/get-size.error", new
Object[]{file}, exc);
+ throw new FileSystemException("vfs.provider/get-size.error", new
Object[]{fileObject}, exc);
}
}
@@ -161,17 +161,17 @@ public final class DefaultFileContent im
throw new
FileSystemException("vfs.provider/get-last-modified-writing.error", file);
}
*/
- if (!file.getType().hasAttributes())
+ if (!fileObject.getType().hasAttributes())
{
- throw new
FileSystemException("vfs.provider/get-last-modified-no-exist.error", file);
+ throw new
FileSystemException("vfs.provider/get-last-modified-no-exist.error",
fileObject);
}
try
{
- return file.doGetLastModifiedTime();
+ return fileObject.doGetLastModifiedTime();
}
catch (final Exception e)
{
- throw new
FileSystemException("vfs.provider/get-last-modified.error", file, e);
+ throw new
FileSystemException("vfs.provider/get-last-modified.error", fileObject, e);
}
}
@@ -188,20 +188,20 @@ public final class DefaultFileContent im
throw new
FileSystemException("vfs.provider/set-last-modified-writing.error", file);
}
*/
- if (!file.getType().hasAttributes())
+ if (!fileObject.getType().hasAttributes())
{
- throw new
FileSystemException("vfs.provider/set-last-modified-no-exist.error", file);
+ throw new
FileSystemException("vfs.provider/set-last-modified-no-exist.error",
fileObject);
}
try
{
- if (!file.doSetLastModifiedTime(modTime))
+ if (!fileObject.doSetLastModifiedTime(modTime))
{
- throw new
FileSystemException("vfs.provider/set-last-modified.error", file);
+ throw new
FileSystemException("vfs.provider/set-last-modified.error", fileObject);
}
}
catch (final Exception e)
{
- throw new
FileSystemException("vfs.provider/set-last-modified.error", file, e);
+ throw new
FileSystemException("vfs.provider/set-last-modified.error", fileObject, e);
}
}
@@ -214,9 +214,9 @@ public final class DefaultFileContent im
*/
public boolean hasAttribute(final String attrName) throws
FileSystemException
{
- if (!file.getType().hasAttributes())
+ if (!fileObject.getType().hasAttributes())
{
- throw new
FileSystemException("vfs.provider/exists-attributes-no-exist.error", file);
+ throw new
FileSystemException("vfs.provider/exists-attributes-no-exist.error",
fileObject);
}
getAttributes();
return attrs.containsKey(attrName);
@@ -229,9 +229,9 @@ public final class DefaultFileContent im
*/
public Map<String, Object> getAttributes() throws FileSystemException
{
- if (!file.getType().hasAttributes())
+ if (!fileObject.getType().hasAttributes())
{
- throw new
FileSystemException("vfs.provider/get-attributes-no-exist.error", file);
+ throw new
FileSystemException("vfs.provider/get-attributes-no-exist.error", fileObject);
}
if (resetAttributes || roAttrs == null)
{
@@ -239,14 +239,14 @@ public final class DefaultFileContent im
{
synchronized (this)
{
- attrs = file.doGetAttributes();
+ attrs = fileObject.doGetAttributes();
roAttrs = Collections.unmodifiableMap(attrs);
resetAttributes = false;
}
}
catch (final Exception e)
{
- throw new
FileSystemException("vfs.provider/get-attributes.error", file, e);
+ throw new
FileSystemException("vfs.provider/get-attributes.error", fileObject, e);
}
}
return roAttrs;
@@ -296,17 +296,17 @@ public final class DefaultFileContent im
public void setAttribute(final String attrName, final Object value)
throws FileSystemException
{
- if (!file.getType().hasAttributes())
+ if (!fileObject.getType().hasAttributes())
{
- throw new
FileSystemException("vfs.provider/set-attribute-no-exist.error", new
Object[]{attrName, file});
+ throw new
FileSystemException("vfs.provider/set-attribute-no-exist.error", new
Object[]{attrName, fileObject});
}
try
{
- file.doSetAttribute(attrName, value);
+ fileObject.doSetAttribute(attrName, value);
}
catch (final Exception e)
{
- throw new FileSystemException("vfs.provider/set-attribute.error",
new Object[]{attrName, file}, e);
+ throw new FileSystemException("vfs.provider/set-attribute.error",
new Object[]{attrName, fileObject}, e);
}
if (attrs != null)
@@ -323,18 +323,18 @@ public final class DefaultFileContent im
*/
public void removeAttribute(final String attrName) throws
FileSystemException
{
- if (!file.getType().hasAttributes())
+ if (!fileObject.getType().hasAttributes())
{
- throw new
FileSystemException("vfs.provider/remove-attribute-no-exist.error", file);
+ throw new
FileSystemException("vfs.provider/remove-attribute-no-exist.error", fileObject);
}
try
{
- file.doRemoveAttribute(attrName);
+ fileObject.doRemoveAttribute(attrName);
}
catch (final Exception e)
{
- throw new
FileSystemException("vfs.provider/remove-attribute.error", new
Object[]{attrName, file}, e);
+ throw new
FileSystemException("vfs.provider/remove-attribute.error", new
Object[]{attrName, fileObject}, e);
}
if (attrs != null)
@@ -350,9 +350,9 @@ public final class DefaultFileContent im
*/
public Certificate[] getCertificates() throws FileSystemException
{
- if (!file.exists())
+ if (!fileObject.exists())
{
- throw new
FileSystemException("vfs.provider/get-certificates-no-exist.error", file);
+ throw new
FileSystemException("vfs.provider/get-certificates-no-exist.error", fileObject);
}
/*
if (getThreadData().getState() == STATE_WRITING ||
getThreadData().getState() == STATE_RANDOM_ACCESS)
@@ -363,7 +363,7 @@ public final class DefaultFileContent im
try
{
- final Certificate[] certs = file.doGetCertificates();
+ final Certificate[] certs = fileObject.doGetCertificates();
if (certs != null)
{
return certs;
@@ -375,7 +375,7 @@ public final class DefaultFileContent im
}
catch (final Exception e)
{
- throw new
FileSystemException("vfs.provider/get-certificates.error", file, e);
+ throw new
FileSystemException("vfs.provider/get-certificates.error", fileObject, e);
}
}
@@ -394,9 +394,9 @@ public final class DefaultFileContent im
*/
// Get the raw input stream
- final InputStream instr = file.getInputStream();
+ final InputStream instr = fileObject.getInputStream();
- final InputStream wrappedInstr = new FileContentInputStream(file,
instr);
+ final InputStream wrappedInstr = new
FileContentInputStream(fileObject, instr);
this.getThreadData().addInstr(wrappedInstr);
streamOpened();
@@ -422,9 +422,9 @@ public final class DefaultFileContent im
*/
// Get the content
- final RandomAccessContent rastr = file.getRandomAccessContent(mode);
+ final RandomAccessContent rastr =
fileObject.getRandomAccessContent(mode);
- FileRandomAccessContent rac = new FileRandomAccessContent(file, rastr);
+ FileRandomAccessContent rac = new FileRandomAccessContent(fileObject,
rastr);
this.getThreadData().addRastr(rac);
streamOpened();
@@ -455,14 +455,14 @@ public final class DefaultFileContent im
*/
if (this.getThreadData().getOutstr() != null)
{
- throw new FileSystemException("vfs.provider/write-in-use.error",
file);
+ throw new FileSystemException("vfs.provider/write-in-use.error",
fileObject);
}
// Get the raw output stream
- final OutputStream outstr = file.getOutputStream(bAppend);
+ final OutputStream outstr = fileObject.getOutputStream(bAppend);
// Create wrapper
- this.getThreadData().setOutstr(new FileContentOutputStream(file,
outstr));
+ this.getThreadData().setOutstr(new FileContentOutputStream(fileObject,
outstr));
streamOpened();
// setState(STATE_OPENED);
@@ -546,7 +546,7 @@ public final class DefaultFileContent im
this.getThreadData().setOutstr(null);
// setState(STATE_CLOSED);
- file.endOutput();
+ fileObject.endOutput();
}
/*