bugGNU TeXmacs - Bugs: bug #44106, Endless loop in...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #44106: Endless loop in edit_cursor_rep::make_cursor_accessible

Submitted by:  Francois Poulain <fpoulain>
Submitted on:  Wed 28 Jan 2015 04:25:26 PM UTC  
 
Category: EditorPriority: 9 - Immediate
Item Group: ErrorStatus: Fixed
Privacy: PublicAssigned to: Joris van der Hoeven <vdhoeven>
Originator Name: Open/Closed: Closed
Release: Release: None
Fixed Release: NoneFixed Release: 
Keywords: 

Sun 19 Mar 2017 09:34:46 AM UTC, comment #2:

With the current svn version (~10600) and using François' file, I cannot reproduce the problem he describes: It seems it was fixed.
=> closing

Philippe Joyez <pjoyez>
Project Member
Sat 07 Mar 2015 10:04:08 PM UTC, comment #1:

Confirmed. I've been using the following ugly hack for a while now to work around the problem and be able to do my presentations.

Miguel de Benito <mdbenito>
Project Member
Wed 28 Jan 2015 04:25:26 PM UTC, original submission:

The function make_cursor_accessible may fall in an infinite loop due to "badly formed" strings when a call to tm_char_forwards is done via the call to closest_inside which return a "non accessible" path.

This can appear e.g. closest_inside is called on something like <image|<tuple|<#__raw_data__>> ... >

I played a bit with it but I am not convinced at which level this should be corrected: in the source or in drds. A dead loop detection could be added in the loop of make_cursor_accessible, but I guess it is not the root of the bug. Maybe the call to next_valid is false due to bad drd…

Also, I guess that serious performances issues are involved by this bug, since tm_char_forwards can be called unnecessarily on quite big strings (usually Mbytes) in raw-data tags.

Here is an example of TeXmacs file which fall into a loop (try to move between screens, then move up and down the cursor).

Francois Poulain <fpoulain>
Project Member

 

Attached Files
file #32922:  test_inserted_image.tm added by fpoulain (5KiB - text/texmacs)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by pjoyez (Posted a comment)
  • -unavailable- added by mdbenito (Posted a comment)
  • -unavailable- added by fpoulain (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 7 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Sun 19 Mar 2017 09:34:46 AM UTCpjoyezStatusConfirmed=>Fixed
      Open/ClosedOpen=>Closed
    Sun 22 Mar 2015 09:56:32 AM UTCmdbenitoPriority8 - Very high=>9 - Immediate
    Sat 07 Mar 2015 10:04:08 PM UTCmdbenitoPriority5 - Normal=>8 - Very high
      Item GroupNone=>Error
      StatusNone=>Confirmed
    Wed 28 Jan 2015 04:25:26 PM UTCfpoulainAttached File-=>Added test_inserted_image.tm, #32922

    Back to the top


    Powered by Savane 3.1-cleanup1