Re: Possible Bug in Java 8 u 112 in javax.swing.JEditorPane.setText leads to stuck UI

2017-01-14 Thread Philippe Mouawad
Hi,
Previous sample showed already very slow rendering when text contains
spaces.
Now for the text without space. Sample attached.

Regards

On Fri, Jan 13, 2017 at 2:20 PM, Philippe Mouawad 
wrote:

> Hello Muneer,
> Find attached  a simple program reproducing issue.
>
> I see you closed the bug
> Regards
>
> On Fri, Jan 6, 2017 at 2:28 PM, Muneer Kolarkunnu <
> abdul.kolarku...@oracle.com> wrote:
>
>> Hi Philippe,
>>
>> Your incident has moved to JDK-8172336: https://bugs.openjdk.java.net/
>> browse/JDK-8172336
>>
>> I tried to reproduce the issue, but I could not reproduce this issue with
>> the information shared in the bug report. If you can provide a standalone
>> test case, it will be great. Also, please let us know if you observe the
>> same issue with JDK 8u122-ea and JDK 9-ea.
>> Have you observed the same issue with other OS(Other than Mac OSX) ?
>>
>> 8u122-ea is available here : https://jdk8.java.net/download.html
>> JDK 9-ea is available here : https://jdk9.java.net/download/
>>
>> Regards,
>> Muneer
>>
>> -Original Message-
>> From: Rory O'Donnell
>> Sent: Thursday, January 05, 2017 5:22 PM
>> To: dev@jmeter.apache.org
>> Cc: Rory O'Donnell; Dalibor Topic; Balchandra Vaidya; Muneer Kolarkunnu
>> Subject: Re: Possible Bug in Java 8 u 112 in
>> javax.swing.JEditorPane.setText leads to stuck UI
>>
>> Thanks Philippe, we'll take a look.
>>
>> Rgds,Rory
>>
>>
>> On 05/01/2017 10:30, Philippe Mouawad wrote:
>> > Hello,
>> > Done:9046713
>> >
>> > Regards
>> >
>> > On Thu, Jan 5, 2017 at 11:14 AM, Rory O'Donnell
>> > 
>> > wrote:
>> >
>> >> Hi Philippe,
>> >>
>> >> Many happy returns!
>> >>
>> >> Can you log a bug and send us the Java Incident id ?
>> >>
>> >> Rgds,Rory
>> >>
>> >>
>> >>
>> >>
>> >> On 05/01/2017 10:12, Philippe Mouawad wrote:
>> >>
>> >>> Greetings,
>> >>> First best wishes for 2017.
>> >>>
>> >>> I'd like to report what seems to be a critical bug we face in JMeter
>> >>> . I noticed it under Mac OSX El Capitan.
>> >>>
>> >>> Calling javax.swing.JEditorPane.setText() from AWT Thread with some
>> >>> long text (without spaces) leads to what seems to be either a very
>> >>> long or infinite loop, I made thread dumps and I have always  such
>> >>> (partial)
>> >>> stacktrace:
>> >>> "AWT-EventQueue-0" #20 prio=6 os_prio=31 tid=0x7fa7a8afc000
>> >>> nid=0xf707 runnable [0x72202000]
>> >>>  java.lang.Thread.State: RUNNABLE
>> >>>   at sun.font.CStrike.getNativeGlyphOutline(Native Method)
>> >>>   at sun.font.CStrike.getGlyphOutline(CStrike.java:215)
>> >>>   at sun.font.CStrike.getGlyphOutlineBounds(CStrike.java:177)
>> >>>   at
>> >>> sun.font.StandardGlyphVector$GlyphStrike.getGlyphOutlineBoun
>> >>> ds(StandardGlyphVector.java:1792)
>> >>>   at
>> >>> sun.font.StandardGlyphVector.getGlyphOutlineBounds(StandardG
>> >>> lyphVector.java:1174)
>> >>>   at
>> >>> sun.font.StandardGlyphVector.getGlyphVisualBounds(StandardGl
>> >>> yphVector.java:586)
>> >>>   at
>> >>> sun.font.StandardGlyphVector.getGlyphInfo(StandardGlyphVecto
>> r.java:864)
>> >>>   at
>> >>> sun.font.ExtendedTextSourceLabel.createCharinfo(ExtendedText
>> >>> SourceLabel.java:622)
>> >>>   at
>> >>> sun.font.ExtendedTextSourceLabel.getCharinfo(ExtendedTextSou
>> >>> rceLabel.java:548)
>> >>>   at
>> >>> sun.font.ExtendedTextSourceLabel.getLineBreakIndex(ExtendedT
>> >>> extSourceLabel.java:480)
>> >>>   at java.awt.font.TextMeasurer.calcLineBreak(TextMeasurer.java:
>> 330)
>> >>>   at java.awt.font.TextMeasurer.getLineBreakIndex(TextMeasurer.
>> >>> java:566)
>> >>>   at
>> >>> java.awt.font.LineBreakMeasurer.nextOffset(LineBreakMeasurer
>> .java:359)
>> >>>   at
>> >>> java.awt.font.LineBreakMeasurer.nextLayout(LineBreakMeasurer
>> .java:440)
>> >>>   at javax.swing.text.TextLayoutStrategy.sync(TextLayoutStrategy.
>> >>> java:324)
>> >>>   at
>> >>> javax.swing.text.TextLayoutStrategy.insertUpdate(TextLayoutS
>> >>> trategy.java:70)
>> >>>   at javax.swing.text.FlowView.insertUpdate(FlowView.java:256)
>> >>>   at javax.swing.text.View.forwardUpdateToView(View.java:1227)
>> >>>   at javax.swing.text.View.forwardUpdate(View.java:1162)
>> >>>   at javax.swing.text.BoxView.forwardUpdate(BoxView.java:240)
>> >>>   at javax.swing.text.View.insertUpdate(View.java:710)
>> >>>   at
>> >>> javax.swing.plaf.basic.BasicTextUI$RootView.insertUpdate(
>> >>> BasicTextUI.java:1610)
>> >>>   at
>> >>> javax.swing.plaf.basic.BasicTextUI$UpdateHandler.insertUpdat
>> >>> e(BasicTextUI.java:1869)
>> >>>   at
>> >>> javax.swing.text.AbstractDocument.fireInsertUpdate(AbstractD
>> >>> ocument.java:201)
>> >>>   at
>> >>> javax.swing.text.AbstractDocument.handleInsertString(Abstrac
>> >>> tDocument.java:748)
>> >>>   at
>> >>> javax.swing.text.AbstractDocument.insertString(AbstractDocum
>> ent.java:707)
>> >>>   at 

[GitHub] jmeter pull request #226: Bug 58274 - Removed split methods from JOrphanUtil...

2017-01-14 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/jmeter/pull/226


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] jmeter issue #240: Improves JMS component robustness

2017-01-14 Thread pmouawad
Github user pmouawad commented on the issue:

https://github.com/apache/jmeter/pull/240
  
Hello,
Thanks for PR.
I have commited it but with some slight changes.
Could you test nightly build and give us feedback ?
Also could you attach the JMX plan that you attached to this PR as a zip 
(which I am not able to read) to the bugzilla 60585

Thank you


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] jmeter pull request #240: Improves JMS component robustness

2017-01-14 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/jmeter/pull/240


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] jmeter pull request #245: Allow table sorting

2017-01-14 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/jmeter/pull/245


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] jmeter issue #250: Feature/bz60564 (1)

2017-01-14 Thread pmouawad
Github user pmouawad commented on the issue:

https://github.com/apache/jmeter/pull/250
  
Thanks for PR.
It looks ok.
Before we start integrating it, would it be possible to propose a PR for 
the configuration part ?
I think it would be interesting to see how configuration will be done from 
users point of view in JMeter before going further.

Thank you


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] jmeter issue #245: Allow table sorting

2017-01-14 Thread pmouawad
Github user pmouawad commented on the issue:

https://github.com/apache/jmeter/pull/245
  
Hello,
I have commited part of the PR and made some changes:
- I didn't want to use everywhere HeaderAsPropertyRenderer.install as it 
removes formatting without introducing sorting
- It would have also broken backward compatibility on the class potentially 
breaking backward compatibility of plugins
- So I have replaced it by a HeaderAsPropertyRendererWrapper class
- I have added sorting to Aggregate Graph
- In ObjectTableModel, I have also replaced the modified method by a new 
one getObjectListAsList to avoid breaking backward compatibility

Thanks for your PR, please test nightly build to see if it works fine for 
you.
Thanks


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---