[piccolo2d-dev] Issue 130 in piccolo2d: PFrame throws IllegalComponentException

2009-10-09 Thread codesite-noreply

Updates:
Status: Verified

Comment #6 on issue 130 by allain.lalonde: PFrame throws  
IllegalComponentException
http://code.google.com/p/piccolo2d/issues/detail?id=130

Totally my bad. :(

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~-~--~~~---~--~~
Piccolo2D Developers Group: http://groups.google.com/group/piccolo2d-dev?hl=en
-~--~~~~--~~--~--~---



[piccolo2d-dev] Issue 130 in piccolo2d: PFrame throws IllegalComponentException

2009-10-09 Thread codesite-noreply

Updates:
Status: Fixed

Comment #5 on issue 130 by re...@colorado.edu: PFrame throws  
IllegalComponentException
http://code.google.com/p/piccolo2d/issues/detail?id=130

(No comment was entered for this change.)

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~-~--~~~---~--~~
Piccolo2D Developers Group: http://groups.google.com/group/piccolo2d-dev?hl=en
-~--~~~~--~~--~--~---



[piccolo2d-dev] Issue 130 in piccolo2d: PFrame throws IllegalComponentException

2009-10-09 Thread codesite-noreply

Updates:
Owner: allain.lalonde

Comment #4 on issue 130 by re...@colorado.edu: PFrame throws  
IllegalComponentException
http://code.google.com/p/piccolo2d/issues/detail?id=130

Reassigning to allain.lalonde to verify the and close.  Are there any other
situations where this bug could have been introduced recently as part of  
CheckStyle
fixes?

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~-~--~~~---~--~~
Piccolo2D Developers Group: http://groups.google.com/group/piccolo2d-dev?hl=en
-~--~~~~--~~--~--~---



[piccolo2d-dev] Issue 130 in piccolo2d: PFrame throws IllegalComponentException

2009-10-09 Thread codesite-noreply


Comment #3 on issue 130 by re...@colorado.edu: PFrame throws  
IllegalComponentException
http://code.google.com/p/piccolo2d/issues/detail?id=130

Resolution committed in r702.

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~-~--~~~---~--~~
Piccolo2D Developers Group: http://groups.google.com/group/piccolo2d-dev?hl=en
-~--~~~~--~~--~--~---



[piccolo2d-dev] Issue 130 in piccolo2d: PFrame throws IllegalComponentException

2009-10-09 Thread codesite-noreply


Comment #2 on issue 130 by re...@colorado.edu: PFrame throws  
IllegalComponentException
http://code.google.com/p/piccolo2d/issues/detail?id=130

The problematic snippet of code is:

 if (canvas == null) {
 this.canvas = new PCanvas();
 }
 else {
 this.canvas = canvas;
 }

 setContentPane(canvas);

Note that even if canvas is null, it is still being set as the content pane.

This bug was introduced in r698 "More CheckStyle Stuff"


--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~-~--~~~---~--~~
Piccolo2D Developers Group: http://groups.google.com/group/piccolo2d-dev?hl=en
-~--~~~~--~~--~--~---



[piccolo2d-dev] Issue 130 in piccolo2d: PFrame throws IllegalComponentException

2009-10-09 Thread codesite-noreply


Comment #1 on issue 130 by re...@colorado.edu: PFrame throws  
IllegalComponentException
http://code.google.com/p/piccolo2d/issues/detail?id=130

The problematic snippet of code is:

 if (canvas == null) {
 this.canvas = new PCanvas();
 }
 else {
 this.canvas = canvas;
 }

 setContentPane(canvas);

Note that even if canvas is null, it is still being set as the content pane.

This bug was introduced in r695 "Fixing FindBugs transient field warnings."

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~-~--~~~---~--~~
Piccolo2D Developers Group: http://groups.google.com/group/piccolo2d-dev?hl=en
-~--~~~~--~~--~--~---



[piccolo2d-dev] Issue 130 in piccolo2d: PFrame throws IllegalComponentException

2009-10-09 Thread codesite-noreply

Status: Accepted
Owner: re...@colorado.edu

New issue 130 by re...@colorado.edu: PFrame throws IllegalComponentException
http://code.google.com/p/piccolo2d/issues/detail?id=130

What steps will reproduce the problem?
1. Run CameraExample, or any other example that uses PFrame

What is the expected output? What do you see instead?
Expect to see the example appear, instead we obtain an Exception:

Exception in thread "main" java.awt.IllegalComponentStateException:
contentPane cannot be set to null.
at javax.swing.JRootPane.setContentPane(JRootPane.java:603)
at javax.swing.JFrame.setContentPane(JFrame.java:676)
at edu.umd.cs.piccolox.PFrame.(PFrame.java:129)
at edu.umd.cs.piccolox.PFrame.(PFrame.java:93)
at 
edu.umd.cs.piccolo.examples.CameraExample.(CameraExample.java:55)
at 
edu.umd.cs.piccolo.examples.CameraExample.(CameraExample.java:51)
at edu.umd.cs.piccolo.examples.CameraExample.main(CameraExample.java:79)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
at
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
at java.lang.reflect.Method.invoke(Method.java:597)
at com.intellij.rt.execution.application.AppMain.main(AppMain.java:90)



--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~-~--~~~---~--~~
Piccolo2D Developers Group: http://groups.google.com/group/piccolo2d-dev?hl=en
-~--~~~~--~~--~--~---