Bug in StandardStore.java (Fourth Submission!!!)

[Mark Hortman <ma...@jcafeinc.com>]

 

Index: StandardStore.java
I found a bug that when a MacroImpl.deleteObject() occurs,
it Enumerates the permissions and calls delete on them one at a time.
In StandardStore.java, the TODO said the vector elements must be cloned,
and it was right, if they are not cloned, then this bug will never
delete
the second permission.

Can someone please run this against the testsuite. I cannot for the life
of me figure out how to run the test suite against a Nightly build.

I tried to run some tests on my copy and it works fine.

Please someone commit this.... 

===================================================================
RCS file:
/home/cvspublic/jakarta-slide/src/share/org/apache/slide/store/StandardS
tore.java,v
retrieving revision 1.18
diff -r1.18 StandardStore.java
312c312
<         // TODO : The vectors elements MUST be cloned
---
>         // TODO : The vectors elements MUST be cloned for non chached
Results
318d317
<                 return permissionsVector.elements();
329d327
<                 return ((Vector)
permissionsVector.clone()).elements();
330a329
>             return ((Vector) permissionsVector.clone()).elements();

 <<StandardStore.java>>