logo       

[ jython-Patches-869096 ] fix for 730156: msg#00014

lang.jython.devel

Subject: [ jython-Patches-869096 ] fix for 730156

Patches item #869096, was opened at 2004-01-01 23:17
Message generated for change (Comment added) made by lycotic
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=312867&aid=869096&group_id=12867

Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Randy Brown (lycotic)
Assigned to: Nobody/Anonymous (nobody)
Summary: fix for 730156

Initial Comment:
While I was arguing with the verifier...

make sure the try: block in try:finally: doesn't use the same
local as the finally's return. This makes the verifier
unhappy.

----------------------------------------------------------------------

>Comment By: Randy Brown (lycotic)
Date: 2004-01-05 13:12

Message:
Logged In: YES
user_id=920303

Hmm; I suspect that Sun will have to lift the bytcode limit
on methods if they go through with this. I suspect most
jsps are turning jsrs back on.

Still, it shouldn't be that hard to inline the finallys; for
most cases, it'll even shorten the code. I'll look into it.

----------------------------------------------------------------------

Comment By: Samuele Pedroni (pedronis)
Date: 2004-01-04 21:53

Message:
Logged In: YES
user_id=61408

related: at the moment our generator impl does not support a
yield inside a finally clause, given that sun itself is not
using jsr anymore

http://developer.java.sun.com/developer/bugParade/bugs/4381996.html

it makes sense to do the same to achieve that and stress the
verifier less.

----------------------------------------------------------------------

You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=312867&aid=869096&group_id=12867


-------------------------------------------------------
This SF.net email is sponsored by: IBM Linux Tutorials.
Become an expert in LINUX or just sharpen your skills. Sign up for IBM's
Free Linux Tutorials. Learn everything from the bash shell to sys admin.
Click now! http://ads.osdn.com/?ad_id=1278&alloc_id=3371&op=click


<Prev in Thread] Current Thread [Next in Thread>
Google Custom Search

News | FAQ | advertise