Re: [10] Review request for 8185634, 8185634: Java Fx-Swing dialogs appearing behind main stage

[Alexander Zvegintsev]

HI Semyon,

Please see my answer to Dmitry


Hi Dmitry,

 From my understanding JavaFX stage can't be easily integrated in JDK to 
support orderWindow() approach,

addChildWindow() is a native(and the simplest) way to maintain one window above 
other one, should be called only once.

IIUC the main concert of JDK-8080729 that child windows jumping to parent's 
display upon focus receiving, this is not an issue with current fix,

because addChildWindow() will be called only upon dialog creation in case of 
JavaFX-Swing interop.

Jump may happen if user want to create a child swing dialog on display other 
than JavaFX stage's one,

but such rare scenario can be easily workarounded on a user side by calling 
setLocation() right after setVisible() call.

So I would prefer to use addChildWindow() to make this fix as simple as 
possible.


Thanks,
Alexander.

On 08/11/2017 02:45, Semyon Sadetsky wrote:


Hi Alexander,

In CPlatformWindow.java change you used addChildWindow(), but we get 
rig of addChildWindow() in 8080729 and start to manage windows order 
on java side. Can you test that this change doesn't cause any ordering 
issues with modal and non-modal child and sibling windows on mac?


--Semyon

On 11/07/2017 10:11 AM, Alexander Zvegintsev wrote:


Hi Sergey,

I am not able to crash it on several platforms, except one case:

if we are terminating JavaFX application while EDT processing some 
long task. But it is unrelated to the fix and reproducible on current 
builds.


I've filed a separate bug JDK-8190329 
<https://bugs.openjdk.java.net/browse/JDK-8190329>.


Will the current fix cover the native dialogs like 
FileDialog/PrintDialog on linux and windows? 
It will not, Windows already fixed by JDK-8088395 
<https://bugs.openjdk.java.net/browse/JDK-8088395>



Test added:
http://cr.openjdk.java.net/~azvegint/jdk/10/8185634/05/
Thanks,
Alexander.
On 13/10/2017 01:14, Sergey Bylokhov wrote:

Looks fine.
I am not sure but it looks like the fix has an assumption that the 
CPlatformWindow.setVisible() code will be executed on EDT/Appkit but 
it is not the case. This code can be executed on any 
thread(intentionally for crash), and it will be good to check that 
it works as expected by some stress test which will try to force the 
possible crash: 4 threads:

 - show/dispose Swing Node
 - show/dispose Dialog1/2/3 using different timeouts

Will the current fix cover the native dialogs like 
FileDialog/PrintDialog on linux and windows?


On 10/10/2017 13:54, Kevin Rushforth wrote:

Note that there is now a 04 version.

It looks good to me, although someone more familiar with AWT should 
also check the AWT changes.


We will need a test program for this (as a follow-on issue if not 
now).


-- Kevin


Alexander Zvegintsev wrote:

Please review the updated version

http://cr.openjdk.java.net/~azvegint/jdk/10/8185634/02/
Now we are postponing actual window closing, it happens only after 
we ensure that native window pointer is valid on Swing side.


Thanks,
Alexander.

On 23/09/2017 08:01, Sergey Bylokhov wrote:

Hi, Alexander.
How can we be sure that the parent frame will not be disposed 
while we use a pointer?


long ownerWindowPtr = peer.getOverridenWindowHandle();
<<<<< Dispose the peer
if (ownerWindowPtr != 0) {
    //Place window above JavaFX stage
    CWrapper.NSWindow.addChildWindow(
    ownerWindowPtr, ptr, CWrapper.NSWindow.NSWindowAbove);
<<<<< Boom
}


On 9/21/17 22:56, Alexander Zvegintsev wrote:

Hi Phil,

Please review the updated fix with reflection incorporated
http://cr.openjdk.java.net/~azvegint/jdk/10/8185634/01/

New issue created JDK-8187803 
<https://bugs.openjdk.java.net/browse/JDK-8187803> as JDK 
counterpart of this issue.


Thanks,
Alexander.

On 21/09/2017 22:25, Phil Race wrote:

Some procedural comments :
Since 90% of this is in AWT code, I'd have thought awt-dev 
should be included here.

I've added that list.

And apart from needing separate bug ids, I don't see why the 
bug below is confidential.



I agree with what Kevin pointed out off-line that as in the 
dialog case, the FX side
of the code can use reflection and simply be a harmless 
non-functional no-op

if the SwingAccessor does not provide the new method.

BTW
264 inline HWND GetOverridenHWnd() { return m_overridenHwnd; }
should be "dd" not "d".

-phil.

On 09/21/2017 03:38 AM, Alexander Zvegintsev wrote:

Hello,

please review the fix

http://cr.openjdk.java.net/~azvegint/jdk/10/8185634/00/

for the issue

https://bugs.openjdk.java.net/browse/JDK-8185634

Re: [10] Review request for 8185634, 8185634: Java Fx-Swing dialogs appearing behind main stage

[Alexander Zvegintsev]

Hi Sergey,

I am not able to crash it on several platforms, except one case:

if we are terminating JavaFX application while EDT processing some long 
task. But it is unrelated to the fix and reproducible on current builds.


I've filed a separate bug JDK-8190329 
<https://bugs.openjdk.java.net/browse/JDK-8190329>.


Will the current fix cover the native dialogs like 
FileDialog/PrintDialog on linux and windows? 
It will not, Windows already fixed by JDK-8088395 
<https://bugs.openjdk.java.net/browse/JDK-8088395>



Test added:
http://cr.openjdk.java.net/~azvegint/jdk/10/8185634/05/

Thanks,
Alexander.

On 13/10/2017 01:14, Sergey Bylokhov wrote:

Looks fine.
I am not sure but it looks like the fix has an assumption that the 
CPlatformWindow.setVisible() code will be executed on EDT/Appkit but 
it is not the case. This code can be executed on any 
thread(intentionally for crash), and it will be good to check that it 
works as expected by some stress test which will try to force the 
possible crash: 4 threads:

 - show/dispose Swing Node
 - show/dispose Dialog1/2/3 using different timeouts

Will the current fix cover the native dialogs like 
FileDialog/PrintDialog on linux and windows?


On 10/10/2017 13:54, Kevin Rushforth wrote:

Note that there is now a 04 version.

It looks good to me, although someone more familiar with AWT should 
also check the AWT changes.


We will need a test program for this (as a follow-on issue if not now).

-- Kevin


Alexander Zvegintsev wrote:

Please review the updated version

http://cr.openjdk.java.net/~azvegint/jdk/10/8185634/02/
Now we are postponing actual window closing, it happens only after 
we ensure that native window pointer is valid on Swing side.


Thanks,
Alexander.

On 23/09/2017 08:01, Sergey Bylokhov wrote:

Hi, Alexander.
How can we be sure that the parent frame will not be disposed while 
we use a pointer?


long ownerWindowPtr = peer.getOverridenWindowHandle();
<<<<< Dispose the peer
if (ownerWindowPtr != 0) {
    //Place window above JavaFX stage
    CWrapper.NSWindow.addChildWindow(
    ownerWindowPtr, ptr, CWrapper.NSWindow.NSWindowAbove);
<<<<< Boom
}


On 9/21/17 22:56, Alexander Zvegintsev wrote:

Hi Phil,

Please review the updated fix with reflection incorporated
http://cr.openjdk.java.net/~azvegint/jdk/10/8185634/01/

New issue created JDK-8187803 
<https://bugs.openjdk.java.net/browse/JDK-8187803> as JDK 
counterpart of this issue.


Thanks,
Alexander.

On 21/09/2017 22:25, Phil Race wrote:

Some procedural comments :
Since 90% of this is in AWT code, I'd have thought awt-dev should 
be included here.

I've added that list.

And apart from needing separate bug ids, I don't see why the bug 
below is confidential.



I agree with what Kevin pointed out off-line that as in the 
dialog case, the FX side
of the code can use reflection and simply be a harmless 
non-functional no-op

if the SwingAccessor does not provide the new method.

BTW
264 inline HWND GetOverridenHWnd() { return m_overridenHwnd; }
should be "dd" not "d".

-phil.

On 09/21/2017 03:38 AM, Alexander Zvegintsev wrote:

Hello,

please review the fix

http://cr.openjdk.java.net/~azvegint/jdk/10/8185634/00/

for the issue

https://bugs.openjdk.java.net/browse/JDK-8185634

[10] Review request for 8181406: [linux] Stage resizing bug

[Alexander Zvegintsev]

Hi all,

please review the fix

http://cr.openjdk.java.net/~azvegint/jdk/10/8181406/00/

for the issue

https://bugs.openjdk.java.net/browse/JDK-8181406

--
Thanks,

Alexander.

Re: CFV: New OpenJFX Committer: Ambarish Rapte

[Alexander Zvegintsev]

Vote: yes

Thanks,
Alexander.

On 01/11/2017 01:48, Kevin Rushforth wrote:

I hereby nominate Ambarish Rapte [1] to OpenJFX Committer.

Ambarish is a member of JavaFX team at Oracle, who has contributed 10 
changesets [2][3] to OpenJFX.


Votes are due by November 14, 2017.

Only current OpenJFX Committers [4] are eligible to vote on this 
nomination. Votes must be cast in the open by replying to this mailing 
list.


For Lazy Consensus voting instructions, see [5]. Nomination to a 
project Committer is described in [6].


Thanks.

-- Kevin

[1] http://openjdk.java.net/census#arapte

[2] 
http://hg.openjdk.java.net/openjfx/10-dev/rt/log?revcount=20&rev=author%28arapte%29

[3] http://hg.openjdk.java.net/openjfx/10-dev/rt/rev/f7d38650fa15

[4] http://openjdk.java.net/census#openjfx

[5] http://openjdk.java.net/bylaws#lazy-consensus

[6] http://openjdk.java.net/projects#project-committer

[10] Review request for 8165562: Upgrade to newer version of SWT for build/test

[Alexander Zvegintsev]

Hello,

please review the fix

http://cr.openjdk.java.net/~azvegint/jdk/10/8165562/00/

for the issue

https://bugs.openjdk.java.net/browse/JDK-8165562

--
Thanks,
Alexander.

[10] Review request 8185827: SEGV in com.sun.glass.ui.mac.MacCommonDialogs.

[Alexander Zvegintsev]

Hello,

please review the fix

http://cr.openjdk.java.net/~azvegint/jdk/10/8185827/01/

for the issue

https://bugs.openjdk.java.net/browse/JDK-8185827

--
Thanks,
Alexander.

Re: [10] Review request for 8185634, 8185634: Java Fx-Swing dialogs appearing behind main stage

[Alexander Zvegintsev]

Please review the updated version

http://cr.openjdk.java.net/~azvegint/jdk/10/8185634/02/
Now we are postponing actual window closing, it happens only after we 
ensure that native window pointer is valid on Swing side.


Thanks,
Alexander.

On 23/09/2017 08:01, Sergey Bylokhov wrote:

Hi, Alexander.
How can we be sure that the parent frame will not be disposed while we 
use a pointer?


long ownerWindowPtr = peer.getOverridenWindowHandle();
<<<<< Dispose the peer
if (ownerWindowPtr != 0) {
    //Place window above JavaFX stage
    CWrapper.NSWindow.addChildWindow(
    ownerWindowPtr, ptr, CWrapper.NSWindow.NSWindowAbove);
<<<<< Boom
}


On 9/21/17 22:56, Alexander Zvegintsev wrote:

Hi Phil,

Please review the updated fix with reflection incorporated
http://cr.openjdk.java.net/~azvegint/jdk/10/8185634/01/

New issue created JDK-8187803 
<https://bugs.openjdk.java.net/browse/JDK-8187803> as JDK counterpart 
of this issue.


Thanks,
Alexander.

On 21/09/2017 22:25, Phil Race wrote:

Some procedural comments :
Since 90% of this is in AWT code, I'd have thought awt-dev should be 
included here.

I've added that list.

And apart from needing separate bug ids, I don't see why the bug 
below is confidential.



I agree with what Kevin pointed out off-line that as in the dialog 
case, the FX side
of the code can use reflection and simply be a harmless 
non-functional no-op

if the SwingAccessor does not provide the new method.

BTW
264 inline HWND GetOverridenHWnd() { return m_overridenHwnd; }
should be "dd" not "d".

-phil.

On 09/21/2017 03:38 AM, Alexander Zvegintsev wrote:

Hello,

please review the fix

http://cr.openjdk.java.net/~azvegint/jdk/10/8185634/00/

for the issue

https://bugs.openjdk.java.net/browse/JDK-8185634

Re: CFV: New OpenJFX Committer: Prasanta Sadhukhan

[Alexander Zvegintsev]

Vote: YES

Thanks,
Alexander.

On 28/09/2017 06:40, Kevin Rushforth wrote:

I hereby nominate Prasanta Sadhukhan [1] to OpenJFX Committer.

Prasanta is a member of JavaFX team at Oracle, who has contributed 12 
changesets [2] to OpenJFX.


Votes are due by October 11, 2017.

Only current OpenJFX Committers [3] are eligible to vote on this 
nomination. Votes must be cast in the open by replying to this mailing 
list.


For Lazy Consensus voting instructions, see [4]. Nomination to a 
project Committer is described in [5].


Thanks.

-- Kevin

[1] http://openjdk.java.net/census#psadhukhan

[2] 
http://hg.openjdk.java.net/openjfx/10-dev/rt/log?revcount=20&rev=author%28psadhukhan%29


[3] http://openjdk.java.net/census#openjfx

[4] http://openjdk.java.net/bylaws#lazy-consensus

[5] http://openjdk.java.net/projects#project-committer

Re: [10] Review request for 8185634, 8185634: Java Fx-Swing dialogs appearing behind main stage

[Alexander Zvegintsev]

Hi Sergey,

Thanks for catching this, this is an issue which should be addressed.

Thanks,
Alexander.

On 23/09/2017 08:01, Sergey Bylokhov wrote:

Hi, Alexander.
How can we be sure that the parent frame will not be disposed while we 
use a pointer?


long ownerWindowPtr = peer.getOverridenWindowHandle();
<<<<< Dispose the peer
if (ownerWindowPtr != 0) {
    //Place window above JavaFX stage
    CWrapper.NSWindow.addChildWindow(
    ownerWindowPtr, ptr, CWrapper.NSWindow.NSWindowAbove);
<<<<< Boom
}


On 9/21/17 22:56, Alexander Zvegintsev wrote:

Hi Phil,

Please review the updated fix with reflection incorporated
http://cr.openjdk.java.net/~azvegint/jdk/10/8185634/01/

New issue created JDK-8187803 
<https://bugs.openjdk.java.net/browse/JDK-8187803> as JDK counterpart 
of this issue.


Thanks,
Alexander.

On 21/09/2017 22:25, Phil Race wrote:

Some procedural comments :
Since 90% of this is in AWT code, I'd have thought awt-dev should be 
included here.

I've added that list.

And apart from needing separate bug ids, I don't see why the bug 
below is confidential.



I agree with what Kevin pointed out off-line that as in the dialog 
case, the FX side
of the code can use reflection and simply be a harmless 
non-functional no-op

if the SwingAccessor does not provide the new method.

BTW
264 inline HWND GetOverridenHWnd() { return m_overridenHwnd; }
should be "dd" not "d".

-phil.

On 09/21/2017 03:38 AM, Alexander Zvegintsev wrote:

Hello,

please review the fix

http://cr.openjdk.java.net/~azvegint/jdk/10/8185634/00/

for the issue

https://bugs.openjdk.java.net/browse/JDK-8185634

Re: [10] Review request for 8185634, 8185634: Java Fx-Swing dialogs appearing behind main stage

[Alexander Zvegintsev]

Hi Dmitry,

From my understanding JavaFX stage can't be easily integrated in JDK to 
support orderWindow() approach,


addChildWindow() is a native(and the simplest) way to maintain one 
window above other one, should be called only once.


IIUC the main concert of JDK-8080729 that child windows jumping to 
parent's display upon focus receiving, this is not an issue with current 
fix,


because addChildWindow() will be called only upon dialog creation in 
case of JavaFX-Swing interop.


Jump may happen if user want to create a child swing dialog on display 
other than JavaFX stage's one,


but such rare scenario can be easily workarounded on a user side by 
calling setLocation() right after setVisible() call.


So I would prefer to use addChildWindow() to make this fix as simple as 
possible.



Thanks,
Alexander.

On 23/09/2017 21:21, Dmitry Markov wrote:

Hi Alexander,

In CPlatformWindow class you call CWrapper.NSWindow.addChildWindow() to place a 
window above JavaFX stage. The usage of addChilWindow() or/and 
removeChildWindow() may cause ‘jumping window issue’ in multi-monitor 
environment, see JDK-8080729 for more details.

To avoid possible problems in multi monitor set-up I suggest that you should 
replace addChildWindow() with orderWindow() or use some another approach to 
locate the window above the stage.

Thanks,
Dmitry

On 22 Sep 2017, at 06:56, Alexander Zvegintsev 
 wrote:

Hi Phil,

Please review the updated fix with reflection incorporated
http://cr.openjdk.java.net/~azvegint/jdk/10/8185634/01/

New issue created JDK-8187803 
<https://bugs.openjdk.java.net/browse/JDK-8187803> as JDK counterpart of this 
issue.

Thanks,
Alexander.

On 21/09/2017 22:25, Phil Race wrote:

Some procedural comments :
Since 90% of this is in AWT code, I'd have thought awt-dev should be included 
here.
I've added that list.

And apart from needing separate bug ids, I don't see why the bug below is 
confidential.


I agree with what Kevin pointed out off-line that as in the dialog case, the FX 
side
of the code can use reflection and simply be a harmless non-functional no-op
if the SwingAccessor does not provide the new method.

BTW
264 inline HWND GetOverridenHWnd() { return m_overridenHwnd; }
should be "dd" not "d".

-phil.

On 09/21/2017 03:38 AM, Alexander Zvegintsev wrote:

Hello,

please review the fix

http://cr.openjdk.java.net/~azvegint/jdk/10/8185634/00/

for the issue

https://bugs.openjdk.java.net/browse/JDK-8185634

Re: [10] Review request for 8185634, 8185634: Java Fx-Swing dialogs appearing behind main stage

[Alexander Zvegintsev]

Hi Phil,

Please review the updated fix with reflection incorporated
http://cr.openjdk.java.net/~azvegint/jdk/10/8185634/01/

New issue created JDK-8187803 
<https://bugs.openjdk.java.net/browse/JDK-8187803> as JDK counterpart of 
this issue.


Thanks,
Alexander.

On 21/09/2017 22:25, Phil Race wrote:

Some procedural comments :
Since 90% of this is in AWT code, I'd have thought awt-dev should be 
included here.

I've added that list.

And apart from needing separate bug ids, I don't see why the bug below 
is confidential.



I agree with what Kevin pointed out off-line that as in the dialog 
case, the FX side
of the code can use reflection and simply be a harmless non-functional 
no-op

if the SwingAccessor does not provide the new method.

BTW
264 inline HWND GetOverridenHWnd() { return m_overridenHwnd; }
should be "dd" not "d".

-phil.

On 09/21/2017 03:38 AM, Alexander Zvegintsev wrote:

Hello,

please review the fix

http://cr.openjdk.java.net/~azvegint/jdk/10/8185634/00/

for the issue

https://bugs.openjdk.java.net/browse/JDK-8185634

[10] Review request for 185634: Java Fx-Swing dialogs appearing behind main stage

[Alexander Zvegintsev]

Hello,

please review the fix

http://cr.openjdk.java.net/~azvegint/jdk/10/8185634/00/

for the issue

https://bugs.openjdk.java.net/browse/JDK-8185634

--
Thanks,
Alexander.

[10] request for review 8181779: Memory leak in FX Glass Embedded Window

[Alexander Zvegintsev]

Hello,

please review the fix

http://cr.openjdk.java.net/~azvegint/jdk/10/8181779/00/

for the issue

https://bugs.openjdk.java.net/browse/JDK-8181779

--
Thanks,
Alexander.

[10] Review request for 8181476 [macos] Stages with StageStyle.UTILITY are always on-top when initialized without an owner

[Alexander Zvegintsev]

Hello,

please review the fix

http://cr.openjdk.java.net/~azvegint/jdk/10/8181476/00/

for the issue

https://bugs.openjdk.java.net/browse/JDK-8181476

--
Thanks,
Alexander.

[10] Review request for 8172847 [macos] If you hit the escape key repeatedly to close the subwindow, the process crashes

[Alexander Zvegintsev]

Hello,

please review the fix
http://cr.openjdk.java.net/~azvegint/jdk/10/8172847/00/
for the issue
https://bugs.openjdk.java.net/browse/JDK-8172847

--
Thanks,
Alexander.

Re: CFV: New OpenJFX Committer: Ajit Ghaisas

[Alexander Zvegintsev]

Vote: yes

--
Thanks,
Alexander.

On 26.05.2017 15:20, Kevin Rushforth wrote:

I hereby nominate Ajit Ghaisas [1] to OpenJFX Committer.

Ajit is a member of JavaFX team at Oracle, who has contributed 10 
changesets to OpenJFX. A list of these changesets is available by the 
following link:


http://hg.openjdk.java.net/openjfx/10-dev/rt/log?revcount=40&rev=author%28aghaisas%29 



Votes are due by June 9, 2017.

Only current OpenJFX Committers [2] are eligible to vote on this 
nomination. Votes must be cast in the open by replying to this mailing 
list.


For Lazy Consensus voting instructions, see [3]. Nomination to a 
project Committer is described in [4].


Thanks.

-- Kevin

[1] http://openjdk.java.net/census#aghaisas

[2] http://openjdk.java.net/census#openjfx

[3] http://openjdk.java.net/bylaws#lazy-consensus

[4] http://openjdk.java.net/projects#project-committer

Re: CFV: New OpenJFX Committer: Semyon Sadetsky

[Alexander Zvegintsev]

Vote: yes

--
Thanks,
Alexander.

On 25.01.2017 22:39, David Hill wrote:


I hereby nominate Semyon Sadetsky to OpenJFX Committer.

Semyon Sadetsky is part of the JavaFX team focusing on glass.

A list of Semyon's commits and reviews is available by the following 
links


http://hg.openjdk.java.net/openjfx/9-dev/rt/log?rev=author%28ssadetsky%29

Votes are due by Feb 9th, 2017.

Only current OpenJFX Committers [1] are eligible to vote on this 
nomination. Votes must be cast in the open by replying to this mailing 
list.


For Lazy Consensus voting instructions, see [2]. Nomination to a 
project Committer is described in [3].


[1] http://openjdk.java.net/census#openjfx

[2] http://openjdk.java.net/bylaws#lazy-consensus

[3] http://openjdk.java.net/projects#project-committer

Thanks,

Dave

Re: CFV: New OpenJFX Committer: Ramesh Gangadhar

[Alexander Zvegintsev]

Vote: yes

Thanks,
Alexander.

On 19/01/2017 00:53, Kevin Rushforth wrote:

I hereby nominate Ramesh Gangadhar [1] to OpenJFX Committer.

Ramesh is a member of JavaFX SQE team at Oracle working on test 
development for the Java packager, who has contributed 20 changesets 
to OpenJFX, at least 8 of which [5] are significant.


Votes are due by February 1, 2017.

Only current OpenJFX Committers [2] are eligible to vote on this 
nomination. Votes must be cast in the open by replying to this mailing 
list.


For Lazy Consensus voting instructions, see [3]. Nomination to a 
project Committer is described in [4].


Thanks,

-- Kevin


[1] http://openjdk.java.net/census#rgangadhar

[2] http://openjdk.java.net/census#openjfx

[3] http://openjdk.java.net/bylaws#lazy-consensus

[4] http://openjdk.java.net/projects#project-committer

[5]  List of significant changesets:
http://hg.openjdk.java.net/openjfx/9-dev/tests/rev/0b702e5db31c
http://hg.openjdk.java.net/openjfx/9-dev/tests/rev/0787febc9b94
http://hg.openjdk.java.net/openjfx/9-dev/tests/rev/b80a8efca46c
http://hg.openjdk.java.net/openjfx/9-dev/tests/rev/9fe019fe7f5c
http://hg.openjdk.java.net/openjfx/9-dev/tests/rev/bf6ae1d345ce
http://hg.openjdk.java.net/openjfx/9-dev/tests/rev/be0c0e2d0d31
http://hg.openjdk.java.net/openjfx/9-dev/tests/rev/b743144c5e11
http://hg.openjdk.java.net/openjfx/9-dev/tests/rev/0181e5e6b9bb
http://hg.openjdk.java.net/openjfx/9-dev/tests/rev/a141695f4a7f
http://hg.openjdk.java.net/openjfx/9-dev/tests/rev/16542d3f79a9
http://hg.openjdk.java.net/openjfx/9-dev/tests/rev/78f45d216138
http://hg.openjdk.java.net/openjfx/9-dev/tests/rev/92b56a32ebd1

[9] Review request for 8166106: JVM crash on resizing JavaFX application with title and icon

[Alexander Zvegintsev]

Hello,

please review the fix

http://cr.openjdk.java.net/~azvegint/jdk/9/8166106/00/

for the issue

https://bugs.openjdk.java.net/browse/JDK-8166106

--
Thanks,
Alexander.

Re: CFV: New OpenJFX Committer: Victor Drozdov

[Alexander Zvegintsev]

Vote: yes

Thanks,
Alexander.

On 12/14/16 3:21 AM, Chris Bensen wrote:

I hereby nominate Victor Drozdov [1] to OpenJFX Committer.

Victor is a member of Java Deployment team at Oracle working on the Java 
Packager tool, who has contributed 11 changesets [5] to OpenJFX, at least 8 of 
which are significant.

Votes are due by January 4, 2017.

Only current OpenJFX Committers [2] are eligible to vote on this nomination. 
Votes must be cast in the open by replying to this mailing list.

For Lazy Consensus voting instructions, see [3]. Nomination to a project 
Committer is described in [4].

Thanks,

[1] http://openjdk.java.net/census#vdrozdov 


[2] http://openjdk.java.net/census#openjfx 


[3] http://openjdk.java.net/bylaws#lazy-consensus 


[4] http://openjdk.java.net/projects#project-committer 


[5] List of changesets:

http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/81268b9c41f6 

http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/4272317a42ad 

http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/3ca13b05a699 

http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/6dfb3daf92cf 

http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/2b649093eb50 

http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/2538169ab7bb 

http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/a6fafaa51082 

http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/ac334db87ed6 

http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/12e6cb1e78af 

http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/32cc74546935 

http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/6b55a303625d 

[9] Review request for 8150175: Need replacement in FXDnD for DropTargetContext::addNotify, removeNotify

[Alexander Zvegintsev]

Hello,

please review the fix

http://cr.openjdk.java.net/~azvegint/jdk/9/8150175/00/

for the issue

https://bugs.openjdk.java.net/browse/JDK-8150175


--
Thanks,
Alexander.

Re: [9] Review request for 8087498 [Mac] : Full screen mode fails for certain StageStyles

[Alexander Zvegintsev]

please see the updated webrev

http://cr.openjdk.java.net/~azvegint/jdk/9/8087498/01/

Thanks,
Alexander.

On 8/4/16 3:40 AM, Alexander Zvegintsev wrote:

Hello,

please review the fix
http://cr.openjdk.java.net/~azvegint/jdk/9/8087498/00
for the issue
https://bugs.openjdk.java.net/browse/JDK-8087498

Re: [PATCH] 8161282: FXCanvas does not forward horizontal mouse scroll events to the embedded scene

[Alexander Zvegintsev]

Hi Alexander,

The fix looks good to me, except the wild card import change in 
FXCanvas.java(there is no need for a new patch, if it is the only issue 
found)



On 8/16/16 10:24 AM, Alexander Nyssen wrote:

You might even take the one I attached. I just recognized I still had some 
unused imports in the manual test case (I am still not familiar with IntelliJ).

Regards,
Alexander




Am 15.08.2016 um 19:26 schrieb Kevin Rushforth :

OK, I'll upload this revised version of the patch today.

-- Kevin


Alexander Nyssen wrote:

Hi Kevin,

please consider the following updated patch instead, which contains an 
additional null-check.

Regards
Alexander





Am 12.08.2016 um 16:04 schrieb Alexander Nyssen mailto:alexan...@nyssen.org>>:

Hi Kevin,

attached please find an initial patch for 
https://bugs.openjdk.java.net/browse/JDK-8161282 
.

The patch is not as minimal as I had hoped, as the EmbeddedSceneInterface had 
to be changed to differentiate between mouse and scroll events (while up to 
now, scroll events are handled as mouse events), but for me this seemed 
necessary to fix this issue properly. As a result, JFXPanel had to be adjusted 
as well to comply to the changes in the EmbeddedSceneInterface, while its 
behavior should not have changed.

As horizontal mouse events cannot be synthesized via Display.post(Event) yet 
(an open issue for SWT), I did not add an automated test, but instead added a 
manual one (FXCanvasMouseWheelEventsTest). Therefore, this patch does not 
depend on the patch I provided earlier for JDK-8160325.

Best Regards,
Alexander





=


--
Thanks,
Alexander.

Re: [PATCH] 8160325: Provide a public API to obtain the FXCanvas for an embedded scene.

[Alexander Zvegintsev]

Looks fine to me too.


On 8/16/16 1:45 AM, Kevin Rushforth wrote:
The 2016-08-12 revision looks fine to me, except for a missing space 
as noted in JBS (no need for a new patch if that is the only issue found).


While we wait for approval from the JDK 9 release team, we need 
another reviewer for this.


Alexander Z: can you take a look?

-- Kevin


Alexander Nyssen wrote:

Hi Kevin,

attached please find a revised patch that contains the corrections 
you requested. The patch is now applicable to the new module 
structure (modues with 'javafx.' prefix).


Regards,
Alexander





Am 12.08.2016 um 00:00 schrieb Kevin Rushforth 
mailto:kevin.rushfo...@oracle.com>>:




Alexander Nyssen wrote:

Hi Kevin,

thanks for your feedback. Please fin my comments inline.

   

Am 09.08.2016 um 03:10 schrieb Kevin Rushforth:

I uploaded the patch, reviewed it, and provided comments in the bug report. The 
short version is:

* The new API looks good

* There is a missing '@since 9' in the javadoc comments along with a few typos 
/ style issues
 

I will take care of it. Is there a style guide?
   


Not a current one. The ones I pointed out in the JBS issue were 
either grammatical or capitalization (other than the '@since' which 
is required for all new API).



* Rather than using reflection and setAccessible in the implementation, please 
add a public getHostContainer method to EmbeddedWindow (since it is an internal 
method, there is no concern with doing that -- it isn't API).
 

Such a getHost() method was already introduced to EmbeddedWindow as part of the 
patch (to obtain the HostContainer from it). The reflection code within 
getFXCanvas() is used to access the enclosing instance (i.e. the FXCanvas) of 
the HostContainer. We could only circumvent this by introducing a constructor 
to HostContainer and by passing in the enclosing FXCanvas instance explicitly 
(so we can query it via an explicit getter, which would have to be introduced 
in addition). Would you prefer that?
   


Ah, I see what you are doing now. There is an easier way, then. Just 
add a default (package) scope 'fxCanvas' variable in the inner class 
initialized to 'FXCanvas.this' and you will be able to access it 
directly, since an instance of an inner class can always get access 
to the instance of the enclosing class.


private class HostContainer implements HostInterface {
final FXCanvas fxCanvas = FXCanvas.this;
...

I updated the bug report with this and the other style issues. I 
haven't tested it yet, but other than the listed issues it looks 
very close to being done.


-- Kevin


   

Additionally, I requested JDK 9 release team approval for this. The approval 
process can proceed in parallel with your addressing the issues I raised.

— Kevin
 

Regards,
Alexander

   

Alexander Nyssen wrote:
 

Hi Kevin,

attached please find a revised patch. My comments are inlined.

   

Am 28.07.2016 um 18:03 schrieb Kevin Rushforth mailto:kevin.rushfo...@oracle.com>>:

Hi

Alexander Nyssen wrote:
 

Hi,

I have added my comments below:


   

Am 28.07.2016 um 17:22 schrieb Kevin Rushforth mailto:kevin.rushfo...@oracle.com>>:

I got the attachment, since Alexander also CCed me directly. I will attach it 
shortly.
 

Thanks!
   

Done.

 

I do have two comments on this:

1) We are past Feature Freeze, so all Enhancements need formal JDK 9 R-team 
approval [1][2]. In this case, the justification can be internal API that is no 
longer accessible in JDK 9 due to Jigsaw (I would be very reluctant to consider 
any other Enhancement request this late in the process), but I will need to 
look at it and then take it through the approval process, provided that I feel 
it is in scope.
 

I was not aware about this, but I would of course appreciate if it could be 
included (due to Jigsaw). Thanks for considering it at least.
   

I'll take a closer look tomorrow or Monday (no more time today). At first 
glance it seems like something reasonable to take forward.
 

That sounds promising. Thanks!

   

2) Some of the changes you list seem unrelated to this enhancement and are 
better done as separate issues (e.g., the rework of the SWTCursorsTest). Also, 
I am unconvinced of the need to force GTK 2; in fact it seems at odds with the 
work we have done with JEP 283 [3].
 

Well, the test case refactoring is somehow related, as I introduced the common 
SWT rule while introducing the second SWT test. However, I could provide it as 
a separate contribution if that was wished (and a JIRA issue was provided), but 
the rest of this contribution of course requires it as a prerequisite. If this 
enhancement could not be included in JDK 9, I would have to provide it as a 
separate contribution, as I would have to re-introduce FXCanvasTest in other 
succeeding bugfix contr

[9] Review request for 8089337 : JDK crash on Mac 10.10 with very large stage

[Alexander Zvegintsev]

Hello,

please review the fix
http://cr.openjdk.java.net/~azvegint/jdk/9/8089337/00
for the issue
https://bugs.openjdk.java.net/browse/JDK-8089337

--
--
Thanks,
Alexander.

[9] Review request for 8087498 [Mac] : Full screen mode fails for certain StageStyles

[Alexander Zvegintsev]

Hello,

please review the fix
http://cr.openjdk.java.net/~azvegint/jdk/9/8087498/00
for the issue
https://bugs.openjdk.java.net/browse/JDK-8087498

--
--
Thanks,
Alexander.

[9] Review request for 8163137 [Mac] : Maximizing a window with Cmd+Green button with Screen-Size hides it

[Alexander Zvegintsev]

Hello,

please review the fix
http://cr.openjdk.java.net/~azvegint/jdk/9/8163137/00
for the issue
https://bugs.openjdk.java.net/browse/JDK-8163137

--
--
Thanks,
Alexander.

Re: [PATCH] 8088147: FXCanvas: implement custom cursors (revised)

[Alexander Zvegintsev]

Looks good to me too.

--
Thanks,
Alexander.

On 28.07.2016 0:21, Kevin Rushforth wrote:
I have reviewed and tested the latest patch on Mac, Windows, Linux. I 
added my approval to the JIRA.


Now we just one more reviewer.

Alexander Z or Sergey: are either of you able to review?

-- Kevin


Alexander Nyssen wrote:

Hi Kevin,

sorry for that (I’m a Git guy and don’t feel very comfortable with 
Mercurial yet). I have attached a revised patch that (hopefully) 
provides the correct changes.


Regards,
Alexander


=



Am 27.07.2016 um 18:51 schrieb Kevin Rushforth 
mailto:kevin.rushfo...@oracle.com>>:


I attached the patch to the JBS issue. You have introduced a bug in 
the SWTCursorsTest.java JUnit test -- it appears to be an 
almost-exact copy of the manual test (including the now-incorrect 
class name). Also, you have inadvertently included an unwanted 
modification to .idea/modules.xml that should be reverted. Please 
send a new patch with the above two fixed.


The rest looks fine to me, but I will wait to verify until you 
provide an update to fix the unit test.


As for the JIGSAW mode tests, I agree that can/should be a follow-on 
effort.


-- Kevin


Alexander Nyssen wrote:

Hi Kevin, all,

attached please find an updated patch and my replies to Kevin’s 
comments at https://bugs.openjdk.java.net/browse/JDK-8088147:



build.gradle

1. In the following:

+ if (IS_JIGSAW_TEST) {
+ enabled = false // FIXME: JIGSAW -- support this with modules
+ logger.info ("JIGSAW Testing disabled for 
swt")


I verified that it correctly skips this in JIGSAW mode. Can you 
look into implementing this, though? It should not be difficult, 
and once we switch to only supporting Jigsaw mode the tests will 
never be run until this is done. If it does turn out to be too 
much work, then we could consider it as a follow-on.




As already indicated in an earlier mail I would prefer to treat 
building up JIGSAW tests as a follow-on. I have not gained much 
experience with JIGSAW yet, but I would be willing to support this 
as far as I can. The problem I currently see is that SWT is still 
an automatic module, and JIGSAW-based SWT tests would have to 
access code from the swt-debug.jar, which is as well an automatic 
module. AFAIK, we would have to turn SWT into a named module first 
in order to make swt-debug.jar available on its module path. That 
seems to be out of scope here and should probably be investigated 
in an own issue.



2. Platform logic:

+ if(IS_MAC){
+ enabled = false
...
+ }
+ if(IS_LINUX){

Normally we would handle this in the test itself by using 
assumeTrue and platform checks. In this case, though, since it 
applies to all SWT tests run on Mac (or Linux in case of the 
warning), this seems fine.


Please fix the spacing to match our coding conventions, though. 
There should be a space after the 'if' and a space before the '{'. 
So:


if (IS_MAC) {


Ok. fine. We could even try to get SWT tests working on Mac by 
falling back to ant-based test execution here, but I would propose 
to handle this in a different issue as well (after all, this issue 
is about SWT image cursors and not about building up an SWT test 
harness).



modules/swt/src/main/java/javafx/embed/swt/SWTCursors.java

3. Spacing issues:

+ if(display == null){

Please add a space after '(' and before '}'


-}
+}
\ No newline at end of file

Please restore the newline after the last line.


Done.


SWTCursorsTest.java

4. Need a blank line before the package declaration:

+ */
+package test.javafx.embed.swt;
+


Done.


5. Can you sort the imports alphabetically?


I organized the imports and removed unused ones. It seems they are 
pretty much consistent with those of the other SWT classes.



6. Since the test loops waiting on a latch, please add a 10 second 
timeout for the test:


@Test(timeout=1)
public void testImageCursor() throws Throwable {


Done.


SWTImageCursorTest.java

7. The following can use a lambda (as the setOnMouseEntered 
already did).


+ rect.setOnMouseExited(new EventHandler() {
+ @Override
+ public void handle(MouseEvent event) {
+ scene.setCursor(null);
+ }
+ });


Done.

Kevin, thanks for picking this up.

Regards,
Alexander

=



Am 27.07.2016 um 01:15 schrieb Kevin Rushforth 
mailto:kevin.rushfo...@oracle.com>>:


Picking this back up again, I have just added my comments to JBS 
issue.


https://bugs.openjdk.java.net/browse/JDK-8088147

The comments are minor in scope, so I suspect it will be ready for 
final review after one more iteration. It will need one other 
reviewer, since I am sponsoring the change.


Could someone else review this as well (maybe Alexander Z or Sergey)?

-- Kevin


Kevin Rushforth wrote:
I'm back, and given that the review will take some time anyway, I 
will sponsor this once the review is complete. I see t

Re: CFV: New OpenJFX Committer: Ankit Srivastava

[Alexander Zvegintsev]

Vote: yes


On 7/19/16 4:37 PM, David Hill wrote:


I hereby nominate Ankit Srivastava to OpenJFX Committer.

Ankit Srivastava is part of the JavaFX team focusing on Web.

A list of Ankit's commits and reviews is available by the following links

http://hg.openjdk.java.net/openjfx/8u-dev/rt/log?rev=a.ankit.srivast...@oracle.com 


http://hg.openjdk.java.net/openjfx/8u-dev/rt/log?rev=asrivastava

Votes are due by August 3th, 2016.

Only current OpenJFX Committers [1] are eligible to vote on this 
nomination. Votes must be cast in the open by replying to this mailing 
list.


For Lazy Consensus voting instructions, see [2]. Nomination to a 
project Committer is described in [3].


[1] http://openjdk.java.net/census#openjfx

[2] http://openjdk.java.net/bylaws#lazy-consensus

[3] http://openjdk.java.net/projects#project-committer

Thanks,

Dave


--
Thanks,
Alexander.

[9] Review request 8160241: Maximizing an Window with Screen-Size hides it

[Alexander Zvegintsev]

Hi,

please review the fix

http://cr.openjdk.java.net/~azvegint/fx/9/8160241/00/

for the issue

https://bugs.openjdk.java.net/browse/JDK-8160241

Re: CFV: New OpenJFX Committer: Arunprasad Rajkumar

[Alexander Zvegintsev]

vote: yes

Thanks,
Alexander.

On 21/04/16 19:48, Kevin Rushforth wrote:

I hereby nominate Arunprasad Rajkumar [1] to OpenJFX Committer.

Arunprasad is a member of JavaFX team at Oracle working on WebKit, who 
has contributed 10 changesets [5] to OpenJFX, at least 8 of which are 
significant.


Votes are due by May 5, 2016.

Only current OpenJFX Committers [2] are eligible to vote on this 
nomination. Votes must be cast in the open by replying to this mailing 
list.


For Lazy Consensus voting instructions, see [3]. Nomination to a 
project Committer is described in [4].


Thanks,

-- Kevin

[1] http://openjdk.java.net/census#arajkumar

[2] http://openjdk.java.net/census#openjfx

[3] http://openjdk.java.net/bylaws#lazy-consensus

[4] http://openjdk.java.net/projects#project-committer

[5] List of changesets:
http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/66eee7a12e81
http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/3f15f2e59063
http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/b75591ee263b
http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/84e09aadadf1
http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/4166cbca577f
http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/de87459ed168
http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/bccedad1f44b
http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/71a5499bcda4
http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/251198e174cd
http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/67a83054f43b

[9] Review request: 8088916: Replace sun.net.www.ParseUtil#decode with public API

[Alexander Zvegintsev]

Hi Kevin,

please review  the fix
http://cr.openjdk.java.net/~azvegint/jdk/9/8088916/00/
for the issue
https://bugs.openjdk.java.net/browse/JDK-8088916

--
--
Thanks,
Alexander.

Re: CFV: New OpenJFX Committer: Murali Billa

[Alexander Zvegintsev]

Vote: yes

--
Thanks,
Alexander.

On 01.04.2016 0:04, Kevin Rushforth wrote:

I hereby nominate Murali Billa [1] to OpenJFX Committer.

Murali is a member of JavaFX team at Oracle working on WebKit, who has 
contributed 10 changesets [5] to OpenJFX, at least 8 of which are 
significant.


Votes are due by April 14, 2016.

Only current OpenJFX Committers [2] are eligible to vote on this 
nomination. Votes must be cast in the open by replying to this mailing 
list.


For Lazy Consensus voting instructions, see [3]. Nomination to a 
project Committer is described in [4].


Thanks,

-- Kevin

[1] http://openjdk.java.net/census#mbilla

[2] http://openjdk.java.net/census#openjfx

[3] http://openjdk.java.net/bylaws#lazy-consensus

[4] http://openjdk.java.net/projects#project-committer

[5] List of changesets:
http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/a251a1d65932
http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/ecea43f5734c
http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/42b461505f27
http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/82ecaebd44cf
http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/8643ca988cef
http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/765fd07f22fc
http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/ae75f92d5e53
http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/25db4b2e47a1
http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/51c2129d282c
http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/cb8a24f5db2a

Re: CFV: New OpenJFX Committer: Alexander Matveev

[Alexander Zvegintsev]

Vote: yes

Thanks,

Alexander.

On 08/28/2015 04:55 AM, Kevin Rushforth wrote:

I hereby nominate Alexander Matveev [1] to OpenJFX Committer.

Alexander was an initial member of JavaFX team at Oracle when the 
OpenJFX project was created, and was on the initial list of approved 
committers [2]. His status as OpenJFX committer was not recorded at 
that time on the Census. This CVF is intended to correct this oversight.


Alexander's changes prior to JavaFX becoming open-source were 
significant (which is why he was on the initial list of committers). 
In particular, Alexander, along with the rest of the media team, 
contributed much of the code that went into the open-source changeset 
that Kirill Kirichenko pushed for OpenJFX in the JDK 8 release:


   hg log -r aee256fde55

Counting his contribution to that changeset, Alexander now also has 
the requisite number of changes in the open-source repo to become a 
committer.


   hg log -M -u "matvee"

Votes are due by September 10, 2015.

Only current OpenJFX Committers [3] are eligible to vote on this 
nomination. Votes must be cast in the open by replying to this mailing 
list.


For Lazy Consensus voting instructions, see [4]. Nomination to a 
project Committer is described in [5].


Thanks,

-- Kevin

[1] http://openjdk.java.net/census#almatvee

[2] 
http://mail.openjdk.java.net/pipermail/announce/2011-November/000113.html


[3] http://openjdk.java.net/census#openjfx

[4] http://openjdk.java.net/bylaws#lazy-consensus

[5] http://openjdk.java.net/projects#project-committer

Re: CFV: New OpenJFX Committer: Anton Nashatyrev

[Alexander Zvegintsev]

Vote: yes

--
Thanks,
Alexander.

On 05/29/2015 04:16 PM, Kevin Rushforth wrote:

I hereby nominate Anton Nashatyrev to OpenJFX Committer.

Anton is a member of JavaFX team at Oracle working on WebKit and Media.

A list of Anton's contributions is available via the following link:

http://hg.openjdk.java.net/openjfx/8u-dev/rt/log?rev=nashat

Votes are due by June 12, 2015.

Only current OpenJFX Committers [1] are eligible to vote on this 
nomination. Votes must be cast in the open by replying to this mailing 
list.


For Lazy Consensus voting instructions, see [2]. Nomination to a 
project Committer is described in [3].


[1] http://openjdk.java.net/census#openjfx
[2] http://openjdk.java.net/bylaws#lazy-consensus
[3] http://openjdk.java.net/projects#project-committer

Thank you.

-- Kevin

Re: Errors while building openjfx from sources

[Alexander Zvegintsev]

Hello Alexander,

It looks like that you have GCC version < 4.6 and it lacks diagnostics 
push/pop functionality[1]
that we are using to suppress deprecation warnings. We have moved to gcc 
4.8 on Linux recently[2].


So the fix is to upgrade GCC to at least 4.6. Or as a workaround you may 
apply following patch:


diff -r 8b7ea03165e9 buildSrc/linux.gradle
--- a/buildSrc/linux.gradle
+++ b/buildSrc/linux.gradle
@@ -138,7 +138,7 @@ LINUX.glass.javahInclude = [
 "com/sun/glass/ui/gtk/*"]
 LINUX.glass.nativeSource = 
file("modules/graphics/src/main/native-glass/gtk")

 LINUX.glass.compiler = compiler
-LINUX.glass.ccFlags = [ccFlags, "-Werror"].flatten()
+LINUX.glass.ccFlags = ccFlags
 LINUX.glass.linker = linker
 LINUX.glass.linkFlags = [linkFlags].flatten()
 LINUX.glass.lib = "glass"

It stops treating warning as errors[3].

[1] https://gcc.gnu.org/onlinedocs/gcc/Diagnostic-Pragmas.html
[2] 
http://mail.openjdk.java.net/pipermail/openjfx-dev/2015-March/016911.html

[3] https://gcc.gnu.org/onlinedocs/gcc/Warning-Options.html

Thanks,

Alexander.

On 04/21/2015 11:48 AM, Александр Свиридов wrote:

  I am trying to build openjfx from sources - centos 6.4. 64 openjdk8 (build 
25.60-b11, mixed mode), I run

/gradle-1.8/bin/gradle --info --debug

and this is what I get:
10:11:09.184 [ERROR] [system.err] cc1plus: warnings being treated as errors
10:11:09.186 [ERROR] [system.err] 
/home/PE/Temp/rt/modules/graphics/src/main/native-glass/gtk/glass_general.cpp:114:
 error: expected [error|warning|ignored] after '#pragma GCC diagnostic'
10:11:09.187 [ERROR] [system.err] 
/home/PE/Temp/rt/modules/graphics/src/main/native-glass/gtk/glass_general.cpp:132:
 error: expected [error|warning|ignored] after '#pragma GCC diagnostic'
10:11:09.187 [DEBUG] [org.gradle.process.internal.DefaultExecHandle] Changing 
state to: FAILED
10:11:09.188 [INFO] [org.gradle.process.internal.DefaultExecHandle] Process 
'command 'gcc'' finished with exit value 1 (state: FAILED)
10:11:09.195 [DEBUG] [org.gradle.logging.internal.DefaultLoggingConfigurer] 
Finished configuring with level: DEBUG, configurers: 
[org.gradle.logging.internal.OutputEventRenderer@675d3402, 
org.gradle.logging.internal.logback.LogbackLoggingConfigurer@51565ec2, 
org.gradle.logging.internal.JavaUtilLoggingConfigurer@482f8f11]
10:11:09.195 [DEBUG] 
[org.gradle.api.internal.tasks.execution.ExecuteAtMostOnceTaskExecuter] 
Finished executing task ':graphics:ccLinuxGlass'
10:11:09.195 [LIFECYCLE] [org.gradle.TaskExecutionLogger] 
:graphics:ccLinuxGlass FAILED
10:11:09.196 [INFO] [org.gradle.execution.taskgraph.AbstractTaskPlanExecutor] 
:graphics:ccLinuxGlass (Thread[main,5,main]) completed. Took 1.1 secs.
10:11:09.196 [DEBUG] [org.gradle.execution.taskgraph.AbstractTaskPlanExecutor] 
Task worker [Thread[main,5,main]] finished, busy: 5.129 secs, idle: 0.044 secs
10:11:09.212 [ERROR] [org.gradle.BuildExceptionReporter]
10:11:09.216 [ERROR] [org.gradle.BuildExceptionReporter] FAILURE: Build failed 
with an exception.
10:11:09.217 [ERROR] [org.gradle.BuildExceptionReporter]
10:11:09.220 [ERROR] [org.gradle.BuildExceptionReporter] * What went wrong:
10:11:09.221 [ERROR] [org.gradle.BuildExceptionReporter] Execution failed for 
task ':graphics:ccLinuxGlass'.
10:11:09.222 [ERROR] [org.gradle.BuildExceptionReporter] > Could not call 
NativeCompileTask.compile() on task ':graphics:ccLinuxGlass'
10:11:09.225 [ERROR] [org.gradle.BuildExceptionReporter]
10:11:09.226 [ERROR] [org.gradle.BuildExceptionReporter] * Exception is:
10:11:09.231 [ERROR] [org.gradle.BuildExceptionReporter] 
org.gradle.api.tasks.TaskExecutionException: Execution failed for task 
':graphics:ccLinuxGlass'.
10:11:09.232 [ERROR] [org.gradle.BuildExceptionReporter] at 
org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter.executeActions(ExecuteActionsTaskExecuter.java:69)

how to solve it?

[8u60, 9] Review request for RT-40548: [GTK] Gdk-ERROR **: g_thread_init() must be called before gdk_threads_init()

[Alexander Zvegintsev]

Hello,

please review the fix
http://cr.openjdk.java.net/~azvegint/fx/8u60/40548/00/
for the issue
https://javafx-jira.kenai.com/browse/RT-40548

--
Thanks,

Alexander.

[8u40] Review request for RT-35310: [Stage, Linux] In HelloSanity, the 'Back from minimize' button does not restore the stage

[Alexander Zvegintsev]

Hello David,

please review the fix
http://cr.openjdk.java.net/~azvegint/fx/8u40/35310/00/
for the issue
https://javafx-jira.kenai.com/browse/RT-35310

--
Thanks,

Alexander.

[8u40] Review request RT-39233: Glass/GTK should support removing Window.MINIMIZABLE, Window.MAXIMIZABLE

[Alexander Zvegintsev]

Hello David,

please review the fix
http://cr.openjdk.java.net/~azvegint/fx/8u40/39233/00/
for the issue
https://javafx-jira.kenai.com/browse/RT-39233

--
Thanks,

Alexander.

[8u40] Review request for RT-38940: [Gtk, Mac] Japanese IME, unable to distinguish focused composing text

[Alexander Zvegintsev]

Hello,

please review the fix
http://cr.openjdk.java.net/~azvegint/fx/8u40/38940/
for
https://javafx-jira.kenai.com/browse/RT-38940

--
--
Thanks,
Alexander.

Re: CFV: New OpenJFX Committer: Morris Meyer

[Alexander Zvegintsev]

Vote: yes

Thanks,

Alexander.

On 09/24/2014 05:17 PM, Kevin Rushforth wrote:

I hereby nominate Morris Meyer to OpenJFX Committer.

Morris was an initial member of JavaFX team at Oracle when the OpenJFX 
project was created, and was on the initial list of approved 
committers [1]. His status as OpenJFX committer was not recorded at 
that time on the Census as he moved to another project within the Java 
organization. This CVF is intended to correct this oversight.


Morris' changes prior to JavaFX becoming open-source were significant 
(which is why he was on the initial list of committers). He also has 
the requisite number of changes in the open-source repo to become a 
committer.


 hg log -M -u "morris"

A list of Morris' commits is also available by the following links:

http://hg.openjdk.java.net/openjfx/8u-dev/rt/log?rev=morris

Votes are due by October 8, 2014.

Only current OpenJFX Committers [2] are eligible to vote on this 
nomination. Votes must be cast in the open by replying to this mailing 
list.


For Lazy Consensus voting instructions, see [3]. Nomination to a 
project Committer is described in [4].


[1] 
http://mail.openjdk.java.net/pipermail/announce/2011-November/000113.html


[2] http://openjdk.java.net/census#openjfx

[3] http://openjdk.java.net/bylaws#lazy-consensus

[4] http://openjdk.java.net/projects#project-committer

Thanks,

-- Kevin

Re: CFV: New OpenJFX Committer: Michael Fang

[Alexander Zvegintsev]

Vote: yes

Thanks,

Alexander.

On 09/24/2014 04:41 AM, Kevin Rushforth wrote:

I hereby nominate Michael Fang to OpenJFX Committer.

Michael is a member of Java Localization team at Oracle. Michael's 
OpenJFX changes are in the localized property files.


 hg log -M -u "mfang"

A list of Michael's commits is also available by the following links 
(be sure to click the "more" link to see more than 10):


http://hg.openjdk.java.net/openjfx/8u-dev/rt/log?rev=mfang

Votes are due by October 7, 2014.

Only current OpenJFX Committers [1] are eligible to vote on this 
nomination. Votes must be cast in the open by replying to this mailing 
list.


For Lazy Consensus voting instructions, see [2]. Nomination to a 
project Committer is described in [3].


[1] http://openjdk.java.net/census#openjfx

[2] http://openjdk.java.net/bylaws#lazy-consensus

[3] http://openjdk.java.net/projects#project-committer

Thanks,

-- Kevin

Re: CFV: New OpenJFX Committer: Elina Kleyman

[Alexander Zvegintsev]

Vote:  YES

--
Thanks,
Alexander.

On 04/10/2014 04:31 PM, David Hill wrote:


I hereby nominate Elina Kleyman to OpenJFX Committer.

Elina is a member of JavaFX Embedded team at Oracle. Elina's changes 
are in Glass support code, as well as our test and sample applications.


  hg log -M -u "Elina Kleyman"

A list of Elina's commits is also available by the following links (be 
sure to click the "more" link to see more than 10):


http://hg.openjdk.java.net/openjfx/8u-dev/rt/log?rev=elina.kley...@oracle.com 



Votes are due by April 25, 2013.

Only current OpenJFX Committers [1] are eligible to vote on this 
nomination. Votes must be cast in the open by replying to this mailing 
list.


For Lazy Consensus voting instructions, see [2]. Nomination to a 
project Committer is described in [3].


[1] http://openjdk.java.net/census#openjfx

[2] http://openjdk.java.net/bylaws#lazy-consensus

[3] http://openjdk.java.net/projects#project-committer

Thanks,

Dave

[8u20] Review request: RT-35976 [Linux, GTK] JavaFX X11 WM_CLASS

[Alexander Zvegintsev]

Hi Antony,

Please review this simple fix.

Jira: https://javafx-jira.kenai.com/browse/RT-35976
Webrev: http://cr.openjdk.java.net/~azvegint/fx/8u20/RT-35976/

--
--
Thanks,
Alexander.

Re: CFV: New OpenJFX Committer: Victor Shubov

[Alexander Zvegintsev]

Vote: YES

--
Thanks,
Alexander.

24.10.2013 17:05, Artem Ananiev wrote:


I hereby nominate Victor Shubov to OpenJFX Committer.

Victor is a member of JavaFX SQE team at Oracle. He has already 
contributed enough changesets into the "tests" repository:


$ hg log -M -u "Victor Shubov" --template '{author}\n' |wc -l
29

Votes are due by Nov 07, 2013.

Only current OpenJFX Committers [1] are eligible to vote on this 
nomination. Votes must be cast in the open by replying to this mailing 
list.


For Lazy Consensus voting instructions, see [2]. Nomination to a 
project Committer is described in [3].


[1] http://openjdk.java.net/census#openjfx

[2] http://openjdk.java.net/bylaws#lazy-consensus

[3] http://openjdk.java.net/projects#project-committer

Thanks,

Artem

Re: Fwd: CFV: New OpenJFX Committer: Vadim Pakhnushev

[Alexander Zvegintsev]

Vote: YES.

Thanks,

Alexander.

On 12/12/2013 01:17 AM, David Hill wrote:


I hereby nominate Vadim Pakhnushev to OpenJFX Committer.

Vadim is a member of JavaFX Embedded team at Oracle. Vadim's changes 
are in Glass Windows/D3d:


  hg log -M -u "vadim"

An incomplete list of Vadim's commits and reviews is also available by 
the following link:


http://hg.openjdk.java.net/openjfx/8/master/rt/log?rev=vadim

Votes are due by Dec 25, 2013.

Only current OpenJFX Committers [1] are eligible to vote on this 
nomination. Votes must be cast in the open by replying to this mailing 
list.


For Lazy Consensus voting instructions, see [2]. Nomination to a 
project Committer is described in [3].


[1] http://openjdk.java.net/census#openjfx

[2] http://openjdk.java.net/bylaws#lazy-consensus

[3] http://openjdk.java.net/projects#project-committer

Thanks,

Dave

[8] Review request for RT-34750 [TextField, PasswordField, TextArea, Linux] selected text is removed on focus leaving

[Alexander Zvegintsev]

Hello Anthony,

Please review fix for https://javafx-jira.kenai.com/browse/RT-34750

--
Thanks,

Alexander.

[8] Review request for RT-34196 [Controls, Menus] Gtk: JAVAFX default menu items are difficult to click

[Alexander Zvegintsev]

Hello Anthony, Martin,

Please review fix for https://javafx-jira.kenai.com/browse/RT-34196

--
Thanks,

Alexander.

[8] Review request for RT-30457 Gtk: setDragView doesn't work on linux

[Alexander Zvegintsev]

Hello Anthony,

Please review fix for https://javafx-jira.kenai.com/browse/RT-30457
http://cr.openjdk.java.net/~azvegint/fx/8/30457/webrev.00/

--
Thanks,

Alexander.

[8] Review request for RT-34272 Gtk: Linux Input Method, partial composed text is not canceled when losing focus.

[Alexander Zvegintsev]

Hello,

Please review fix for https://javafx-jira.kenai.com/browse/RT-34272
http://cr.openjdk.java.net/~azvegint/fx/8/34272/webrev.00/

--
Thanks,

Alexander.

[8] Review request for RT-32477 Gtk: TextField: Ctrl+A don't work with Russian keyboard layout.

[Alexander Zvegintsev]

Hello,

Please review fix for https://javafx-jira.kenai.com/browse/RT-32477
http://cr.openjdk.java.net/~azvegint/fx/8/32477/webrev.00/

--
Thanks,

Alexander.

Heads-up: changing minimum build number to b115

[Alexander Zvegintsev]

Hi all,

We need to change the minimum JDK 8 version number required for building 
FX to b115 (is currently b113) to fix a problem with GTK initialization 
for Linux distros with GLib version < 2.24.



FX JIRA:  https://javafx-jira.kenai.com/browse/RT-32436

I will push this change shortly.

--
Thanks,

Alexander.

[8] Review request for RT-32436 Gtk: OpenGL with JavaFX causes Gthread error and halts

[Alexander Zvegintsev]

Hello,

Please review fix for https://javafx-jira.kenai.com/browse/RT-32436

--
Thanks,

Alexander.