Mercurial > hg > RemoteEditor > REPSessionManager
changeset 249:e44c1773d121
*** empty log message ***
| author | kono |
|---|---|
| date | Wed, 03 Sep 2008 19:35:41 +0900 |
| parents | e72e0eae1261 |
| children | 5f357ca66714 |
| files | rep/optimizers/REPCommandOptimizer.java |
| diffstat | 1 files changed, 8 insertions(+), 0 deletions(-) [+] |
line wrap: on
line diff
--- a/rep/optimizers/REPCommandOptimizer.java Wed Sep 03 18:44:08 2008 +0900 +++ b/rep/optimizers/REPCommandOptimizer.java Wed Sep 03 19:35:41 2008 +0900 @@ -7,7 +7,15 @@ public abstract class REPCommandOptimizer { + // なんで、Interfaceじゃないの? + // Interface じゃないせいで、NullOptimizer でも余計なコピーが生じている。 + // 本来、最適化はコピーしながら作業するので、余計にコピーするのは良くない。 + // まぁ、この方が安全臭いことはあるかも知れないけど。kono + // リストとその中身のREPCommandをコピーするため. finalでよい? + + // final だとoverride できないので、このコピーを外すことは誰にも出来なくなって + // しまう。kono public final List<REPCommand> optimize(List<REPCommand> cmdlist){ List<REPCommand> copied = new LinkedList<REPCommand>(); for (REPCommand cmd: cmdlist){