bugGNU Octave - Bugs: bug #37087, empty struct not empty after being...

 
 

bug #37087: empty struct not empty after being saved in MATLAB binary data format.

Submitted by:  Guillaume <gyom>
Submitted on:  Mon 13 Aug 2012 07:14:00 PM UTC  
 
Category: LibrariesSeverity: 3 - Normal
Priority: 5 - NormalItem Group: Matlab Compatibility
Status: FixedAssigned to: None
Originator Name: GuillaumeOpen/Closed: Closed
Release: devOperating System: Any

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)

Tue 16 Oct 2012 07:39:32 PM UTC, comment #8:

Fixed in this changeset (http://hg.savannah.gnu.org/hgweb/octave/rev/39fd2ad26fb7). You will need to pull from the latest Mercurial sources and build from scratch in order to see the fix.

Rik <rik5>
Project Administrator
Tue 16 Oct 2012 12:51:16 AM UTC, comment #7:

I tested the patch and it works fine. How would you like to be credited in the Mercurial ChangeLog? Ordinarily we do "Firstname Lastname <email@address>". If you have privacy concerns we can also work out something else.

Rik <rik5>
Project Administrator
Thu 20 Sep 2012 04:44:07 PM UTC, comment #6:

I've been using this patch for some time now:

It fixed the bug I was reporting and I didn't see any side effect since then.
Could someone review the change and push it if OK? Many thanks.

Guillaume <gyom>
Fri 17 Aug 2012 04:45:18 PM UTC, comment #5:

It seems to work if I change l.1147 of src/interp-core/ls-mat5.cc so that instead of:
Octave_map m (dim_vector (1, 1));
it reads:
Octave_map m (dims);
but then I don't understand how it could work before when one of the dimension of the struct array was greater than 1.

Guillaume <gyom>
Wed 15 Aug 2012 06:27:08 PM UTC, comment #4:

I did actually look at this file earlier but couldn't figure out where the problem was taking place.

Just got GDB running following this:
http://wiki.octave.org/Debugging_Octave
but nothing meaningful yet...

Guillaume <gyom>
Wed 15 Aug 2012 04:04:45 PM UTC, comment #3:

Sorry, I meant, here:

http://hg.savannah.gnu.org/hgweb/octave/file/3b8275fed4dd/src/interp-core/ls-mat5.cc#l1144

Jordi GutiƩrrez Hermoso <jordigh>
Project Administrator
Wed 15 Aug 2012 03:58:10 PM UTC, comment #2:

Wanna have a stab at debugging the problem yourself? Run Octave in a debugger and set a breakpoint here:

http://hg.savannah.gnu.org/hgweb/octave/file/3b8275fed4dd/src/interp-core/ls-mat5.cc#l118

Jordi GutiƩrrez Hermoso <jordigh>
Project Administrator
Wed 15 Aug 2012 01:57:12 PM UTC, comment #1:

To be more precise, the problem is with load() and not save(), i.e. file A.mat is fine but is read erroneously (found out by cross-checking with MATLAB).

Guillaume <gyom>
Mon 13 Aug 2012 07:14:00 PM UTC, original submission:

The size of an empty struct (struct([])) changes when saved in MATLAB binary data format:

Guillaume <gyom>

 

(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)
  • -unavailable- added by jordigh (Posted a comment)
  • -unavailable- added by gyom (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 4 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Tue 16 Oct 2012 07:39:32 PM UTCrik5StatusIn Progress=>Fixed
      Open/ClosedOpen=>Closed
    Tue 16 Oct 2012 12:51:16 AM UTCrik5CategoryNone=>Libraries
      StatusNone=>In Progress

    Back to the top


    Powered by Savane 3.1-cleanup1