bugGNU Octave - Bugs: bug #52408, [io] odsread fails because JOD...

 
 

bug #52408: [io] odsread fails because JOD interface falsely recognized as available

Submitted by:  Marshall <marsian>
Submitted on:  Tue 14 Nov 2017 08:49:07 PM UTC  
 
Category: Octave Forge PackageSeverity: 3 - Normal
Priority: 5 - NormalItem Group: Incorrect Result
Status: FixedAssigned to: Philip Nienhuis <philipnienhuis>
Originator Name: MarsianOpen/Closed: Closed
Release: otherOperating System: Any

Add a New Comment(Rich Markup)
   

You are not logged in

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

 

Sun 19 Nov 2017 03:55:21 PM UTC, comment #2:

After checking the other interface dependency check functions I fixed it here:
http://hg.code.sf.net/p/octave/io/rev/47fe0439ac07

Thanks for reporting & the hint.

Closing report

Philip Nienhuis <philipnienhuis>
Project MemberIn charge of this item.
Wed 15 Nov 2017 05:43:44 PM UTC, comment #1:

Thanks that's a good one, I'll fix it in the repo.

I didn't note it because as developer I have all Java support classes available. So I'd better think about a test case for incomplete Java support class cases.

What strikes me is that you're the first one to report this fairly old bug while there are several bug reports and private emails to me about other issues where in hindsight this should have been mentioned as well.

BTW, if you have Java installed I'd advise to download & install jOpenDocument anyway. It is by far the fastest .ods I/O interface; OCT is by far the slowest. Plus, jOpenDocument's license is GPL.

Philip Nienhuis <philipnienhuis>
Project MemberIn charge of this item.
Tue 14 Nov 2017 08:49:07 PM UTC, original submission:

I run the following and get the following error:

It happened in io version 2.4.5, but still happens when updating to 2.4.8. I believe this happens because the JOD interface is being "detected" as available, and being used, even though it is not available. I think I traced it down to _JOD_chk_sprt_ where the last line always sets the check to valid for JOD:

If I comment out this last line with chk = 1;, then it works.

Marshall <marsian>

 

(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 marsian (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 7 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Sun 19 Nov 2017 03:55:21 PM UTCphilipnienhuisStatusConfirmed=>Fixed
      Open/ClosedOpen=>Closed
      Release4.2.1=>other
      Operating SystemMicrosoft Windows=>Any
    Sun 19 Nov 2017 03:55:20 PM UTCphilipnienhuisItem GroupUnexpected Error=>Incorrect Result
    Wed 15 Nov 2017 05:43:44 PM UTCphilipnienhuisStatusNone=>Confirmed
      Assigned toNone=>philipnienhuis

    Back to the top


    Powered by Savane 3.1-cleanup1