You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cocoon.apache.org by Antonio Gallardo <ag...@agssa.net> on 2005/03/09 23:22:42 UTC

Java 1.3 issue in portal block.

Hi:

Please check this code it does not compile in java 1.3:

/home/agallardo/svn/cocoon-2.1/src/blocks/portal/java/org/apache/cocoon/portal/tools/service/UserrightsService.java:124:
cannot resolve symbol
symbol  : constructor RuntimeException  (java.io.IOException)
location: class java.lang.RuntimeException
            throw new RuntimeException(e);
                  ^

Please to portal gurus to review why we need here the "new". In my view
this is not needed, but please confirm I can remove the "new".

Best Regards,

Antonio Gallardo.

Re: Java 1.3 issue in portal block.

Posted by Vadim Gritsenko <va...@reverycodes.com>.
Antonio Gallardo wrote:
> throw new CascadingRuntimeException(e.getMessage(), e);
> 
> This will be OK?

Perfect.

:-)

Vadim


> Best Regards,
> 
> Antonio Gallardo
> 
> On Mie, 9 de Marzo de 2005, 16:30, Vadim Gritsenko dijo:
> 
>>Antonio Gallardo wrote:
>>
>>>Hi:
>>>
>>>Please check this code it does not compile in java 1.3:
>>>
>>>/home/agallardo/svn/cocoon-2.1/src/blocks/portal/java/org/apache/cocoon/portal/tools/service/UserrightsService.java:124:
>>>cannot resolve symbol
>>>symbol  : constructor RuntimeException  (java.io.IOException)
>>>location: class java.lang.RuntimeException
>>>            throw new RuntimeException(e);
>>>                  ^
>>
>>Use CascadingRuntimeException instead.
>>
>>
>>
>>>Please to portal gurus to review why we need here the "new". In my view
>>>this is not needed, but please confirm I can remove the "new".
>>
>>They want to wrap IOException into RuntimeException.
>>
>>Vadim

Re: Java 1.3 issue in portal block.

Posted by Antonio Gallardo <ag...@agssa.net>.
throw new CascadingRuntimeException(e.getMessage(), e);

This will be OK?

Best Regards,

Antonio Gallardo

On Mie, 9 de Marzo de 2005, 16:30, Vadim Gritsenko dijo:
> Antonio Gallardo wrote:
>> Hi:
>>
>> Please check this code it does not compile in java 1.3:
>>
>> /home/agallardo/svn/cocoon-2.1/src/blocks/portal/java/org/apache/cocoon/portal/tools/service/UserrightsService.java:124:
>> cannot resolve symbol
>> symbol  : constructor RuntimeException  (java.io.IOException)
>> location: class java.lang.RuntimeException
>>             throw new RuntimeException(e);
>>                   ^
>
> Use CascadingRuntimeException instead.
>
>
>> Please to portal gurus to review why we need here the "new". In my view
>> this is not needed, but please confirm I can remove the "new".
>
> They want to wrap IOException into RuntimeException.
>
> Vadim
>


Re: Java 1.3 issue in portal block.

Posted by Antonio Gallardo <ag...@agssa.net>.
Thanks Vadim. Doing the change now in 2.1.7-dev. It will stay as is in 2.2
trunk => the minimal Java for 2.2 is 1.4

Best Regards,

Antonio Gallardo.

On Mie, 9 de Marzo de 2005, 16:30, Vadim Gritsenko dijo:
> Antonio Gallardo wrote:
>> Hi:
>>
>> Please check this code it does not compile in java 1.3:
>>
>> /home/agallardo/svn/cocoon-2.1/src/blocks/portal/java/org/apache/cocoon/portal/tools/service/UserrightsService.java:124:
>> cannot resolve symbol
>> symbol  : constructor RuntimeException  (java.io.IOException)
>> location: class java.lang.RuntimeException
>>             throw new RuntimeException(e);
>>                   ^
>
> Use CascadingRuntimeException instead.
>
>
>> Please to portal gurus to review why we need here the "new". In my view
>> this is not needed, but please confirm I can remove the "new".
>
> They want to wrap IOException into RuntimeException.
>
> Vadim
>


Re: Java 1.3 issue in portal block.

Posted by Vadim Gritsenko <va...@reverycodes.com>.
Antonio Gallardo wrote:
> Hi:
> 
> Please check this code it does not compile in java 1.3:
> 
> /home/agallardo/svn/cocoon-2.1/src/blocks/portal/java/org/apache/cocoon/portal/tools/service/UserrightsService.java:124:
> cannot resolve symbol
> symbol  : constructor RuntimeException  (java.io.IOException)
> location: class java.lang.RuntimeException
>             throw new RuntimeException(e);
>                   ^

Use CascadingRuntimeException instead.


> Please to portal gurus to review why we need here the "new". In my view
> this is not needed, but please confirm I can remove the "new".

They want to wrap IOException into RuntimeException.

Vadim