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 Function Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Incorrect Result
Status:  Fixed Assigned to:  Philip Nienhuis <philipnienhuis>
Originator Name:  Open/Closed:  Closed
Release:  3.8.1 Operating 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)

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.

textscan('123','','whitespace',' ')

sergey plotnikov <nul0m>

 

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

Attach Files:
   
   
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
  • -email is unavailable- added by rik5 (Posted a comment)
  • -email is unavailable- added by nul0m (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can add your encouragement to it.
    This task has 0 encouragements so far.

    Only project members can vote.

     

     

     

    Follow 9 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2014-03-22 rik5 StatusPatch Submitted => Fixed
        Open/ClosedOpen => Closed
    2014-03-13 rik5 Item GroupSegfault, Bus Error, etc. => Incorrect Result
    2014-03-11 philipnienhuis Attached File- => Added bug41824-v2.cs, #30861
        StatusConfirmed => Patch Submitted
        Releasedev => 3.8.1
    2014-03-10 philipnienhuis StatusNone => Confirmed
        Assigned toNone => philipnienhuis
    2014-03-10 nul0m Attached File- => Added bug41824.patch, #30847

    Back to the top


    Powered by Savane 3.5