bugGNU Octave - Bugs: bug #37406, GUI/IDE can't track more than one...

 
 

bug #37406: GUI/IDE can't track more than one file saved outside the editor

Submitted by:  Dan Sebald <sebald>
Submitted on:  Thu 20 Sep 2012 09:36:14 AM UTC  
 
Category: GUISeverity: 3 - Normal
Priority: 5 - NormalItem Group: Incorrect Result
Status: FixedAssigned to: None
Originator Name: Open/Closed: Closed
Release: devOperating System: GNU/Linux

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

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

 

Tue 25 Dec 2012 09:26:58 PM UTC, comment #2:

Fixed with changeset
http://hg.savannah.gnu.org/hgweb/octave/rev/424edeca3c66

Torsten <ttl>
Project Member
Thu 04 Oct 2012 05:24:05 AM UTC, comment #1:

[patch #7857] is addresses this bug.

Dan Sebald <sebald>
Thu 20 Sep 2012 09:36:14 AM UTC, original submission:

We're all learning here with this Qt stuff...

Here's an observation about the use of the static function warning. The "warning" function sort of halts everything, and isn't the type of thing to use for notification of a file-update-outside-application. Note from the Qt documentation:

QMessageBox::warning

Warning: Do not delete parent during the execution of the dialog. If you want to do this, you should create the dialog yourself using one of the QMessageBox constructors.

Yes, we are issuing a warning to the user about the file being saved outside the file editor, but the problem is that when the warning message is displayed Qt will not handle another file being updated outside the application. (Open two files in the GUI/IDE editor and at the console command line type "touch file1.m; touch file2.m". The second file will not be recognized.)

Generally, a program isn't supposed to wait on entry from the user, i.e., polling loop. Instead, we'll have to figure out a way, per the documentation's suggestion, to construct the QMessageBox for the warning and connect some slots to the signals it issues to handle the user's response.

Let's save this for later when we clear up a few more show-stoppers. Please downgrade the report priority.

Dan Sebald <sebald>

 

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

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by ttl (Posted a comment)
  • -unavailable- added by jordigh (Updated the item)
  • -unavailable- added by sebald (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 project members can vote.

     

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

     

     

    Follow 3 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Tue 25 Dec 2012 09:26:58 PM UTCttlStatusPatch Submitted=>Fixed
      Open/ClosedOpen=>Closed
    Thu 04 Oct 2012 01:06:22 PM UTCjordighStatusNone=>Patch Submitted

    Back to the top


    Powered by Savane 3.1-cleanup