bugGNU Octave - Bugs: bug #45046, textread.m: eol_char uninitialized...

 
 

bug #45046: textread.m: eol_char uninitialized when using endofline keyword argument

Submitted by:  None
Submitted on:  Thu 07 May 2015 03:48:37 PM UTC  
 
Category: Octave FunctionSeverity: 3 - Normal
Priority: 5 - NormalItem Group: Incorrect Result
Status: FixedAssigned to: Philip Nienhuis <philipnienhuis>
Originator Name: Originator Email: -unavailable-
Open/Closed: ClosedRelease: dev
Operating System: Any

Add a New Comment(Rich Markup)
   

You are not logged in

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

 

Tue 19 May 2015 06:22:20 PM UTC, comment #5:

Fixed in this cs:
http://hg.savannah.gnu.org/hgweb/octave/rev/935832827f47

Philip Nienhuis <philipnienhuis>
Project MemberIn charge of this item.
Mon 18 May 2015 06:38:03 PM UTC, comment #4:

Yes, stable looks like the correct branch.

Rik <rik5>
Project Administrator
Mon 18 May 2015 06:25:48 PM UTC, comment #3:

"Expert"... I'm flattered :-)

I'll have a look soon (perhaps tomorrow night).
You want the fix on default or stable? I think it'll be a plain bug fix so stable looks appropriate to me

Philip Nienhuis <philipnienhuis>
Project MemberIn charge of this item.
Thu 14 May 2015 03:06:06 PM UTC, comment #2:

@Philip: Since you are the expert on textread, how should the change be made? Should we copy from texstscan or add an else case near line 120?

Rik <rik5>
Project Administrator
Fri 08 May 2015 09:36:58 AM UTC, comment #1:

Confirmed.
The corresponding stanza in textscan.m (L.224+) seems better.

Philip Nienhuis <philipnienhuis>
Project MemberIn charge of this item.
Thu 07 May 2015 03:48:37 PM UTC, original submission:

Missing else case in scripts/io/textread.m @ line 120 causes eol_char to be uninitialized on first use @ line 143 if 'endofline' keyword argument is utilized when provided with a valid end of line character.

Anonymous

 

(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 rik5 (Posted a comment)
  •  

    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 5 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Tue 19 May 2015 06:22:20 PM UTCphilipnienhuisStatusConfirmed=>Fixed
      Open/ClosedOpen=>Closed
      Release3.8.2=>dev
    Mon 18 May 2015 06:25:48 PM UTCphilipnienhuisAssigned toNone=>philipnienhuis
    Fri 08 May 2015 09:36:58 AM UTCphilipnienhuisStatusNone=>Confirmed

    Back to the top


    Powered by Savane 3.1-cleanup1