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

 
 

bug #41824: textscan: undefined whitespace variable

Submitter:  sergey plotnikov <nul0m>
Submitted:  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:  philipnienhuis
Originator Name:  Open/Closed:  * Closed
Release:  * 3.8.1 Operating System:  * GNU/Linux
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Jump to the original submission

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

Thanks for pushing.

Philip Nienhuis <philipnienhuis>
Group Member
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>
Group administrator
Thu 13 Mar 2014 09:46:28 PM UTC, comment #4: 

Go ahead and push it to stable.

Rik <rik5>
Group 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>
Group Member
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>
Group Member
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)
  •  

    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 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.13-d3ae.
    Corresponding source code