From de86073aa30c1bcd899821a16fe215533d7c4eca Mon Sep 17 00:00:00 2001
From: Mark Raynsford <code@io7m.com>
Date: Tue, 28 Oct 2014 20:46:31 +0000
Subject: Bug 1100 - Document lifecycle-heavy concept and mark some of the
 relevant methods.

---
 src/newt/classes/com/jogamp/newt/Window.java | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)

(limited to 'src/newt')

diff --git a/src/newt/classes/com/jogamp/newt/Window.java b/src/newt/classes/com/jogamp/newt/Window.java
index ec6d767fa..060eaab03 100644
--- a/src/newt/classes/com/jogamp/newt/Window.java
+++ b/src/newt/classes/com/jogamp/newt/Window.java
@@ -95,6 +95,17 @@ import javax.media.nativewindow.util.SurfaceSize;
  *   System.setProperty("newt.window.icons", "newt/data/jogamp-16x16.png newt/data/jogamp-32x32.png");
  * </pre>
  * </p>
+ * 
+ * <a name="lifecycleHeavy"><h5>Use of Lifecycle Heavy functions</h5></a>
+ * <p>
+ * Some of the methods specified here are lifecycle-heavy. That is, they are able
+ * to destroy and/or reattach resources to/from the window. Because of this, the methods
+ * are <i>not safe</i> to be called from EDT related threads. For example, it is not
+ * safe for a method in an attached {@link KeyListener} to call {@link #setFullscreen(boolean)}
+ * on a {@link Window} directly. It is safe, however, for that thread to spawn a background
+ * worker thread which calls the method directly. The documentation for individual methods 
+ * indicates whether or not they are lifecycle-heavy.
+ * </p>
  */
 public interface Window extends NativeWindow, WindowClosingProtocol, ScalableSurface {
     public static final boolean DEBUG_MOUSE_EVENT = Debug.debug("Window.MouseEvent");
@@ -174,6 +185,7 @@ public interface Window extends NativeWindow, WindowClosingProtocol, ScalableSur
      * <p>
      * The Window can be recreate via {@link #setVisible(boolean) setVisible(true)}.
      * </p>
+     * <p>This method is <a href="lifecycleHeavy">lifecycle heavy</a>.</p>
      * @see #destroy()
      * @see #setVisible(boolean)
      */
@@ -193,6 +205,7 @@ public interface Window extends NativeWindow, WindowClosingProtocol, ScalableSur
     /**
      * Calls {@link #setVisible(boolean, boolean) setVisible(true, visible)},
      * i.e. blocks until the window becomes visible.
+     * <p>This method is <a href="lifecycleHeavy">lifecycle heavy</a>.</p>
      * @see #setVisible(boolean, boolean)
      */
     void setVisible(boolean visible);
@@ -226,6 +239,7 @@ public interface Window extends NativeWindow, WindowClosingProtocol, ScalableSur
      * i.e. {@link javax.media.nativewindow.NativeWindow#getWindowHandle()} returns <code>null</code>.<br>
      * <code>setVisible(wait, true)</code> shall be repeated when the parent becomes valid.
      * </p>
+     * <p>This method is <a href="lifecycleHeavy">lifecycle heavy</a>.</p>
      */
     void setVisible(boolean wait, boolean visible);
 
@@ -470,6 +484,7 @@ public interface Window extends NativeWindow, WindowClosingProtocol, ScalableSur
      * this window is removed from it's list of children.<br>
      * In case the new parent is not null and a Window,
      * this window is added to it's list of children.<br></P>
+     * <p>This method is <a href="lifecycleHeavy">lifecycle heavy</a>.</p>
      *
      * @param newParent The new parent NativeWindow. If null, this Window becomes a top level window.
      * @param x new top-level position in window units, use -1 for default position.
@@ -485,6 +500,7 @@ public interface Window extends NativeWindow, WindowClosingProtocol, ScalableSur
      * <p>
      * Fullscreen mode is established on the {@link #getMainMonitor() main monitor}.
      * </p>
+     * <p>This method is <a href="lifecycleHeavy">lifecycle heavy</a>.</p>
      * @param fullscreen enable or disable fullscreen mode
      * @return success
      * @see #setFullscreen(List)
@@ -498,6 +514,7 @@ public interface Window extends NativeWindow, WindowClosingProtocol, ScalableSur
      * <p>
      * Disable fullscreen via {@link #setFullscreen(boolean)}.
      * </p>
+     * <p>This method is <a href="lifecycleHeavy">lifecycle heavy</a>.</p>
      * @param monitors if <code>null</code> fullscreen will be spanned across all {@link MonitorDevice}s,
      *                 otherwise across the given list of {@link MonitorDevice}.
      * @return success
-- 
cgit v1.2.3