Finally your fix is checked-in. Sorry for the delay!

Regards,
Peter
 

> -----Original Message-----
> From: Mark Hortman [mailto:mark.hortman@;jcafeinc.com]
> Sent: Friday, November 08, 2002 17:25
> To: '[EMAIL PROTECTED]'
> Subject: Bug in StandardStore.java (Fourth Submission!!!)
> 
> 
>  
> 
> 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>> 
> 
> 

--
To unsubscribe, e-mail:   <mailto:slide-dev-unsubscribe@;jakarta.apache.org>
For additional commands, e-mail: <mailto:slide-dev-help@;jakarta.apache.org>

Reply via email to