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

Group

Main
- Main
- View members
- Search
Homepage
Download
Mailing lists
Source code
Bugs
- Submit new
- Browse
- Reset to open
- Digest
- Dependencies
- ~~Export~~
- Get statistics
- Search
Tasks
- ~~Submit new~~
- Browse
- Reset to open
- Digest
- Dependencies
- ~~Export~~
- Get statistics
- Search
Patches
- Submit new
- Browse
- Reset to open
- Digest
- Dependencies
- ~~Export~~
- Get statistics
- Search

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>
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>
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>

(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.

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
2020-01-18	siko1056	Status	In Progress	None
2019-02-26	mtmiller	Carbon-Copy	Removed 80942	-
2016-07-07	lachlan	Category	None	Core : other
2016-02-19	mtmiller	Attached File	-	Added zlib.diff, #36393

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

patch #8913: remove zlib from public API

<a href="#postcomment">Post a Comment</a>

<a href="#discussion">Discussion</a>

<a href="#attached">Attached Files</a>

<a href="#dependencies">Dependencies</a>

<a href="#cc">Mail Notification Carbon-Copy List</a>

<a href="#votes">Votes</a>

<a href="#history">History</a>