bugGNU Octave - Bugs: bug #60469, F9 to run selected code stopped...

 
 

bug #60469: F9 to run selected code stopped working

Submitted by:  Philip Nienhuis <philipnienhuis>
Submitted on:  Wed 28 Apr 2021 07:54:33 PM UTC  
 
Category:  GUI Severity:  3 - Normal
Priority:  5 - Normal Item Group:  None
Status:  Confirmed Assigned to:  None
Originator Name:  Philip Nienhuis Open/Closed:  Open
Release:  dev Operating System:  Microsoft Windows

Add a New Comment (Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

( Jump to the original submission)

Mon 03 May 2021 07:05:20 PM UTC, comment #7: 

It looks like the connection of QScintilla's own signals is failing, among them &octave_qscintilla::copyAvailable (causing this bug) or, e.g., signals for indicating file mofification or updating line and row indicator in the status bar.

When octave is started with the environment variable "OCTAVE_SHOW_GUI_MESSAGES=1", there are altogether 13 occurrences of the error message "QObject::connect: signal not found in octave::octave_qscintilla" for each opened editor file.

Could the way how Qscintilla is linked to octave in the mxe build might prevent the new signal/slot connection method from finding the signals?

Torsten Lilge <ttl>
Project Member
Sun 02 May 2021 07:25:55 PM UTC, comment #6: 

As I wrote, it happened somewhere between 14th and 20th of April. The cset Rik refers to fits in perfectly.

Philip Nienhuis <philipnienhuis>
Project Member
Sun 02 May 2021 04:51:30 PM UTC, comment #5: 

comment #4:

> This might be related to jwe's attempt to get rid of SIGNAL/SLOT usage.


I was thinking about this, too and am starting to look for the break in the signal chain. But whatever the result will be, interestingly only Windows is affected. Let's see.

Torsten Lilge <ttl>
Project Member
Sun 02 May 2021 02:22:45 PM UTC, comment #4: 

This might be related to jwe's attempt to get rid of SIGNAL/SLOT usage.  See this cset

changeset:   29542:3d34b70b5a49
user:        John W. Eaton <jwe@octave.org>
date:        Fri Apr 16 23:06:32 2021 -0400
summary:     connect many Qt signals and slots without SIGNAL and SLOT macros

Using 'hg bisect' for Windows is slow because it takes so long to cross-build through the MXE system.  But this might be a clue.

Rik <rik5>
Project Administrator
Sun 02 May 2021 01:58:44 PM UTC, comment #3: 

After a complete rebuild of my MXE envrionment (I had to do a "make clean" after a system upgrade), I was finally able to test this on my windows virtual machine.

The "Run selected code" action remains disabled even when text is selected. The same holds for the copy action, which stays disabled, too, although selected text is available. From this, it seems that the signal chain that should inform the main editor widget from state changes in the current tab's qscintilla area is not working anymore in Windows.

Torsten Lilge <ttl>
Project Member
Thu 29 Apr 2021 06:19:39 PM UTC, comment #2: 

I will do a recent windows build (will take some time) and test on my virtual box.

Torsten Lilge <ttl>
Project Member
Wed 28 Apr 2021 09:59:57 PM UTC, comment #1: 

FWIW, I tried on a Linux system with the text 'x = pi' and F9 worked fine.  This is with cset

parent: 29565:7733ca1db419
use shared_ptr to manage memory for mutex object

Rik <rik5>
Project Administrator
Wed 28 Apr 2021 07:54:33 PM UTC, original submission:  

Steps to reproduce;

  • In GUI editor, enter any command, e.g.: a = 1
  • Select it  & press F9
  • expected result: code is executed in command window
  • observed result: nothing

The last version I built where it worked is from April 14.
The next version I built is from April 20, doesn't work there.

It does still work in the 6.2.1 nightlies

Philip Nienhuis <philipnienhuis>
Project Member

 

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

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by ttl (Posted a comment)
  • -email is unavailable- added by rik5 (Posted a comment)
  • -email is unavailable- added by philipnienhuis (Submitted the item)
  •  

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

    Only project members can vote.

     

     

     

    Follows 1 latest change.

    Date Changed by Updated Field Previous Value => Replaced by
    2021-05-02 ttl StatusNone => Confirmed

    Back to the top


    Powered by Savane 3.6