Revision: 3758
Author: [email protected]
Date: Sat Sep 26 18:12:27 2009
Log: Bug 940: Access to FileSystemEnvironment
http://codereview.appspot.com/125048
From http://code.google.com/p/google-caja/issues/detail?id=940
While the class FileSystemEnvironment is public, its only
constructor isn't, which means there's currently no way to derive
from it outside of the caja.plugin package. Is this by design?
It's a useful base class and it'd be nice to be able to take
advantage of it.
Similarly, it'd be pretty useful if PluginCompilerMain's private
CachingEnvironment were factored out into an independent public
class.
[email protected]
http://code.google.com/p/google-caja/source/detail?r=3758
Modified:
/trunk/src/com/google/caja/plugin/FileSystemEnvironment.java
=======================================
--- /trunk/src/com/google/caja/plugin/FileSystemEnvironment.java Fri Jul 10
16:04:28 2009
+++ /trunk/src/com/google/caja/plugin/FileSystemEnvironment.java Sat Sep 26
18:12:27 2009
@@ -31,7 +31,7 @@
public abstract class FileSystemEnvironment implements PluginEnvironment {
private final File directory;
- FileSystemEnvironment(File directory) {
+ public FileSystemEnvironment(File directory) {
this.directory = directory;
}