Subject: Re: cvs commit:
jakarta-slide/src/webdav/client/src/org/apache/webdav/lib/methods
DepthSupport.java

> - Active lock is not a property. To be consistent, I think you should
create
> a Lock object similar to the Privilege object.

Yes, that's not a property.  I didn't notice it yet.  ^^;
Actually, I didn't consider of lock thingy like lockentry, lockinfo and
more.


> - This patch is not very respectful for my work. I spent a lot of time
> integrating the UI code, and this API change breaks the build. While it
> needs more work, if you change the API, you should at least make sure it
> still builds fine.

I'm sorry...  I didn't think of UI code enoughly..  :(
>From the next time, I will take care of the UI code also.

Thank you for correcting me...

Sung-Gu


Reply via email to