patchGNU Octave - Patches: patch #8913, remove zlib from public API

 
 

patch #8913: remove zlib from public API

Submitter:  Mike Miller <mtmiller>
Submitted:  Fri 19 Feb 2016 05:07:44 AM UTC
   
 
Category:  Core : other Priority:  5 - Normal
Status:  None Privacy:  Public
Assigned to:  None Open/Closed:  Open
* Mandatory Fields

Add a New Comment Rich Markup
   

Sat 18 Jan 2020 11:03:53 AM UTC, comment #2: 

Review for Octave 6.

No progress for 4 years.

Kai Torben Ohlhus <siko1056>
Group Member
Fri 19 Feb 2016 02:22:56 PM UTC, comment #1: 

I thought about a solution like this, and maybe it is sufficient for now.

Longer term, I am thinking about simplifying the octave_stdiostream classes by hiding the difference between FILE and gzFile in a separate class.  Then we might be able to hide our use of zlib in a single file instead of having details of zlib spread over multiple files in Octave.

John W. Eaton <jwe>
Group administrator
Fri 19 Feb 2016 05:07:44 AM UTC, original submission:  

First attempt at removing zlib from the public Octave API.

I think this may fail to build when building without zlib, just posting for now to show my work. Building with zlib succeeds and all tests pass as before. Public header files no longer include zlib.h or need a HAVE_ZLIB macro.

This is related to bug #41027.

Mike Miller <mtmiller>
Group Member

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #36393:  zlib.diff added by mtmiller (13KiB - text/x-diff)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by siko1056 (Posted a comment)
  • -email is unavailable- added by lachlan (Updated the item)
  • -email is unavailable- added by jwe (Posted a comment)
  •  

    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 logged-in users can vote.

     

    Follow 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2020-01-18 siko1056 StatusIn Progress None
    2019-02-26 mtmiller Carbon-CopyRemoved 80942 -
    2016-07-07 lachlan CategoryNone Core : other
    2016-02-19 mtmiller Attached File- Added zlib.diff, #36393

    Back to the top

    Powered by Savane 3.13-758e.
    Corresponding source code