3 years agoBug 1085: Fix GLJPanel regression while printing w/ invisible GLJPanel: Zero size... v2.2.3
Sven Gothel [Thu, 2 Oct 2014 18:42:19 +0000 (20:42 +0200)]
Bug 1085: Fix GLJPanel regression while printing w/ invisible GLJPanel: Zero size panel size

This is a regression due to commit 84f367a73c5b16dcebfd877e82e1c2cb90ae74ce:
   GLJPanel Cleanup: Remove initial FBO reshape;
   ** Propagate reshape only if differs from panel-size; ** <- this one
   Use pre-fetched panel-size.

Above commit only issued 'sendReshape'
if the reshape-size differs from the actual panel-size.

Note: The reshape-size is propagated to panel-size either in
  [1] initializeBackendImpl(..) or
  [2] handleReshape(..) @ paintComponent.

While printing w/ an invisible GLJPanel the reshape-size
has not yet propagated to the panel-size (see above)
and two consecutive reshape calls will cause the last one to be dropped.

With this patch we have:

GLJPanel.reshape.0 null resize [paint] [ this 0x0, pixelScale 1x1, panel 560x420] -> 0x0 * 1x1 -> 0x0, reshapeSize 0x0
GLJPanel.reshape.0 null resize [paint] [ this 560x420, pixelScale 1x1, panel 560x420] -> 560x420 * 1x1 -> 560x420, reshapeSize 560x420
GLJPanel.setupPrint: scale 1.000000 / 1.000000, samples 0, tileSz -1 x -1
GLJPanel.createAndInitializeBackend.1: [printing] 560x420 @ scale 1x1 -> 560x420 @ scale 1x1


3 years agoBug 1085: Increase and fix DEBUG verbosity in GLJPanel to catch reshape regression...
Sven Gothel [Thu, 2 Oct 2014 18:34:38 +0000 (20:34 +0200)]
Bug 1085: Increase and fix DEBUG verbosity in GLJPanel to catch reshape regression during print

3 years agoBug 1084: Fix GLProfile Mapping regarding hardware priority, honor software/hardware...
Sven Gothel [Wed, 1 Oct 2014 21:51:33 +0000 (23:51 +0200)]
Bug 1084: Fix GLProfile Mapping regarding hardware priority, honor software/hardware attribute of all profiles

GLProfile's mapping code does not consider the following combination:
  - GL4 software
  - GL3 hardware

and hence maps GL4-software -> [GL2ES2, GL2GL3],
where GL3-hardware -> [GL2ES2, GL2GL3] is desired.

This combination has recently been observed on
Mac OSX 10.9.5, which includes a software GL 4.1 implementation.

However, other systems could be affected as well.


Fix GLProfile.computeProfileImpl(..):

Only use the higher profile, if hardware-accelerated or none of the
lower profiles offers hardware-acceleration!

This extra condition was missing for certain profiles,
e.g. GL4, GL4bc, GL3, GL3bc and GL2.

3 years agoBug 1078, Bug 1082: Fix regression (typo), add missing assignment of printGLAD in...
Sven Gothel [Wed, 1 Oct 2014 02:54:42 +0000 (04:54 +0200)]
Bug 1078, Bug 1082: Fix regression (typo), add missing assignment of printGLAD in NewtCanvasAWT.setupPrint()

3 years agoBug 1081: Fix GLJPanel Regression: Honor pre-init reshape-size at initializeBackendImpl()
Sven Gothel [Tue, 30 Sep 2014 22:36:09 +0000 (00:36 +0200)]
Bug 1081: Fix GLJPanel Regression: Honor pre-init reshape-size at initializeBackendImpl()

Commit 84f367a73c5b16dcebfd877e82e1c2cb90ae74ce removed utilization of reshape-size
in case panel-size is valid, even if a reshape event happened in between:
  - addNotify
  - paintComponent

initializeBackendImpl() includes now uses reshape-size IFF handleReshape is set.
Before it was using reshape-size only if panel-size was invalid.

TestAWT03GLJPanelRecreate01 covers this issue.

3 years agoBug 1078: Add Fallback in AWTPrintLifecycle.setupPrint(): Use Onscreen GLAD if Offscr...
Sven Gothel [Tue, 30 Sep 2014 21:18:15 +0000 (23:18 +0200)]
Bug 1078: Add Fallback in AWTPrintLifecycle.setupPrint(): Use Onscreen GLAD if Offscreen-GLAD Realization throws an Exception (Stability)

- GLDrawableFactoryImpl: createOffscreenDrawable(..) and createDummyAutoDrawable(..)
  Temporary catch exception during setRealized(true) of newly created GLDrawable,
  to unrealize the instance before propagating the exception.

  This handling removes a memory leak in case the exception of this method is handled
  and application continues to operate, e.g. as in AWTPrintLifecycle.setupPrint().

  The underlying drawable gets unrealized, since it's setRealized(boolean)
  implementation toggles its realize-state before delegating the realize-operation.
  Hence this is functional.

- AWTPrintLifecycle.setupPrint() Stability
  Catch exception thrown by factory.createOffscreenAutoDrawable(..)'s setRealize(true)
  to continue operation w/ onscreen GLAD.

3 years agoTestVersionSemanticsNOUI: Use 'V22X' to mark current version on branch b22x
Sven Gothel [Tue, 23 Sep 2014 23:31:53 +0000 (01:31 +0200)]
TestVersionSemanticsNOUI: Use 'V22X' to mark current version on branch b22x

3 years agoFix all backward compatibility issues w/ 2.2.0 and 2.2.1
Sven Gothel [Tue, 23 Sep 2014 23:25:49 +0000 (01:25 +0200)]
Fix all backward compatibility issues w/ 2.2.0 and 2.2.1

- Preserve fields, but make them deprecated and don't use them anymore.
  Using a new version with less visibility.

- Revert incompatible method prototype changes

- GLFBODrawable:
  - Completly remove FBOMODE_DEFAULT and FBOMODE_USE_DEPTH usage
    and deprecate them

- GLRendererQuirks:
  - Completly remove COUNT usage and deprecate it.
  - Add getCount() method for future compatibility.

3 years agoFix synchronization issues in Animator* Exception case v2.2.2
Sven Gothel [Tue, 23 Sep 2014 01:56:04 +0000 (03:56 +0200)]
Fix synchronization issues in Animator* Exception case

Refines commit cef7ba607ad7e8eb1ff2a438d77710a29aa0bda6

- The animator monitor-lock was still hold in the post finally block
  issuing flushGLRunnables(), due to intrinsic monitor release (in finally):
   - <>
   - <>

- Further: AnimatorBase.flushGLRunnables() acquired the lock itself (duh!)

This commit removes the requirement for finally altogether
by simply return a boolean from handleUncaughtException(caughtException),
where false denotes the caller to propagate the exception itself (no handler).

Post synchronized block then issues flushGLRunnables() and
exceptation propagation as required.

AnimatorBase.flushGLRunnables() 'synchronized' modifier is removed.

Further, ThreadDeath is being propagated if caught.
Here the finally block is also removed - redundant.

3 years agoFix GLContextImpl.setRendererQuirks(..) CTX_IMPL_ACCEL_SOFT profile state
Sven Gothel [Mon, 22 Sep 2014 21:22:57 +0000 (23:22 +0200)]
Fix GLContextImpl.setRendererQuirks(..) CTX_IMPL_ACCEL_SOFT profile state

- GLContextImpl.setRendererQuirks(..) in called in GLContextImpl.setGLFunctionAvailability(..)

- GLContextImpl.setRendererQuirks(..) was called before fixing CTX_IMPL_ACCEL_SOFT
  via isCurrentContextHardwareRasterizer().

  The latter set CTX_IMPL_ACCEL_SOFT based on known software renderer string within GL_RENDERER.

  This lead to incorrect hwAccel assumption and hence wrong setting of GLRendererQuirks:
    - NoDoubleBufferedPBuffer (was selected even w/ later CTX_IMPL_ACCEL_SOFT)
    - BuggyColorRenderbuffer  (was never selected)

- Fix performs GLContextImpl.setRendererQuirks(..) _after_
  fixing CTX_IMPL_ACCEL_SOFT via isCurrentContextHardwareRasterizer().

3 years agoFix synchronization issues in GLDrawableHelper.flushGLRunnables(), fixes rare deadloc...
Sven Gothel [Mon, 22 Sep 2014 21:17:28 +0000 (23:17 +0200)]
Fix synchronization issues in GLDrawableHelper.flushGLRunnables(), fixes rare deadlock with animator-exception and invoke(wait=true, ..)

Fix synchronization issues in GLDrawableHelper.flushGLRunnables():
  - Querying 'glRunnables.size()' is not synchronized, only its reference is volatile,
    not the instance's own states.

  - 'flushGLRunnable()' must operates while acquired the 'glRunnable' lock.

  - 'glRunnables' are no more volatile

  - introduced volatile 'glRunnableCount', allowing 'display(..)' method
    to pre-query whether blocking 'execGLRunnables(..)' must be called.
    This is risk (deadlock) free.

Also fixes rare deadlock in animator display-exception / GLAD.invoke(wait=true, ..) case:
  - 'GLDrawableHelper.invoke(.., GLRunnable)' acquires the 'glRunnable' lock.
  - Then it queries animator state, which is blocking.
  - Hence animator's 'flushGLRunnable()' call must happen outside the animator lock

3 years agoStabilize, fix regression: GLDrawable.invoke(..) and Animator* display-loop 'closure'
Sven Gothel [Mon, 22 Sep 2014 05:07:42 +0000 (07:07 +0200)]
Stabilize, fix regression: GLDrawable.invoke(..) and Animator* display-loop 'closure'

GLDrawable.invoke(..) regression of commit c77b8f586cb2553582a42f5b90aeee5ef85f1efe:
  'wait' was not set to false, if 'deferredHere' was forced to 'false'.
  This could lead to the situation where GLRunnableTask
  will catch the exception and supresses it.

Animator/FPSAnimator post exception propagation code
   animThread = null; notifyAll();
must be complete to finalize animator state in case of an exception.
Decorate 'handleUncaughtException(..)' w/ try { } finally { }
where the latter ensures the mentioned 'closure'.

3 years agoAdding missing applet launcher html page for GLJPanelsAndGLCanvasDemoGL2Applet
Sven Gothel [Sat, 20 Sep 2014 21:01:13 +0000 (23:01 +0200)]
Adding missing applet launcher html page for GLJPanelsAndGLCanvasDemoGL2Applet

3 years agoTestVersionSemanticsNOUI: V221 -> V222 Add exception for GLFBODrawable.FBOMODE_DEFAULT
Sven Gothel [Sat, 20 Sep 2014 14:48:57 +0000 (16:48 +0200)]
TestVersionSemanticsNOUI: V221 -> V222 Add exception for GLFBODrawable.FBOMODE_DEFAULT

3 years agoAdd GLJPanelsAndGLCanvasDemoGL2Applet: Simple applet test for GLJPanel/GL2 testing...
Sven Gothel [Sat, 20 Sep 2014 14:48:18 +0000 (16:48 +0200)]
Add GLJPanelsAndGLCanvasDemoGL2Applet: Simple applet test for GLJPanel/GL2 testing (offscreen, ..); Clean up GL2 Gears and Teapot state enable/disable

3 years agoFBObject: Fix depth- and stencil bit count selection for attachRenderbuffer(..);
Sven Gothel [Sat, 20 Sep 2014 14:06:56 +0000 (16:06 +0200)]
FBObject: Fix depth- and stencil bit count selection for attachRenderbuffer(..);

- Fix depth- and stencil bit count selection for attachRenderbuffer(..)

  - Refactor depth- and stencil bit-count -> format into own method

  - Allow depth- and stencil bit-count select a higher bit-count if required (fix)

- GLFBODrawable.FBOMODE_USE_DEPTH is deprecated, using GLCapabilities.[get|set]DepthBits(..)
  - It was an oversight to introduce the bit flag in the first place,
    since we should have used the capabilities depth bit-count

- Graph Test: GLEventListenerButton shall use requested capabilities for FBO drawable.

3 years agoTestVersionSemanticsNOUI: Add v2.2.1 jogl-all.jar
Sven Gothel [Sat, 20 Sep 2014 06:02:32 +0000 (08:02 +0200)]
TestVersionSemanticsNOUI: Add v2.2.1 jogl-all.jar

3 years agoTestVersionSemanticsNOUI: Enable test V220->V221 and V221->V222 (w/ 2 extra exclusions)
Sven Gothel [Sat, 20 Sep 2014 05:24:06 +0000 (07:24 +0200)]
TestVersionSemanticsNOUI: Enable test V220->V221 and V221->V222 (w/ 2 extra exclusions)

Enable tests:

- V220->V221

- V221->V222 (w/ 2 extra exclusions)
  - GLRendererQuirks COUNT has been increased (-> use a method, FIXME)
  - Animator pauseIssued is volatile now

3 years agoGLJPanel: Fix backward compatibility, regression of commit fa6ba9d70d4fe47f05facc7260...
Sven Gothel [Sat, 20 Sep 2014 05:21:55 +0000 (07:21 +0200)]
GLJPanel: Fix backward compatibility, regression of commit fa6ba9d70d4fe47f05facc72608fb10c889f9d09

- bring back removed 'void dispose()' -> deprecated now

3 years agoFBObject: Fix merge issues of commit 034d843359508833094b6a87eb4b58fd5231bafa
Sven Gothel [Sat, 20 Sep 2014 05:20:35 +0000 (07:20 +0200)]
FBObject: Fix merge issues of commit 034d843359508833094b6a87eb4b58fd5231bafa

- Handle deprecated reset(..) variants return type in a backward compatible manner

3 years agoTestGearsES2GLJPanelAWT: Update frame title (awt-size, pixel-size) for GLEventListene...
Sven Gothel [Sat, 20 Sep 2014 00:52:27 +0000 (02:52 +0200)]
TestGearsES2GLJPanelAWT: Update frame title (awt-size, pixel-size) for GLEventListener.reshape(..)

3 years agoFBObject: Simplify API (init/reset); Only issue automatic resetSamplingSink(..) if...
Sven Gothel [Sat, 20 Sep 2014 00:51:44 +0000 (02:51 +0200)]
FBObject: Simplify API (init/reset); Only issue automatic resetSamplingSink(..) if required; Fix resetSamplingSink(..), isBound(), ..

- Simplify API (init/reset)
  - use new unique methods for init and reset:
    - void init(final GL gl, final int newWidth, final int newHeight, final int newSamples)
      - does not issue resetSamplingSink(..)
    - boolean reset(final GL gl, final int newWidth, final int newHeight, final int newSamples)
      - always issues resetSamplingSink(..)

  - deprecated dual-use (init/reset):
    - boolean reset(final GL gl, final int newWidth, final int newHeight)
    - boolean reset(final GL gl, int newWidth, int newHeight, int newSamples, final boolean resetSamplingSink)

- reset(..) no more creates a dummy 'samplingSink' instance if sampling > 0,
  left up to resetSamplingSink(..)

- Track 'modified' state of FBObject,
  if size, format or any attachment has been changed since last
    - use(..)
    - syncSamplingSink(..)
    - resetSamplingSink(..)

- Only issue resetSamplingSink(..) from syncSamplingSink(..)/use(..)
  if 'modified == true'


- Fix setSamplingSink(..), i.e. samplingSink state handling:
  - Validated whether given samplingSink is initialized,
    throws Exception if not.

- Fix resetSamplingSink(..)
  - resets the bound state, i.e. leaves it untouched
    - also unbinds the samplingSink

  - sampleSinkDepthStencilMismatch() also returns true if
    this.depth/stencil == null, but samplingSink is not.

  - Newly created colorbuffer/-texture matches
    exiting colorbuffer's internal-format, if exists.

  - Using simplified resetSizeImpl(..) for size mismatch

  - Simplified samplingColorSink init check

- Fix isBound()
    was: 'bound = bound && fbName != gl.getBoundFramebuffer(GL.GL_FRAMEBUFFER)'
    fix: 'bound = bound && fbName == gl.getBoundFramebuffer(GL.GL_FRAMEBUFFER)'

- Fix detachRenderbuffer(..) validates whether detachment was successful,
  similar to detachColorbuffer(..)


3 years agoStereoClientRenderer: Remove redundant FBObject.detachAllColorbuffer(gl) for init...
Sven Gothel [Fri, 19 Sep 2014 17:21:29 +0000 (19:21 +0200)]
StereoClientRenderer: Remove redundant FBObject.detachAllColorbuffer(gl) for init call

3 years agoFBOobject GLEL Tests: Remove redundant detachAllColorbuffer(gl) for init call
Sven Gothel [Fri, 19 Sep 2014 17:20:48 +0000 (19:20 +0200)]
FBOobject GLEL Tests: Remove redundant detachAllColorbuffer(gl) for init call

3 years agoNew GLRendererQuirks.BuggyColorRenderbuffer: On Mesa 7.2 software, FBO color renderbu...
Sven Gothel [Thu, 18 Sep 2014 23:24:32 +0000 (01:24 +0200)]
New GLRendererQuirks.BuggyColorRenderbuffer: On Mesa 7.2 software, FBO color renderbuffer may cause a crash

Workaround crash caused by Mesa 7.2 software rendering
using color renderbuffer target in FBO.

If Mesa < 8.0 and software - or -
property 'jogl.fbo.force.nocolorrenderbuffer' is set,
set quirks:
- GLRendererQuirks.BuggyColorRenderbuffer
- GLRendererQuirks.NoFullFBOSupport (to disable MSAA)

if GLRendererQuirks.BuggyColorRenderbuffer is set.


Crash Report:
  GNU C Library       : 2.13 stable
  OpenGL              : software
  Operating System    : Linux 3.2.0-4-amd64 #1 SMP Debian 3.2.51-1 x86_64
  Processor ID        : x86 Family 6 Model 44 Stepping 2, GenuineIntel

Abnormal termination:
Segmentation violation

Register State (from fault):
  RAX = 00000000ff1818f0  RBX = 00000000beaf8afc
  RCX = 0000000000000004  RDX = 00007f85ed9c9010
  RSP = 00007f8252d24fd0  RBP = 00007f8252d25020
  RSI = 0000000017b9b330  RDI = 0000000015bca400

   R8 = 0000000000000000   R9 = 00007f81edcd3014
  R10 = 00007f823565f6ce  R11 = 00007f827bee49aa
  R12 = 0000000000001406  R13 = 0000000000000001
  R14 = 00000000154d5458  R15 = 00000000154d4f10

  RIP = 00007f823565f7bc  EFL = 0000000000010206

   CS = 0033   FS = 0000   GS = 0000

Stack Trace (from fault):
[  0] 0x00007f823565f7bc put_row_ubyte4 at /mesa/main/renderbuffer.c:665 (in /lib/
[  1] 0x00007f8235727239 _swrast_write_rgba_span at /mesa/swrast/s_span.c:1450 (in /lib/
[  2] 0x00007f823574b071 smooth_rgba_triangle at /mesa/swrast/s_tritemp.h:862 (in /lib/
[  3] 0x00007f82357155f0 _swrast_Triangle at /mesa/swrast/s_context.c:692 (in /lib/
[  4] 0x00007f8235771780 triangle_offset_twoside_rgba at /mesa/swrast_setup/ss_tritmp.h:188 (in /lib/
[  5] 0x00007f82356d2cea _tnl_render_poly_elts at /mesa/tnl/t_vb_rendertmp.h:313 (in /lib/
[  6] 0x00007f82356d335e _tnl_RenderClippedPolygon at /mesa/tnl/t_vb_render.c:244 (in /lib/
[  7] 0x00007f82356c9313 clip_tri_4 at /mesa/tnl/t_vb_cliptmp.h:230 (in /lib/
[  8] 0x00007f82356cd026 clip_render_triangles_verts at /mesa/tnl/t_vb_rendertmp.h:163 (in /lib/
[  9] 0x00007f82356d37d9 run_render at /mesa/tnl/t_vb_render.c:320 (in /lib/
[ 10] 0x00007f82356c2436 _tnl_run_pipeline at /mesa/tnl/t_pipeline.c:158 (in /lib/
[ 11] 0x00007f82356c37da _tnl_draw_prims at /mesa/tnl/t_draw.c:402 (in /lib/
[ 12] 0x00007f82356b673a vbo_exec_DrawArrays at /mesa/vbo/vbo_exec_array.c:263 (in /lib/
[ 13] 0x00007f823583e5b0 glDrawArrays at /mesa/glapi/glapitemp.h:1645 (in /lib/


3 years agoGLContextImpl.setRendererQuirks(..): Use GLRendererQuirks.addQuirk(int), dropping...
Sven Gothel [Thu, 18 Sep 2014 23:15:07 +0000 (01:15 +0200)]
GLContextImpl.setRendererQuirks(..): Use GLRendererQuirks.addQuirk(int), dropping usage of .addQuirks(int[] quirks, offset, len)

Directly adding a quirk allows dropping usage of erroneous fixed-sized int[] array for accumulated quirks.

3 years agoGLFBODrawableImpl: Utilize common self-contained setupFBO(..) method for FBO creation...
Sven Gothel [Thu, 18 Sep 2014 01:59:04 +0000 (03:59 +0200)]
GLFBODrawableImpl: Utilize common self-contained setupFBO(..) method for FBO creation incl. Bug 1020 glClear(..) ; Fixes missing fbo.unbind(gl)

Utilize common self-contained setupFBO(..) method for FBO creation incl. Bug 1020 glClear(..)
  initialize(..) and reset(..)'s fallback code
  contained duplicated FBO setup/creation code.

  The former was updated to suite the new FBO mode bits,
  where the latter left untouched, hence was buggy.

  Now setup and creation of a single FBO has been merged into setupFBO(..).

Fixes missing fbo.unbind(gl):
  We did no issue unbind() on each fbo instance
  while adding the workaround for Bug 1020.

  Adding fbo.markUnbound() call to all fbos but the last.

  Note: This might not cause any issues, since fbo.bind(gl) tests the currently bound fbo.

3 years agoGLJPanel Cleanup: Remove initial FBO reshape; Propagate reshape only if differs from...
Sven Gothel [Thu, 18 Sep 2014 01:43:13 +0000 (03:43 +0200)]
GLJPanel Cleanup: Remove initial FBO reshape; Propagate reshape only if differs from panel-size; Use pre-fetched panel-size.

3 years agoGLJPanel: Allow reconfiguration of offscreen's GLCapabilitiesImmutable
Sven Gothel [Sat, 13 Sep 2014 23:14:38 +0000 (01:14 +0200)]
GLJPanel: Allow reconfiguration of offscreen's GLCapabilitiesImmutable

Offscreen's GLCapabilitiesImmutable reconfiguration will dispose a realized instance
and issues recreation via initializeBackendImpl() immedietly.

Implementation performs operation on AWT-EDT.

  - TestGearsES2GLJPanelAWT:
      - Toggle MSAA via 'm'

  - TestGearsGLJPanelAWT:
      - Toggle MSAA via 'm'
      - Toggle Bitmap via 'b'

3 years agoGears* Demos: Spin at same speed (0.5f tan per frame) ; GearsES2: Bring back default...
Sven Gothel [Sat, 13 Sep 2014 23:09:45 +0000 (01:09 +0200)]
Gears* Demos: Spin at same speed (0.5f tan per frame) ; GearsES2: Bring back default Z values, allow Z customization (StereoDemo01)

3 years agoFPSAnimator: Remove toString() override as accidently added in commit fa0115efb3989c2...
Sven Gothel [Sat, 13 Sep 2014 22:44:09 +0000 (00:44 +0200)]
FPSAnimator: Remove toString() override as accidently added in commit fa0115efb3989c28af21fc5f570ae49723566107

3 years agoFPSAnimator: Align code and pause/resume conditions w/ Animator, simplifying review.
Sven Gothel [Sat, 13 Sep 2014 13:28:58 +0000 (15:28 +0200)]
FPSAnimator: Align code and pause/resume conditions w/ Animator, simplifying review.

3 years agoAnimatorBase.finishLifecycleAction(): Non blocking call shall return true, success...
Sven Gothel [Sat, 13 Sep 2014 13:27:43 +0000 (15:27 +0200)]
AnimatorBase.finishLifecycleAction(): Non blocking call shall return true, success - otherwise pause()/.. return value is inconsistent.

Caller of e.g. pause() running on the anim-thread or AWT-EDT (AWTAnimatorImpl)
will be non-blocking.

Before this change, a non-blocking simply did not wait until the 'hold' condition is reached
and returned its negated value.
This ofc is 'false', indicated unsuccessful operation.

Caller use the return value to determine whether the call actually paused (or ..)
the animator. Despite the non-blocking nature, the pause state was set, even if not reached.

Hence a resume() would be required to continue operation after a temporary pause.


This change ignores the non-blocking nature's unmet condition.
finishLifecycleAction() returns !nok || !blocking,
i.e. either true for the reached condition (blocking) or true if non-blocking.
Blocking calls with unmet condition still return false.


In case an animated GLAutoDrawableis being pulled after a non-blocking animator pause() call,
the GLAutoDrawable's implementation thread-safety must ensure proper operation.


3 years agoGLBuffers.sizeof(..): Add support for ES3, reading supported glPixelStorei states
Sven Gothel [Tue, 9 Sep 2014 19:02:55 +0000 (21:02 +0200)]
GLBuffers.sizeof(..): Add support for ES3, reading supported glPixelStorei states

3 years agoFix GLPixelStorageModes: Add missing import GLContext ; Regression of commit 01c709ff...
Sven Gothel [Tue, 9 Sep 2014 20:18:39 +0000 (22:18 +0200)]
Fix GLPixelStorageModes: Add missing import GLContext ; Regression of commit 01c709ff0e81ddd84e3bc78ec180a53587ec855c

(happens when you try to do things fast w/ line commits and w/o checking .. duh!)

3 years agoDon't utilize glPixelStorei's PACK/UNPACK IMAGE_HEIGHT and SKIP_IMAGES for Desktop...
Sven Gothel [Tue, 9 Sep 2014 17:58:47 +0000 (19:58 +0200)]
Don't utilize glPixelStorei's PACK/UNPACK IMAGE_HEIGHT and SKIP_IMAGES for Desktop GL < 1.2, avoiding GL-Error

Commit fc1e98790a02b4fa7922f3cdd9d437f87d7c99e5
added handling of PACK/UNPACK IMAGE_HEIGHT and SKIP_IMAGES in GLPixelStorageModes.

However, it has been overseen that the four states are not available in OpenGL 1.1.

Adding exclusion of the same if desktop GL < 1.2
and hence avoiding GL errors.

Same applies to GLBuffers.sizeof(..) method.

3 years agoBug 1065: Handle NULL result of TISGetInputSourceProperty(keyboard, kTISPropertyUnico... v2.2.1
Sven Gothel [Mon, 8 Sep 2014 06:27:03 +0000 (08:27 +0200)]
Bug 1065: Handle NULL result of TISGetInputSourceProperty(keyboard, kTISPropertyUnicodeKeyLayoutData), avoiding crash

3 years agoLICENSE.txt: Emphasice incompatible A.7) module is OPTIONAL
Sven Gothel [Mon, 8 Sep 2014 03:59:05 +0000 (05:59 +0200)]
LICENSE.txt: Emphasice incompatible A.7) module is OPTIONAL

3 years agoBug 1047 - jogamp.opengl.glu.mipmap.Mipmap now uses already parsed GL version number...
Sven Gothel [Mon, 8 Sep 2014 03:58:35 +0000 (05:58 +0200)]
Bug 1047 - jogamp.opengl.glu.mipmap.Mipmap now uses already parsed GL version number and GL profile selection

Since Bug 1047 didn't provide a unit tests, this must be fine for now.

3 years agoBug 1060 - Add GLProfile.isInitialized(): Returns true if JOGL has been initialized
Sven Gothel [Wed, 3 Sep 2014 21:32:52 +0000 (23:32 +0200)]
Bug 1060 - Add GLProfile.isInitialized(): Returns true if JOGL has been initialized

3 years agoAdding missing '@since 2.2.1' tags to ShaderCode and GLContext
Sven Gothel [Wed, 3 Sep 2014 21:25:52 +0000 (23:25 +0200)]
Adding missing '@since 2.2.1' tags to ShaderCode and GLContext

3 years agoBug 1059 _and_ version-compat breackage of commit c78ceb642d0ef5bb5bf27ff8ff1495175ee...
Sven Gothel [Wed, 3 Sep 2014 13:42:37 +0000 (15:42 +0200)]
Bug 1059 _and_ version-compat breackage of commit c78ceb642d0ef5bb5bf27ff8ff1495175ee2e983

Commit c78ceb642d0ef5bb5bf27ff8ff1495175ee2e983 changed:
  -    public static final String GL4   = "GL4";
  +    public static final String GL4   = "GL4".intern();

which is identified by semver as incompatible,
due to Bug 1059 (no more inlining of interned string references).

3 years agoHowToBuild: Debian i386 on x64: Add symbolic links for shared libs
Sven Gothel [Wed, 3 Sep 2014 04:25:40 +0000 (06:25 +0200)]
HowToBuild: Debian i386 on x64: Add symbolic links for shared libs

3 years agoBug 1058 - Fix GLContext.getGLSLVersionString(): Add 'profile' after version for...
Sven Gothel [Tue, 2 Sep 2014 05:16:58 +0000 (07:16 +0200)]
Bug 1058 - Fix GLContext.getGLSLVersionString(): Add 'profile' after version for GLSL >= 150 allowing GLSL compatibility profile

3 years agoBug 1043 - Add Tessellation Control and Evaluation Shader Support
Sven Gothel [Tue, 2 Sep 2014 05:11:18 +0000 (07:11 +0200)]
Bug 1043 - Add Tessellation Control and Evaluation Shader Support

- Add GL4.GL_TESS_CONTROL_SHADER and GL4.GL_TESS_EVALUATION_SHADER support for GLSL util class ShaderCode
- Add unit test TestTessellationShader01GL4NEWT, testing TessellationShader01aGL4 and TessellationShader01bGL4

3 years agoBug1044: Offscreen drawable AWT/ImageIO results in black image on OSX/[Java7-Java8]
Sven Gothel [Tue, 2 Sep 2014 03:38:39 +0000 (05:38 +0200)]
Bug1044: Offscreen drawable AWT/ImageIO results in black image on OSX/[Java7-Java8]

- Using our PNGJ writer results in proper images (RGB and RGBA) on all platforms
- Seems to be a bug w/ AWT/ImageIO

3 years agoBug 1048: Add unit tests demonstrating multiple NewtCanvasAWT instances are working
Sven Gothel [Tue, 2 Sep 2014 02:53:30 +0000 (04:53 +0200)]
Bug 1048: Add unit tests demonstrating multiple NewtCanvasAWT instances are working

Enhance following performance test cases, adding NewtCanvasAWT,
beside GLCanvas and GLJPanel, 25 instances:

Adding simple com.jogamp.opengl.test.junit.newt.TestMultipleNewtCanvasAWT
w/ two instances.

Manually tested on GNU/Linux and OSX (java7 and java8).

3 years agoGLProfile: Use String.intern() on static final profile strings ('GL4bc', 'GL4', ...
Sven Gothel [Tue, 2 Sep 2014 00:38:14 +0000 (02:38 +0200)]
GLProfile: Use String.intern() on static final profile strings ('GL4bc', 'GL4', ..) consequently using reference comparison.

GLProfile already compared profile strings by reference, hence interning those strings
to become canonical references was missing!

Consequently using reference comparison for all profile strings in GLProfile.

3 years agoBug 1052 - OpenGL ES 3.0 Mesa 10.1.3 Caught GLException: Not a GL4ES3 implementation...
Sven Gothel [Tue, 2 Sep 2014 00:20:52 +0000 (02:20 +0200)]
Bug 1052 - OpenGL ES 3.0 Mesa 10.1.3 Caught GLException: Not a GL4ES3 implementation - Part 2/2

- TestGLProfile01NEWT: Allow ctx.isGLES3Compatible() and hence GL4ES3 on GL3bc and GL3

- GLProfile: Remove GL4ES3 mapping using GL3bc and GL3, only GL4bc, GL4 and GLES3 are allowed in static mapping.

3 years agoBug 1052 - OpenGL ES 3.0 Mesa 10.1.3 Caught GLException: Not a GL4ES3 implementation...
Sven Gothel [Tue, 2 Sep 2014 00:04:22 +0000 (02:04 +0200)]
Bug 1052 - OpenGL ES 3.0 Mesa 10.1.3 Caught GLException: Not a GL4ES3 implementation - Part 1/2

Test enhancements triggering issue 'Bug 1052 - OpenGL ES 3.0 Mesa 10.1.3 Caught GLException: Not a GL4ES3 implementation'

- TestGLProfile01NEWT: Complete GLProfile and GL-object and GLContext validation

- On OpenGL ES 3.0 Mesa 10.1.4 it produces:

1) test06GLProfileGL4ES3(com.jogamp.opengl.test.junit.jogl.acore.TestGLProfile01NEWT) GL4ES3 is neither GL4bc, GL4 nor GLES3
    at com.jogamp.opengl.test.junit.jogl.acore.TestGLProfile01NEWT.validateGLProfileGL4ES3(
    at com.jogamp.opengl.test.junit.jogl.acore.TestGLProfile01NEWT.validateOffline(
    at com.jogamp.opengl.test.junit.jogl.acore.TestGLProfile01NEWT.test06GLProfileGL4ES3(

i.e. wrong mapping of request GL4ES3 -> GL3

3 years agoBug 1017 - TextureIO.write(Texture, File) throws GLException Not a GL2 implementation...
Sven Gothel [Mon, 1 Sep 2014 14:14:43 +0000 (16:14 +0200)]
Bug 1017 - TextureIO.write(Texture, File) throws GLException Not a GL2 implementation on GL3 contexts

_gl.getGL2() -> _gl.getGL2GL3()

3 years agoScripts: Bump to java 1.8.0_20
Sven Gothel [Sun, 31 Aug 2014 19:17:43 +0000 (21:17 +0200)]
Scripts: Bump to java 1.8.0_20

3 years agoTestSharedContextNewtAWTBug523: Fix junit usage ..
Sven Gothel [Sun, 31 Aug 2014 15:50:05 +0000 (17:50 +0200)]
TestSharedContextNewtAWTBug523: Fix junit usage ..

3 years agoBug 1020 - First MSAA FBO frame on a mac osx nvidia card not antialiased
Sven Gothel [Sun, 31 Aug 2014 03:53:28 +0000 (05:53 +0200)]
Bug 1020 - First MSAA FBO frame on a mac osx nvidia card not antialiased

OSX/Nvidia's FBO needs to be cleared before blitting,
otherwise first MSAA frame lacks antialiasing.

GLFBODrawableImpl.initialize(..) can clear

FBObject cannot clear the buffer(s) due to it's low-level API,
i.e. it cannot know when the first bind occurs _after_ user
completed FBO setup (attaching buffers).

Hence plain FBObject usage required manual injection
of glClear(..) after setup as demonstrated in GLJPanel.

We may need to elaborate in this case,
i.e. add an FBObject API entry like 'fbo.postInitNotify()'.

3 years agoTestGLReadBuffer01GLJPanelAWT: Add keyFrame-wait after init
Sven Gothel [Sun, 31 Aug 2014 02:25:52 +0000 (04:25 +0200)]
TestGLReadBuffer01GLJPanelAWT: Add keyFrame-wait after init

3 years agoFBObject Cleanup: Add comments about maxSamples > 0 implies fullFBOSupport; bind...
Sven Gothel [Sun, 31 Aug 2014 02:24:56 +0000 (04:24 +0200)]
FBObject Cleanup: Add comments about maxSamples > 0 implies fullFBOSupport; bind(): Set dedicated read/write if fullFBOSupport

- Add comments about maxSamples > 0 implies fullFBOSupport

- bind(): Set dedicated read/write if fullFBOSupport
  as done in syncSamplingSink() and unbind()

3 years agoRefine test cases for synchronous GLAutoDrawable display, swap-buffer and read-pixels...
Sven Gothel [Sat, 30 Aug 2014 18:32:56 +0000 (20:32 +0200)]
Refine test cases for synchronous GLAutoDrawable display, swap-buffer and read-pixels (non-MSAA and MSAA) Bugs: 841, 975 and 1020

.. add Platform and GL info !

3 years agoRefine test cases for synchronous GLAutoDrawable display, swap-buffer and read-pixels...
Sven Gothel [Sat, 30 Aug 2014 18:05:27 +0000 (20:05 +0200)]
Refine test cases for synchronous GLAutoDrawable display, swap-buffer and read-pixels (non-MSAA and MSAA) Bugs: 841, 975 and 1020

3 years agoBug 1054: Revert dfb9ed47ac6d8e85f6ae5fe166e7a6e28ca8ff83: Cannot change protected...
Sven Gothel [Sat, 30 Aug 2014 10:03:03 +0000 (12:03 +0200)]
Bug 1054: Revert dfb9ed47ac6d8e85f6ae5fe166e7a6e28ca8ff83: Cannot change protected field name w/o breaking backward compatibility

3 years agoBug 1055 - Access and query shared master GLContext in a deterministic fashion ;...
Sven Gothel [Sat, 30 Aug 2014 09:59:13 +0000 (11:59 +0200)]
Bug 1055 - Access and query shared master GLContext in a deterministic fashion ; Don't use arbitrary shared context as 'master'.

GLContext* passes the shared-master to GLContextShareSet,
which only creates a sets of shared contexts
without differentiating the master context.

GLContext*'s shared-slave attempts to lock
the realized shared-master's surface at creation.

Currently only an arbitrary shared context is selected
due to the missing 'master' identity.
The arbitrary shared context's surface is locked
and its shared context handle used to create the slave context.

Lacking of using the user given shared-master
can lead to deadlock situations - and locking a 'wrong' surface.


The patch:

  - Allows query the user given shared-master!

  - Use the user given shared-master for locking and it's
    context handle for the slave's creation.

  - The shared-context mapping maps each shared-master
    to a shared-slave within one shared-context-set,
    allowing deterministic and individual shared-master queries.

3 years agoBug 1054: GLContext: makeCurrent() needs a null-check of [mutable] drawable; Review...
Sven Gothel [Fri, 29 Aug 2014 22:21:00 +0000 (00:21 +0200)]
Bug 1054:  GLContext: makeCurrent() needs a null-check of [mutable] drawable; Review null checks and synchronization/locking.

'drawable' field of GLContextImpl is mutable via setGLDrawable(..),
which requires high-level locking as documented.

The required high-level locking allows us to _not_ add special
synchronization to this field (and drawableRead).

A simple null-check in makeCurrent() shall be sufficient,
plus ensuring mentioned high-level locking is applied.

GLContextImpl 'drawable' and 'drawableRead' synchronization:
  - commit ad79bd072b600a3f2416cc6f0c61e2925000069d check of null drawable is sufficient
  - Add GLAutoDrawable upstream-lock locking to:
    - AWT GLCanvas setupPrint/releasePrint
    - AWT GLJPanel (was missing)

  - validate shared-context native-surface locking, throw exception if not successful
  - pixelDataEvaluated does not need to be synchronized, since it's being called while context is current, locking
  - GLDrawableHelper.recreateGLDrawable(..): Remove redundant glFinish() call

3 years agoBug 1054: Cleanup GLContext: 'lock' -> 'contextLock'
Sven Gothel [Fri, 29 Aug 2014 14:11:08 +0000 (16:11 +0200)]
Bug 1054: Cleanup GLContext: 'lock' -> 'contextLock'

3 years agoBug 1054: Cleanup GLContext 'lock' and 'drawable' usage, perform drawable null check...
Sven Gothel [Fri, 29 Aug 2014 13:19:50 +0000 (15:19 +0200)]
Bug 1054: Cleanup GLContext 'lock' and 'drawable' usage, perform drawable null check in constructor.

This patch merely cleans up 'lock' and 'drawable' usage,
while fixing:
  - constructor: Add drawable null check -> IllegalArgumentException
  - setGLReadDrawable: Proper precondition checks
  - setGLDrawable: Proper precondition checks

Affected methods of mutable drawable for which we have to consider locking:
  - setGLReadDrawable
  - setGLDrawable
  - release
  - destroy
  - makeCurrent

3 years agoGLSharedContextSetter: Fix API doc and all html references (fix URL and complete...
Sven Gothel [Thu, 28 Aug 2014 23:31:03 +0000 (01:31 +0200)]
GLSharedContextSetter: Fix API doc and all html references (fix URL and complete notes in all implementations)

3 years agoglsl.sdk.CompileShader: Use IOUtil.StreamMonitor
Sven Gothel [Thu, 28 Aug 2014 17:07:36 +0000 (19:07 +0200)]
glsl.sdk.CompileShader: Use IOUtil.StreamMonitor

3 years agoTestVersionSemantics: BACKWARD_COMPATIBLE_USER current vs 2.2.0
Sven Gothel [Thu, 28 Aug 2014 17:00:42 +0000 (19:00 +0200)]
TestVersionSemantics: BACKWARD_COMPATIBLE_USER current vs 2.2.0

3 years agoHowToBuild: Revert required git version back to 1.6.0 (from 1.7.10, commit 1882b65513...
Sven Gothel [Thu, 28 Aug 2014 05:26:33 +0000 (07:26 +0200)]
HowToBuild: Revert required git version back to 1.6.0 (from 1.7.10, commit 1882b6551326e583c311b6b169be2222125df4d9)

git version 1.7.10 is not available on certail 'long life' / LTS systems,
we have to reschedule the gluegen patch for a later time.

3 years agoHowToBuild: Bump requirements: ant >= 1.9.0, git >= 1.7.10
Sven Gothel [Thu, 28 Aug 2014 04:56:16 +0000 (06:56 +0200)]
HowToBuild: Bump requirements: ant >= 1.9.0, git >= 1.7.10

- ant >= 1.9.0
 - Due to java8 build support

- git >= 1.7.10
 - Due to gluegen's new git query of HEAD branch and sha1-tip

3 years agoGraph/Text: Prepare unit tests for font rendering validation (WIP)
Sven Gothel [Thu, 28 Aug 2014 04:15:49 +0000 (06:15 +0200)]
Graph/Text: Prepare unit tests for font rendering validation (WIP)

3 years agoGraph/Text: Prepare unit tests for font rendering validation (WIP)
Sven Gothel [Thu, 28 Aug 2014 04:15:23 +0000 (06:15 +0200)]
Graph/Text: Prepare unit tests for font rendering validation (WIP)

3 years agograph/font: Use 'pixelSize' var-name, remove redundancies
Sven Gothel [Thu, 28 Aug 2014 04:10:37 +0000 (06:10 +0200)]
graph/font: Use 'pixelSize' var-name, remove redundancies

3 years agoFix commit b5910f18f0b82a8a1f6f6252dc19971d5e487f39 for toString(): Call super.hashCo...
Sven Gothel [Thu, 28 Aug 2014 04:03:17 +0000 (06:03 +0200)]
Fix commit b5910f18f0b82a8a1f6f6252dc19971d5e487f39 for toString(): Call super.hashCode(), due to InternalError("hashCode not designed")

3 years agoScripts: Bump to java 1.7.0_67 and apache-ant 1.9.4
Sven Gothel [Thu, 28 Aug 2014 03:57:29 +0000 (05:57 +0200)]
Scripts: Bump to java 1.7.0_67 and apache-ant 1.9.4

3 years agoGLStateTracker: Avoid private access wrapper, reuse fetched pixelStateMap
Sven Gothel [Thu, 28 Aug 2014 03:55:56 +0000 (05:55 +0200)]
GLStateTracker: Avoid private access wrapper, reuse fetched pixelStateMap

3 years agoFix javadoc package property, use comma as separator (dropped newt and oculusvr apidoc)
Sven Gothel [Mon, 11 Aug 2014 01:54:29 +0000 (03:54 +0200)]
Fix javadoc package property, use comma as separator (dropped newt and oculusvr apidoc)

3 years agoNEWT Window setSurfaceSize(..): Add API doc comment about setting pixel-unit size... v2.2.0
Sven Gothel [Wed, 6 Aug 2014 22:11:37 +0000 (00:11 +0200)]
NEWT Window setSurfaceSize(..): Add API doc comment about setting pixel-unit size after creation in multiple monitor mode.

3 years agoBug 1021: Refine Stereo API; Fix GenericStereoDevice; Fix StereoDemo01 for movie...
Sven Gothel [Wed, 6 Aug 2014 22:10:41 +0000 (00:10 +0200)]
Bug 1021: Refine Stereo API; Fix GenericStereoDevice; Fix StereoDemo01 for movie playback and OSX usage (HiDPI surfaceSize)

- StereoDevice.DeviceType: Add API doc

- StereoDevice: Add getFactory()

- GenericStereoDevice
  - Use common static vars for configurations for simplicity

  - Fix createRenderer(..)'s eyeViewport in case no post-processing
    is performed, i.e. needs viewport X offset.

- StereoDemo01
  - Use 'movie' eyePosition instead of default if:
    - using a movie player _and_ using lenses!

  - Fix NEWT window pixel-unit size after window creation!

3 years agoBug 1021: Fix OculusVR detection, need to probe HMD count, otherwise native SDK code...
Sven Gothel [Wed, 6 Aug 2014 22:05:47 +0000 (00:05 +0200)]
Bug 1021: Fix OculusVR detection, need to probe HMD count, otherwise native SDK code crashes w/ NPE

3 years agoBug 1039: Rename GLAnimatorControl.UncaughtGLAnimatorExceptionHandler -> GLAnimatorCo...
Sven Gothel [Wed, 6 Aug 2014 04:06:01 +0000 (06:06 +0200)]
Bug 1039: Rename GLAnimatorControl.UncaughtGLAnimatorExceptionHandler -> GLAnimatorControl.UncaughtExceptionHandler

3 years agoBug 1039 - Specify behavior of GLEventListener Exceptions occurring while GLAutoDrawa...
Sven Gothel [Wed, 6 Aug 2014 03:59:08 +0000 (05:59 +0200)]
Bug 1039 - Specify behavior of GLEventListener Exceptions occurring while GLAutoDrawable processing [part-3]

Add GLAnimatorControl.UncaughtGLAnimatorExceptionHandler interface to optionally handle
uncaught exception within an animator thread by the user.

Implementation also requires to flush all enqueued GLRunnable instances
via GLAutoDrawable.invoked(..) in case such exception occurs.
Hence 'GLAutoDrawable.flushGLRunnables()' has been added.

Only subsequent exceptions, which cannot be thrown are dumped to System.stderr.


Handling of exceptions during dispose()

Exception in NEWT's disposeGL*() are also caught and re-thrown after
the NEWT window has been destroyed in WindowImpl.destroyAction:

  - GLEventListener.dispose(..)
    - GLDrawableHelper.disposeAllGLEventListener(..)
      - GLDrawableHelper.disposeGL(..)
        - GLAutoDrawableBase.destroyImplInLock(..)
          - GLWindow.GLLifecycleHook.destroyActionInLock(..)
            - WindowImpl.destroyAction on NEWT-EDT
              - WindowImpl.destroy

Further more, exceptions occuring in native windowing toolkit triggered destroy()
are ignored:
  - GLAutoDrawableBase.defaultWindowDestroyNotifyOp(..)

It has to be seen whether such exception handling for
dispose() shall be added to AWT/SWT.


TestGLException01NEWT covers all GLEventListener exception cases
on-thread and off-thread (via animator).


3 years agoBug 1039 - Specify behavior of GLEventListener Exceptions occurring while GLAutoDrawa...
Sven Gothel [Tue, 5 Aug 2014 21:01:28 +0000 (23:01 +0200)]
Bug 1039 - Specify behavior of GLEventListener Exceptions occurring while GLAutoDrawable processing [part-2]

In case of an exception thrown within an GLEventListener
called off-thread by Animator:
  - Animator shall stop
  - Animator shall forward the exception

GLDrawableHelper shall also flush all queued GLRunnable tasks
in case of an exception, so that another thread waiting until it's completion
is notified and continues processing.

3 years agoBug 1028 - AMD Windows driver thread hinders JVM process to exit/end, caused by _not_...
Sven Gothel [Tue, 5 Aug 2014 15:09:40 +0000 (17:09 +0200)]
Bug 1028 - AMD Windows driver thread hinders JVM process to exit/end, caused by _not_ destroying the SharedResource context

SharedResourceRunner stop() method
is invoked by JMV shutdown hook and GLProfile/GLDrawableFactory.shutdown*().

It shall issue SharedResource.releaseSharedResource() for all implementations,
e.g. X11/GLX and Windows/WGL.


Root cause is a GL driver thread keeping the process alive.


On X11/GLX we destroy the shared context and the shared drawable.

On Windows/WGL we only destroy the shared drawable,
knowing that destroying the shared context caused a driver bug in the past.

Will enable the shared context destruction, which is the proper way.


Commiting this patch to see whether our jenkins builds won't crash
due to previous experienced issues.

3 years agoBug 1036: Renamed property to switch off NVidia Windows workaround '
Sven Gothel [Mon, 4 Aug 2014 12:10:47 +0000 (14:10 +0200)]
Bug 1036: Renamed property to switch off NVidia Windows workaround '' (dropping '.debug')

3 years agoFix NPE regression of commit ba1ffe66697c3175b423cb7ab9b686d73959708d
Sven Gothel [Thu, 31 Jul 2014 05:32:08 +0000 (07:32 +0200)]
Fix NPE regression of commit ba1ffe66697c3175b423cb7ab9b686d73959708d

3 years agoBug 1039 - Specify behavior of GLEventListener Exceptions occurring while GLAutoDrawa...
Sven Gothel [Thu, 31 Jul 2014 00:42:39 +0000 (02:42 +0200)]
Bug 1039 - Specify behavior of GLEventListener Exceptions occurring while GLAutoDrawable processing [part-1]

Implements Specification as described on 'Bug 1039 Comment 1'

  - Offthread exception handler


GLDrawableHelper is used in all GLAutoDrawable implementations
and for most operations.

GLAutoDrawable/GLDrawableHelper invoke(..) method:

- invoke(..) forwards a caught exception
  - if blocking, it forwards an exception
    happening within the passed GLRunnable(s).
    Here the exception is caught, printed
    and then thrown by invoke itself.

  - if non-blocking, an exception
    happening within the passed GLRunnable(s)
    will be thrown in the thread issuing it's execution,
    i.e. display() call.
    Here the exception is not caught and simply thrown
    by the GLRunnable.

GLAutoDrawable.destroy() -> GLDrawableHelper.disposeGL(..) method:

- disposeAllGLEventListener() being invoked by disposeGL(..),
  catches exception thrown by GLEventListener.dispose(..)
  and prints them to stderr.
  The first caught exception is re-thrown at the end as an GLException.

- disposeGL() catches re-thrown GLException by disposeAllGLEventListener()
  for GLEventListener.dispose(..)
  and re-throws it when operation is complete.

- disposeGL() catches an exception thrown at context destruction or release
  and re-throws it when operation is complete.
  An early exception at context.makeCurrent() is _not_ caught,
  since it is the first operation which simply shall unwind the stack.

GLAutoDrawable.display() -> GLDrawableHelper.invokeGLImpl(..) method:

- invokeGLImpl(..) for display() follows disposeGL() mechanism, i.e.
  it catches exception thrown at
  GLEventListener's init(..), reshape(..) and display(..) methods
  and re-throws it when operation is complete.

  It also catches an exception thrown at context release
  and re-throws it when operation is complete.
  An early exception at context.makeCurrent() is _not_ caught,
  since it is the first operation which simply shall unwind the stack.


None of the above thrown exception shall be caught and suppressed
on the caller side.

If an operation must be completed while an exception is caught,
it shall be cached and re-thrown after the operations.

In case multiple exception at multiple places are caught within
an operation, they all shall be cached and the first one
shall be re-thrown.

In case of multiple exception from the same place,
i.e. a loop through all GLEventListener,
the first shall be cached and re-thrown after operation is completed.

It has to be determined, whether we like to dump the exceptions,
especially the ones who get suppressed in case of multiple exceptions.

3 years agoBug 830 - Refine Heuristics for to query whether GLDrawableUtil.swapGLContextAndAllGL...
Sven Gothel [Wed, 30 Jul 2014 22:54:24 +0000 (00:54 +0200)]
Bug 830 - Refine Heuristics for to query whether GLDrawableUtil.swapGLContextAndAllGLEventListener is safe: Add Accumulator Buffer bits

3 years agoGLDrawable: Expose getRequestedGLCapabilities() (Include to public API)
Sven Gothel [Wed, 30 Jul 2014 22:48:40 +0000 (00:48 +0200)]
GLDrawable: Expose getRequestedGLCapabilities() (Include to public API)

In certain cases, it is required to read the user requested capabilities
from places other than the user code.

Hence adding public method to GLDrawable interface.

This removes the need to cast to private GLDrawableImpl,
which included such method.

3 years agoRefine test ff5dba28610b4f680c9320e9e52669ed54d4de43: Perform context switch on GL...
Sven Gothel [Wed, 30 Jul 2014 20:05:52 +0000 (22:05 +0200)]
Refine test ff5dba28610b4f680c9320e9e52669ed54d4de43: Perform context switch on GL capable thread if required. Add API doc note about this requirement.

3 years agoFix commit adf8e6e40aa9513036864489642cfef252804d08 (Bug 1036): long -> jlong in...
Sven Gothel [Wed, 30 Jul 2014 18:41:08 +0000 (20:41 +0200)]
Fix commit adf8e6e40aa9513036864489642cfef252804d08 (Bug 1036): long -> jlong in JNI func spec

3 years agoBug 830 - Add Heuristics for to query whether GLDrawableUtil.swapGLContextAndAllGLEve...
Sven Gothel [Wed, 30 Jul 2014 18:13:34 +0000 (20:13 +0200)]
Bug 830 - Add Heuristics for to query whether GLDrawableUtil.swapGLContextAndAllGLEventListener is safe (Doesn't work w/ pre MSAA onscreen drawable)

GLDrawableUtil.isSwapGLContextSafe(..) allows user to query whether 'we think' it's safe
to utilize swapping of GLContext between GLAutoDrawable instances.

Currently known unsafe cases are:
  - between on- and offscreen and one of the following:
    - MSAA involved, or
    - STEREO involved

Enhanced unit tests in this regard:
  - TestGLContextDrawableSwitch02AWT
    - using GLContextDrawableSwitchBase0
  - TestGLContextDrawableSwitch02NEWT
    - using GLContextDrawableSwitchBase0

Utilized safe query for setupPrint(..) action in:
  - AWT GLCanvas
  - AWT GLJPanel
  - NewtCanvasAWT

3 years agoBug 1036: NVidia's Windows Driver Threaded optimization: Alternative affinity mask...
Sven Gothel [Wed, 30 Jul 2014 14:21:17 +0000 (16:21 +0200)]
Bug 1036: NVidia's Windows Driver Threaded optimization: Alternative affinity mask setting on all threads of process - Didn't work (disabled)

3 years agoBug 1038 - Fix: Allow skipping detection of certain GLProfiles: Skip 'ARB_create_context'
Sven Gothel [Wed, 30 Jul 2014 02:51:14 +0000 (04:51 +0200)]
Bug 1038 - Fix: Allow skipping detection of certain GLProfiles: Skip 'ARB_create_context'

Commit e5a55ede324ce500f50991d56491758803063a58 was incomplete,
i.e. it lacked the required mappings for the non ARB profile, i.e.:
  GL4bc -> GL3bc, etc.

These profile mappings have been added now.


Further more, GLContext's profile queries, isGL*()
test the ctxOptions for CTX_IS_ARB_CREATED.
This has to be removed to properly work w/ Skip 'ARB_create_context'.

To remove the risk of inconcistency, i.e. context created via ARB and non-ARB,
the 'GLX/WGL profile >= GL3 via non ARB' validation removed
in commit e5a55ede324ce500f50991d56491758803063a58 has been brought back
and refined. Note:
  if( glp.isGL3() && createContextARBTried ) {
    // We shall not allow context creation >= GL3 w/ non ARB methods if ARB is used,
    // otherwise context of similar profile but different creation method may not be share-able.
This limited validation removes the possibility of such having a context
of same profile, one created via ARB and one without.
Hence also validates the isGL*() change, where the CTX_IS_ARB_CREATED criteria is removed.


Note regarding commit 7314b47ae1e42997e9e6974b84709640f0ac2a1b (revert):

While analyzing the mapping, it turns out that commit c8b99d197769eaec53c2def562c0ef3fc0e6a9d2
  "Don't map compatibility profiles to core profile if the latter are not available (restrict profile aliasing)"
is not fully consistent with GLProfile's and GLContext's profile queries, i.e. isGL*().
We may reiterate over this change .. but have it be reverted for now.

3 years agoRevert "GLContext: Don't map compatibility profiles to core profile if the latter...
Sven Gothel [Wed, 30 Jul 2014 02:24:46 +0000 (04:24 +0200)]
Revert "GLContext: Don't map compatibility profiles to core profile if the latter are not available (restrict profile aliasing) ; GLProfile does this - Simplification."

This reverts commit c8b99d197769eaec53c2def562c0ef3fc0e6a9d2.

3 years agoBug 1038 - Allow skipping detection of certain GLProfiles: Skip 'ARB_create_context...
Sven Gothel [Wed, 30 Jul 2014 01:06:30 +0000 (03:06 +0200)]
Bug 1038 - Allow skipping detection of certain GLProfiles: Skip 'ARB_create_context' context creation extension via property 'jogl.disable.openglarbcontext'; ...

Only allow the exclusions if platform OS is not OSX:
  - jogl.disable.openglcore
  - jogl.disable.openglarbcontext

Since on OSX they are known to work reliable and there is not other method
if receiving a higher GL profile than core and ARB.

This also removes the restrictions on X11 and Windows,
where profiles >= GL3 must be created using ARB_create_context.
Hence this is allowed now.

3 years agoGLContext: Don't map compatibility profiles to core profile if the latter are not...
Sven Gothel [Wed, 30 Jul 2014 00:37:47 +0000 (02:37 +0200)]
GLContext: Don't map compatibility profiles to core profile if the latter are not available (restrict profile aliasing) ; GLProfile does this - Simplification.

GLContext shall not map compatibility profiles to core profile if the latter is are not available (restrict profile aliasing).

If a user requests a straight GL3 core profile, don't answer with a compatibility profile, e.g. GL4bc.

Hence this patch exposes the true GL profile situation more honestly the the user!

User can already query profile mappings via GLProfile, i.e.
  - GL2GL3,
  - GL4ES3, or via
  - getMaximum
  - getMaxFixedFunc
  - getMaxProgrammable
  - getMaxProgrammableCore

This also fixes an issue when a user requests 'getMaxProgrammableCore'
but would receive a GL4bc profile.

3 years agoBug 1038 - Allow skipping detection of certain GLProfiles: Skip native core profiles...
Sven Gothel [Tue, 29 Jul 2014 20:55:56 +0000 (22:55 +0200)]
Bug 1038 - Allow skipping detection of certain GLProfiles: Skip native core profiles via property 'jogl.disable.openglcore'

Also moved all GL profile properties to GLProfile class and made them public for better documentation.

3 years agoFixedFuncHook: Use downstream gl.getGLProfile().getImpl() for custom GL2ES1 impl...
Sven Gothel [Tue, 29 Jul 2014 19:13:17 +0000 (21:13 +0200)]
FixedFuncHook: Use downstream gl.getGLProfile().getImpl() for custom GL2ES1 impl. profile, reflecting true implementing profile.

3 years agoFix ShaderCode.es3_default_precision_fp: ES 3.x requires same precision for uniforms...
Sven Gothel [Tue, 29 Jul 2014 19:12:23 +0000 (21:12 +0200)]
Fix ShaderCode.es3_default_precision_fp: ES 3.x requires same precision for uniforms -> re-use es3_default_precision_vp; Fixes PointsDemoES2

3 years agoBuildComposablePipeline: Handle synthetic isGL* and getGL* more generic, allow FixedF...
Sven Gothel [Tue, 29 Jul 2014 18:52:10 +0000 (20:52 +0200)]
BuildComposablePipeline: Handle synthetic isGL* and getGL* more generic, allow FixedFunctionHook to properly determine it's identity

BuildComposablePipeline: Handle synthetic isGL* and getGL* more generic, allow using a prologue hook
as needed for FixedFunctionHook's 'isGL*core()', 'isGLES*Compatible()' and 'getGLProfile()' methods.

The latter FixedFunctionHook take the emulated GL profile GL2ES1 into account,
allowing JOGL code to assume only having GL2ES1 available.
Otherwise methods like Texture.enable(..) would skip the glEnable(TEXTURE_2D)
call and FixedFunctionHook could not enable it's usage.

GLProfile received a 'public static GLProfile createCustomGLProfile(final String profile, final GLProfile profileImpl)'
allowing utilities like FixedFunctionHook to create a generic profile.

BuildComposablePipeline sorts the methods before emitting for better readability.

3 years agoGLContext.hasNoDefaultVAO(): Remove ES 3.x, fixing issues w/ ES 3.x client side verte...
Sven Gothel [Tue, 29 Jul 2014 18:43:06 +0000 (20:43 +0200)]
GLContext.hasNoDefaultVAO(): Remove ES 3.x, fixing issues w/ ES 3.x client side vertex arrays

Commit 6136457f10d020c779adc78641d0048f77ab1635 defined hasNoDefaultVAO() as [ GL4, GL3, GLES3 ],
however ES 3.x still supports (deprecated):
  - client side vertex arrays
  - default vertex array object (VAO)

Setting a custom VAO leads to GL_INVALID_OPERATION for client side vertex arrays
used w/ glVertexPointer(..).

Hence removing GLES3 from hasNoDefaultVAO(). git info: FAQ, tutorial and man pages.