<Swing Dev> Is the following code supposed to leak?

neugens.limasoftware@gmail.com neugens.limasoftware at gmail.com
Wed Aug 24 11:19:27 UTC 2011


The vm doesn't terminate.

One VM to rule them all: the instances are separated via the appcontext, but there is just one server vm running.

Cheers,
Mario
-- 
Sent from HTC Desire...

pgp key: http://subkeys.pgp.net/ PGP Key ID: 80F240CF
Fingerprint: BA39 9666 94EC 8B73 27FA  FC7C 4086 63E3 80F2 40CF

http://www.icedrobot.org

Proud GNU Classpath developer: http://www.classpath.org/
Read About us at: http://planet.classpath.org
OpenJDK: http://openjdk.java.net/projects/caciocavallo/

Please, support open standards:
http://endsoftpatents.org/


----- Reply message -----
Da: "Pavel Porvatov" <pavel.porvatov at oracle.com>
Data: mer, ago 24, 2011 12:57
Oggetto: <Swing Dev> Is the following code supposed to leak?
A: "Clemens Eisserer" <linuxhippy at gmail.com>
Cc: <swing-dev at openjdk.java.net>


Hi Clemens,
> Hi Pavel,
>
>     In common case you shouldn't remove listeners of course. But in
>     your case you must remove listeners because of editor is shared
>     between all JTextComponents (see javadoc of the
>     JTextComponent#getActions method and BasicTextUI#getEditorKit
>     implementation). So you added listeners not to a JTextComponent
>     instance, but to the shared editor.
>
>
> Thanks a lot for clarification, I wasn't sure this behaviour is 
> intended :/
> Seems I have to adopt the jfc-demo Notepad to not leak memory when 
> running on caciocavallo-web ;)
I took a look at the Notepad demo and didn't find any problems. It adds 
listeners only when the demo starts and there is no need to remove 
listeners, because the demo closes via System.exit(0). Could you please 
explain how did you get memory leak in the demo?

Regards, Pavel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/swing-dev/attachments/20110824/1fdb5a35/attachment.html>


More information about the swing-dev mailing list