Well, Vincent seemed to be happy with the new setting for kerning tuning.

I would say, that my issue is not a regression and therefore should be not a blocker.

Felix

Am 16.03.22 um 16:22 schrieb Philippe Mouawad:
Hello Felix,
Unless somebody has identified a specific issue in this version, it's not
clear for me if Vincent's report is specific to this version.

Regards

On Wed, Mar 16, 2022 at 4:15 PM Felix Schumacher <
felix.schumac...@internetallee.de> wrote:

Am 15.03.22 um 22:42 schrieb Philippe Mouawad:

Hello,
I have same behaviour on Java 8 and Java 11 on MacOSX, GUI freezes.

Setting below property does not improve behaviour:

    - text.kerning.max_document_size=1


Note that I observe similar freeze with 5.4.1

That is right, so it is not a regression per se.

Does that mean, we should move on with the release?

Felix


Regards



On Tue, Mar 15, 2022 at 12:58 PM Felix Schumacher <
felix.schumac...@internetallee.de> wrote:

I tried the attached file.

It samples heise.de/newsticker which will send a redirect and the
result, which gets stored in the View Results Tree. When I open the sub
results and switch between the redirect and the content sample, my GUI gets
stuck.

On linux it spends a long time in

AWT-EventQueue-0" #18 prio=6 os_prio=0 tid=0x00007ff961140800
nid=0x25a240 runnable [0x00007ff8c3b98000]
    java.lang.Thread.State: RUNNABLE
     at javax.swing.text.GapContent.getChars(GapContent.java:213)
     at
javax.swing.text.AbstractDocument.getText(AbstractDocument.java:810)
     at javax.swing.text.GlyphView.getText(GlyphView.java:135)
     at javax.swing.text.GlyphView.getBreakSpot(GlyphView.java:791)
     at javax.swing.text.GlyphView.getMinimumSpan(GlyphView.java:551)
     at
javax.swing.text.ParagraphView.calculateMinorAxisRequirements(ParagraphView.java:724)

     at
javax.swing.JEditorPane$PlainEditorKit$PlainParagraph.calculateMinorAxisRequirements(JEditorPane.java:2163)

     at javax.swing.text.BoxView.checkRequests(BoxView.java:935)
     at javax.swing.text.BoxView.getMinimumSpan(BoxView.java:568)
     at
javax.swing.text.BoxView.calculateMinorAxisRequirements(BoxView.java:903)
     at javax.swing.text.BoxView.checkRequests(BoxView.java:935)
     at javax.swing.text.BoxView.setSpanOnAxis(BoxView.java:343)
     at javax.swing.text.BoxView.layout(BoxView.java:708)
     at javax.swing.text.BoxView.setSize(BoxView.java:397)
     at
javax.swing.plaf.basic.BasicTextUI$RootView.setSize(BasicTextUI.java:1722)
     at
javax.swing.plaf.basic.BasicTextUI.getPreferredSize(BasicTextUI.java:912)
     at
com.github.weisj.darklaf.ui.text.DarkTextUI.getPreferredSize(DarkTextUI.java:223)

     at javax.swing.JComponent.getPreferredSize(JComponent.java:1662)
     at javax.swing.JEditorPane.getPreferredSize(JEditorPane.java:1333)
     at javax.swing.JViewport.getViewSize(JViewport.java:999)
     at
javax.swing.plaf.basic.BasicScrollPaneUI.syncScrollPaneWithViewport(BasicScrollPaneUI.java:278)

     at
javax.swing.plaf.basic.BasicScrollPaneUI$Handler.stateChanged(BasicScrollPaneUI.java:1034)

     at javax.swing.JViewport.fireStateChanged(JViewport.java:1369)
     at javax.swing.JViewport.setView(JViewport.java:969)
     at javax.swing.JScrollPane.setViewportView(JScrollPane.java:1007)
     at
org.apache.jmeter.visualizers.RenderAsText.showTextResponse(RenderAsText.java:36)


The result is about 700 KByte, so should trigger the kerning mechanism
anyway.

Felix

PS. My linux computer is currently maxed out with running a test suite
and the lagging can be observed with linux, too.
Am 15.03.22 um 12:48 schrieb Vladimir Sitnikov:

Could anyone confirm, that switching between 0 and biggish (200-600 KByte)

samples will inflict an unresponsive GUI?)

Felix would you please share the test script?

As a workaround, we might disable kerning for Java <17 by default.
It looks like we might need another RC for that.

Vladimir



--
Cordialement.
Philippe Mouawad.



Attachment: OpenPGP_0xEA6C3728EA91C4AF.asc
Description: OpenPGP public key

Attachment: OpenPGP_signature
Description: OpenPGP digital signature

Reply via email to