bugGNU Octave - Bugs: bug #41123, GUI: paste may eat newlines

 
 

bug #41123: GUI: paste may eat newlines

Submitter:  Felipe G. Nievinski <fgnievinski>
Submitted:  Mon 06 Jan 2014 02:58:42 PM UTC
   
 
Category:  GUI Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Incorrect Result
Status:  Fixed Assigned to:  lostbard
Originator Name:  Open/Closed:  * Closed
Release:  * 3.8.0 Operating System:  * Microsoft Windows
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Jump to the original submission

Wed 08 Jan 2014 06:09:05 AM UTC, comment #15: 

We could use the community news pane to ask for feedback.

Is there free software (or a free software friendly site) for doing an survey?

John W. Eaton <jwe>
Group administrator
Wed 08 Jan 2014 05:57:26 AM UTC, comment #14: 

I really meant a gui satisfaction wiki...

Michael Godfrey <godfrey>
Group Member
Wed 08 Jan 2014 05:56:03 AM UTC, comment #13: 

Ah, I remember.  And, so long as the merge
sequence stable -> gui-release -> default
works correctly all should be OK.

And, the sooner the goal of gui enabled by
default the better. (Obvious remark!)

Is there more that could be done to generate
feedback about the gui?  Not just bug reports,
but users at least saying yes/no to the question
Is it good enough for me to use routinely?

Maybe a gui-bug wiki?

Michael Godfrey <godfrey>
Group Member
Wed 08 Jan 2014 05:26:56 AM UTC, comment #12: 

I agree with Rik that serious problems with the GUI that can be fixed easily and without breaking binary compatibility with previous 3.8.x releases should go on stable.

We will keep the gui-release branch until the GUI is good enough that we can release a version of Octave with the GUI enabled by default.

I don't think it's wise to call 3.8.x a stable release series and at the same time make potentially disruptive GUI-related changes there.  And now that default has classdef, which is unlikely to be ready for release very soon, we would be in the position of delaying the release of the GUI while we wait for classdef to stabilize.  So I think we are stuck with the 3 branches for now.

If people want to get rid of the gui-release branch, then the solution is to work toward making the GUI good enough to release.
That work should be done primarily on the gui-release branch, but some changes may be appropriate for stable.

John W. Eaton <jwe>
Group administrator
Wed 08 Jan 2014 05:04:34 AM UTC, comment #11: 

Following Jordi's comment, I cannot remember
when (or exactly why) the gui-release branch
is planned to closed, but the sooner the better.
Whenever there is a choice about what
branch patches should go in there is a good chance
that some will go in a wrong place.

Michael Godfrey <godfrey>
Group Member
Wed 08 Jan 2014 04:48:49 AM UTC, comment #10: 

Not being able to paste correctly seemed serious enough to put on the stable branch so it is picked up in the 3.8.1 release.  More trivial things like the height of tabs in the document viewer can wait for the gui-release branch to finalize.

Rik <rik5>
Group administrator
Wed 08 Jan 2014 04:34:24 AM UTC, comment #9: 

Shouldn't this have gone into the gui-release branch instead?

Perhaps it was a bad idea to introduce that branch and it's causing more confusion about where to place patches than it's helping.

Jordi GutiƩrrez Hermoso <jordigh>
Group Member
Wed 08 Jan 2014 01:30:51 AM UTC, comment #8: 
John Donoghue <lostbard>
Group Member
Wed 08 Jan 2014 01:21:57 AM UTC, comment #7: 

I don't experience this problem on Windows XP, but I cross-built Octave using MXE and your patch and everything still works for me.  So, if it solves the problem on your system then go ahead and commit the change to the stable branch.

Rik <rik5>
Group administrator
Tue 07 Jan 2014 04:53:57 PM UTC, comment #6: 

starting some have processing outside octave (e.g., video compressing) should make this bug more noticeable.

Felipe G. Nievinski <fgnievinski>
Tue 07 Jan 2014 03:58:43 PM UTC, comment #5: 

This appears to fixe it on my system. (attached)

With luck it wont break it for systems that don't appear to not work.





(file #30215)

John Donoghue <lostbard>
Group Member
Tue 07 Jan 2014 04:56:49 AM UTC, comment #4: 

I'm in Win 8

pasting seems to suffer a lag retrieving the contents from the clipboard.  I noticed that the editor never suffers from this problem -- it seems specific to the GUI command window.  Does it do any parsing before return is entered?

Felipe G. Nievinski <fgnievinski>
Tue 07 Jan 2014 02:01:52 AM UTC, comment #3: 

Trying the same without the semicolon:

>> a=1
a =  1
>> b=2c=3
parse error:

  syntax error

>>> b=2c=3
       ^

>> d=4
d =  4
>> e=5
e =  5


It only happens in the GUI, and only in Windows - both using Ctrl-V and using the right click paste (although it was harder to get it to happen with right click paste)

John Donoghue <lostbard>
Group Member
Tue 07 Jan 2014 01:14:43 AM UTC, comment #2: 

After 4-5 tries, I could get it to behave the same. Octave 3.8.0, Windows 7.


>> a = 1;
>> b = 2;
>> c = 3;
>> d = 4;
>> e = 5;
>> a = 1;
>> b = 2;
>> c = 3;d = 4;
>>
>> e = 5;
>> a = 1;
>> b = 2;c = 3;
>>
>> d = 4;
>> e = 5;
>> a = 1;
>> b = 2;c = 3;
>> d = 4;
>>
>> e = 5;


Felipe, what OS were you running?

John Donoghue <lostbard>
Group Member
Mon 06 Jan 2014 09:33:42 PM UTC, comment #1: 

This works for me with 3.8.0 cross-built using MXE.  I pasted several times using the keyboard shortcut Ctrl+V and also using right-click->paste and it works every time.  This was on Windows XP.

Rik <rik5>
Group administrator
Mon 06 Jan 2014 02:58:42 PM UTC, original submission:  

Please copy this block and paste it multiple times into the GUI terminal/console:


a = 1;
b = 2;
c = 3;
d = 4;
e = 5;


You'll likely get results similar to these:


>> a = 1;
>> b = 2;
>> c = 3;
>> d = 4;
>> e = 5;
>>
>> b = 2;
>> c = 3;
>> d = 4;e = 5;
>>
>> a = 1;
>> b = 2;c = 3;
>> d = 4;
>> e = 5;
>>
>> a = 1;
>> b = 2;
>> c = 3;
>> d = 4;e = 5;
>>
>> a = 1;
>> b = 2;
>> c = 3;
>> d = 4;e = 5;
>>
>> a = 1;
>> b = 2;c = 3;
>> d = 4;e = 5;


This is a minimal reproducible harmless example, but I've seen this behavior eventually breaking code pasted on the GUI console.

Felipe G. Nievinski <fgnievinski>

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #30215:  newline.patch added by lostbard (2KiB - 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 jwe (Posted a comment)
  • -email is unavailable- added by godfrey (Posted a comment)
  • -email is unavailable- added by jordigh (Posted a comment)
  • -email is unavailable- added by lostbard (Posted a comment)
  • -email is unavailable- added by rik5 (Posted a comment)
  • -email is unavailable- added by fgnievinski (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.

    Only group members can vote.

     

    Follow 8 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2014-01-09 rik5 StatusReady For Test Fixed
        Open/ClosedOpen Closed
    2014-01-08 lostbard StatusPatch Submitted Ready For Test
        Assigned toNone lostbard
    2014-01-07 lostbard Attached File- Added newline.patch, #30215
        Item GroupNone Incorrect Result
        StatusWorks For Me Patch Submitted
    2014-01-06 rik5 StatusNone Works For Me

    Back to the top

    Powered by Savane 3.13-4448.
    Corresponding source code