FLTK logo

Re: [fltk/fltk] test/tree: hitting 'select all' hangs the application (Issue #300)

FLTK matrix user chat room
(using Element browser app)   FLTK gitter user chat room   GitHub FLTK Project   FLTK News RSS Feed  
  FLTK Apps      FLTK Library      Forums      Links     Login 
 All Forums  |  Back to fltk.issues  ]
 
Previous Message ]New Message | Reply ]Next Message ]

Re: [fltk/fltk] test/tree: hitting 'select all' hangs the application (Issue #300) ManoloFLTK Nov 29, 2021  
 

OK. The culprit is eb772d0.

Luckily, this is a one-liner : add
oldTAWidth - 1;
after line 494 (as of eb772d0) in function Fl_Text_Display::resize().

This is visible as follows :
git reset -- hard eb772d0,
make a copy of src/Fl_Text_Display.cxx and FL/Fl_Text_Display.H
git pull -ff
copy the two saved files to src/ and FL/, respectively.
Build and run test/tree: 'SelectAll' and 'Deselect All' are slow and become slower and slower
each time they are used because the Simple Terminal contains more and more text.

Comment out the single line #495 in src/Fl_Text_Display.cxx to get
// oldTAWidth - 1;
This undoes eb772d0.
Build and run test/tree: 'SelectAll' and 'Deselect all' run quickly, and their run-time doesn't increase.

Greg: I propose to stop here and let you continue the analysis of this issue, if you agree.


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

Direct Link to Message ]
 
     
Previous Message ]New Message | Reply ]Next Message ]
 
 

Comments are owned by the poster. All other content is copyright 1998-2024 by Bill Spitzak and others. This project is hosted by The FLTK Team. Please report site problems to 'erco@seriss.com'.