bugGNU Octave - Bugs: bug #41824, textscan: undefined whitespace...

 
 

bug #41824: textscan: undefined whitespace variable

Submitted by:  sergey plotnikov <nul0m>
Submitted on:  Mon 10 Mar 2014 06:05:26 PM UTC  
 
Category: Octave FunctionSeverity: 3 - Normal
Priority: 5 - NormalItem Group: Incorrect Result
Status: FixedAssigned to: Philip Nienhuis <philipnienhuis>
Originator Name: Open/Closed: Closed
Release: 3.8.1Operating System: GNU/Linux

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)

Sat 22 Mar 2014 03:44:48 PM UTC, comment #6:

Thanks for pushing.

Philip Nienhuis <philipnienhuis>
Project MemberIn charge of this item.
Sat 22 Mar 2014 04:21:28 AM UTC, comment #5:

Pushed the changeset to the stable branch here (http://hg.savannah.gnu.org/hgweb/octave/rev/2ce1f1966fc3).

Rik <rik5>
Project Administrator
Thu 13 Mar 2014 09:46:28 PM UTC, comment #4:

Go ahead and push it to stable.

Rik <rik5>
Project Administrator
Tue 11 Mar 2014 09:10:31 PM UTC, comment #3:

Your analysis wasn't so naive, rather right to the point.
I made a new cs following Octave coding style and added a test based on your example.

@other devs: where do we push this? I'd vouch for stable as it just fixes a bug dating back from July 2011 (3.6.0 IIRC) and doesn't add, or change, functionality.

(file #30861)

Philip Nienhuis <philipnienhuis>
Project MemberIn charge of this item.
Mon 10 Mar 2014 10:02:09 PM UTC, comment #2:

Confirmed (good catch!)

I'll have a look. Your patch is at least in the right region.

Philip Nienhuis <philipnienhuis>
Project MemberIn charge of this item.
Mon 10 Mar 2014 06:14:54 PM UTC, comment #1:

I believe it was just because of a missing line. In that case the naive patch i've just attached could be a solution.

sergey plotnikov <nul0m>
Mon 10 Mar 2014 06:05:26 PM UTC, original submission:

In case of empty format string and non-empty 'whitespace' option textscan() crashes crying that 'whitespace' is undefined. Just try the following code.

sergey plotnikov <nul0m>

 

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

Attach File(s):
   
   
Comment:
   

Attached Files
file #30861:  bug41824-v2.cs added by philipnienhuis (938B - text/x-csharp)
file #30847:  bug41824.patch added by nul0m (655B - text/x-patch - this one could be a solution)

 

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)
  • -unavailable- added by nul0m (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 9 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Sat 22 Mar 2014 04:21:28 AM UTCrik5StatusPatch Submitted=>Fixed
      Open/ClosedOpen=>Closed
    Thu 13 Mar 2014 09:46:28 PM UTCrik5Item GroupSegfault, Bus Error, etc.=>Incorrect Result
    Tue 11 Mar 2014 09:10:31 PM UTCphilipnienhuisAttached File-=>Added bug41824-v2.cs, #30861
      StatusConfirmed=>Patch Submitted
      Releasedev=>3.8.1
    Mon 10 Mar 2014 10:02:09 PM UTCphilipnienhuisStatusNone=>Confirmed
      Assigned toNone=>philipnienhuis
    Mon 10 Mar 2014 06:11:18 PM UTCnul0mAttached File-=>Added bug41824.patch, #30847

    Back to the top


    Powered by Savane 3.1-cleanup1