Hallo Jörg,
>
> class RequestContext {
> // [snip]
> @Deprecated // since 1.3
> int getContentLength();
> }
>
> // @since 1.3
> class RequestContextExt extends RequestContext {
> // @since 1.3
> long contentLength();
> }
>
should work like a charm, it will require a little more work to
propagate the new interface in the rest of the classes but we should
make Clirr happy :)
Thanks for your suggestion!
-Simo
http://people.apache.org/~simonetripodi/
http://simonetripodi.livejournal.com/
http://twitter.com/simonetripodi
http://www.99soft.org/
On Wed, Mar 13, 2013 at 9:41 AM, Jörg Schaible
<joerg.schai...@scalaris.com> wrote:
> Hi Simo,
>
> Simone Tripodi wrote:
>
>>> Adding a new method to the public RequestContext interface also creates
>>> an Clirr error, as classes implementing this interface are required to
>>> implement this method.
>>
>> yup I know it, but the alternative of suppressing this breakage is
>> pushing back FILEUPLOAD-188 and FILEUPLOAD-195.
>> Do you have better ideas?
>
> class RequestContext {
> // [snip]
> @Deprecated // since 1.3
> int getContentLength();
> }
>
> // @since 1.3
> class RequestContextExt extends RequestContext {
> // @since 1.3
> long contentLength();
> }
>
>
> And later for 2.x:
>
> class RequestContext {
> // [snip]
> // int getContentLength(); removed
> // @since 1.3 moved here in 2.0 from RequestContextEx
> long contentLength();
> }
>
> // @since 1.3
> @Deprecated // As of 2.0
> class RequestContextExt extends RequestContext {
> // compatibility only
> }
>
>
> WDYT? If 2.0 breaks compatibility significantly, we might even drop
> RequestContextEx directly.
>
> - Jörg
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
> For additional commands, e-mail: dev-h...@commons.apache.org
>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org
