bugGNU TeXmacs - Bugs: bug #62850, Crash on highlighting movement in...

 
 

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

bug #62850: Crash on highlighting movement in macro definition

Submitter:  Gennady Uraltsev <guraltsev>
Submitted:  Mon 01 Aug 2022 04:10:05 PM UTC
   
 
Category:  Editor Priority:  5 - Normal
Item Group:  Error Status:  Confirmed
Privacy:  Public Assigned to:  None
Originator Name:  Open/Closed:  Open
Release:  None Release:  2.1.1
Fixed Release:  None Fixed Release: 
Keywords: 

Thu 05 Jan 2023 07:33:06 PM UTC, comment #3: 

The crash can be prevented by changing the first test in edit_select_rep::compute_selection

if (is_table_selection (et, p1, p2, true)) {
->
if (!in_source () && is_table_selection (et, p1, p2, true)) {

but it's just a ad hoc change for this bug and a broader perspective on the code is probably needed. Also I find it surprising the bug does not occur more systematically when working with tables in source mode.

Philippe Joyez <pjoyez>
Group Member
Wed 10 Aug 2022 05:46:01 PM UTC, comment #2: 

Somehow the row calculation is off. The `row` argument to `table_search_cell`, the function where the error occurs equals 86913880, which is obviously far outside the table in question.

Jeroen Wouters <jeronim>
Group Member
Wed 10 Aug 2022 05:39:44 PM UTC, comment #1: 

I get an error `Throwing path is too short`. The debugger says the error happens on line `src/Data/Tree/tree_select.cpp:321`, "t= t [row];"

Jeroen Wouters <jeronim>
Group Member
Mon 01 Aug 2022 04:10:05 PM UTC, original submission:  

I defined a table-based macro in my preamble. It works perfectly well, but TeXmacs crashes when trying to select a piece of code with the keyboard or with the mouse (I am talking about the actual definition code in the preamble).

I am attaching the offending file. To reproduce, move your cursor just in front of "body", hold down "Shift" and move cursor up once or twice.

I reproduced it both under Linux and under windows

Instead of using the template you can also paste the following TeXmacs scheme code:


(assign "ltae"
        (macro "body"
          (tabular*
           (tformat
            (twith "table-valign" "T")
            (twith "table-min-cols" "1")
            (twith "table-max-cols" "1")
            (cwith "1" "-1" "1" "-1" "cell-halign" "l")
            (cwith "1" "-1" "1" "-1" "cell-lsep" "0ln")
            (cwith "1" "-1" "1" "-1" "cell-bsep" "0sep")
            (cwith "1" "-1" "1" "-1" "cell-tsep" "0sep")
            (arg "body")))))


Gennady Uraltsev <guraltsev>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attached Files
file #53498:  crash-on-tabledef.tm added by guraltsev (549B - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by pjoyez (Posted a comment)
  • -email is unavailable- added by jeronim (Updated the item)
  • -email is unavailable- added by guraltsev (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

     

    Follow 3 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2022-08-07 jeronim Item GroupNone Error
        StatusNone Confirmed
    2022-08-01 guraltsev Attached File- Added crash-on-tabledef.tm, #53498

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code