> 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.
Sorry, my words were too strong (I don't know English that well either
;-) ).
It would be really cool if you made sure the UI code still builds after
making changes (I spent a lot of time to get there). It's built separately
at the moment.
Remy
