bug 523

classic Classic list List threaded Threaded
5 messages Options
Reply | Threaded
Open this post in threaded view
|

bug 523

Michael Weber
I logged bug 523 that I encountered with the Sept 16 2011 signed release build. Seems to be fix in the latest source, but I logged the bug in order to submit a new unit test case for consideration that tests for this issue. I've submitted the pull request via github as per instructions. I will be away for a week, in case there are any questions / issues with the submission, will be back November 14.
Reply | Threaded
Open this post in threaded view
|

Re: bug 523

Wade Walker
Administrator
Thanks Michael! We really appreciate new test cases like you've submitted, since they insure the bug won't show up again in the future. Nice comments in that new unit test, by the way

Sven is right in the middle of working on some difficult OS X issues, so it may be a while before he can incorporate your changes safely, but feel free to ping back later if you don't see the pull request resolved.
Reply | Threaded
Open this post in threaded view
|

Re: bug 523

Sven Gothel
Administrator
On Saturday, November 05, 2011 02:24:53 AM Wade Walker [via jogamp] wrote:
>
> Thanks Michael! We really appreciate new test cases like you've submitted,
> since they insure the bug won't show up again in the future. Nice comments
> in that new unit test, by the way

Yup, thanks a lot - really appreciated.

>
> Sven is right in the middle of working on some difficult OS X issues, so it
> may be a while before he can incorporate your changes safely, but feel free
> to ping back later if you don't see the pull request resolved.

yup :)
Reply | Threaded
Open this post in threaded view
|

Re: bug 523

Michael Weber
My pleasure, just a minor contribution. I use a Mac at home, so, I'm happy to see you are supporting Mac.
Reply | Threaded
Open this post in threaded view
|

Re: bug 523

Sven Gothel
Administrator
On Monday, November 14, 2011 06:25:48 PM Michael Weber [via jogamp] wrote:
>
> My pleasure, just a minor contribution. I use a Mac at home, so, I'm happy to
> see you are supporting Mac.

Thx Michael, I have reviewed your test case and saw the flaw in GLDrawableFactory's
exposure of the getOrCreateSharedContext(..) method.

Fixed it and modified your test case - commit 12342c9e6122051fbdc91493c4d63515ad4613d1

~Sven