diff options
author | mpociecha <michal.pociecha@gmail.com> | 2014-11-30 23:52:37 +0100 |
---|---|---|
committer | mpociecha <michal.pociecha@gmail.com> | 2014-12-05 01:21:04 +0100 |
commit | a8c43dc5caf7439dbcb47f6c25b33fb6b3ed8705 (patch) | |
tree | cca10ee42c3f7c9ad576fc657ca22f8a24f0749a /src/reflect/scala/reflect/io/VirtualFile.scala | |
parent | dfc5c1d7225163994e3bc1cf67ccbee8c4de75fc (diff) | |
download | scala-a8c43dc5caf7439dbcb47f6c25b33fb6b3ed8705.tar.gz scala-a8c43dc5caf7439dbcb47f6c25b33fb6b3ed8705.tar.bz2 scala-a8c43dc5caf7439dbcb47f6c25b33fb6b3ed8705.zip |
Cleanup and refactoring - semicolons, unused or commented out code
This commit contains some minor changes made by the way when
implementing flat classpath.
Sample JUnit test that shows that all pieces of JUnit infrastructure
work correctly now uses assert method form JUnit as it should do from
the beginning.
I removed commented out lines which were obvious to me. In the case
of less obvious commented out lines I added TODOs as someone should
look at such places some day and clean them up.
I removed also some unnecessary semicolons and unused imports.
Many string concatenations using + have been changed to string
interpolation.
There's removed unused, private walkIterator method from ZipArchive.
It seems that it was unused since this commit:
https://github.com/scala/scala/commit/9d4994b96c77d914687433586eb6d1f9e49c520f
However, I had to add an exception for the compatibility checker
because it was complaining about this change.
I made some trivial corrections/optimisations like use 'findClassFile'
method instead of 'findClass' in combination with 'binary' to find
the class file.
Diffstat (limited to 'src/reflect/scala/reflect/io/VirtualFile.scala')
-rw-r--r-- | src/reflect/scala/reflect/io/VirtualFile.scala | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/reflect/scala/reflect/io/VirtualFile.scala b/src/reflect/scala/reflect/io/VirtualFile.scala index 45f38db745..1cb4f2fe6f 100644 --- a/src/reflect/scala/reflect/io/VirtualFile.scala +++ b/src/reflect/scala/reflect/io/VirtualFile.scala @@ -75,10 +75,10 @@ class VirtualFile(val name: String, override val path: String) extends AbstractF } /** Does this abstract file denote an existing file? */ - def create() { unsupported() } + def create(): Unit = unsupported() /** Delete the underlying file or directory (recursively). */ - def delete() { unsupported() } + def delete(): Unit = unsupported() /** * Returns the abstract file in this abstract directory with the |