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

 
 

bug #41123: GUI: paste may eat newlines

Submitted by:  Felipe G. Nievinski <fgnievinski>
Submitted on:  Mon 06 Jan 2014 02:58:42 PM UTC  
 
Category: GUISeverity: 3 - Normal
Priority: 5 - NormalItem Group: Incorrect Result
Status: FixedAssigned to: John Donoghue <lostbard>
Originator Name: Open/Closed: Closed
Release: 3.8.0Operating 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 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>
Project Administrator
Wed 08 Jan 2014 05:57:26 AM UTC, comment #14:

I really meant a gui satisfaction wiki...

Michael Godfrey <godfrey>
Project 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>
Project 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>
Project 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>
Project 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>
Project 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>
Project Administrator
Wed 08 Jan 2014 01:30:51 AM UTC, comment #8:

Pushed http://hg.savannah.gnu.org/hgweb/octave/rev/450f50d3eb18

John Donoghue <lostbard>
Project MemberIn charge of this item.
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>
Project 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>
Project MemberIn charge of this item.
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:

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>
Project MemberIn charge of this item.
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.

Felipe, what OS were you running?

John Donoghue <lostbard>
Project MemberIn charge of this item.
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>
Project 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:

You'll likely get results similar to these:

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 File(s):
   
   
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
  • -unavailable- added by jwe (Posted a comment)
  • -unavailable- added by godfrey (Posted a comment)
  • -unavailable- added by jordigh (Posted a comment)
  • -unavailable- added by lostbard (Posted a comment)
  • -unavailable- added by rik5 (Posted a comment)
  • -unavailable- added by fgnievinski (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 8 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Thu 09 Jan 2014 05:33:30 PM UTCrik5StatusReady For Test=>Fixed
      Open/ClosedOpen=>Closed
    Wed 08 Jan 2014 01:30:51 AM UTClostbardStatusPatch Submitted=>Ready For Test
      Assigned toNone=>lostbard
    Tue 07 Jan 2014 03:58:43 PM UTClostbardAttached File-=>Added newline.patch, #30215
      Item GroupNone=>Incorrect Result
      StatusWorks For Me=>Patch Submitted
    Mon 06 Jan 2014 09:33:42 PM UTCrik5StatusNone=>Works For Me

    Back to the top


    Powered by Savane 3.1-cleanup1