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;
   }

Reply via email to