View Issue Details

IDProjectCategoryView StatusLast Update
0005690JEDI Code LibraryJclCompressionpublic2012-01-30 22:24
ReporterpvanlogchemAssigned Tooutchy 
PrioritynormalSeverityfeatureReproducibilityN/A
Status resolvedResolutionfixed 
Product VersionVersion 2.4 
Target VersionFixed in VersionVersion 2.4 
Summary0005690: Add option to use Delphi zlib
DescriptionCurrently, there are 3 methods to use ZLIB : ZLIB_LINKDLL, ZLIB_LINKONREQUEST and ZLIB_STATICLINK. I propose a ZLIB_USEDELPHIZLIB which uses Delphi's own Zlib unit.

This might involve a bit of work, but it could reduce executable size a bit (by preventing linking in multiple versions of the zlib OBJect files) and at the same time solves any 64 bit porting issues (as that seems to have been done by Embarcadero themselves already).


By the way : Now that Delphi XE2 has it's own System.ZLib unit, adding a HAS_UNIT_ZLIB define to the jcl.inc might be a good idea (and while we're at it, you another define, HAS_UNIT_ZIP, could be added for the System.Zip unit that comes with Delphi XE2).
TagsNo tags attached.
Fixed in GIT commit
Fixed in SVN revision3695
IDE versionDelphi/C++Builder XE2

Activities

outchy

2011-10-24 17:09

administrator   ~0019056

This applies to PCRE too.

outchy

2012-01-21 17:10

administrator   ~0019330

This is committed in revision 3695.

Only XE2's ZLib is up-to-date, the ZLIB_RTL won't work for XE and older.

Static link of ZLib has been supported for 32 and 64 bit application since revision 3688.

Issue History

Date Modified Username Field Change
2011-10-24 13:24 pvanlogchem New Issue
2011-10-24 13:24 pvanlogchem IDE version => Delphi/C++Builder XE2
2011-10-24 17:09 outchy Note Added: 0019056
2011-10-24 17:09 outchy Assigned To => outchy
2011-10-24 17:09 outchy Status new => acknowledged
2012-01-21 17:10 outchy Note Added: 0019330
2012-01-21 17:10 outchy Status acknowledged => feedback
2012-01-30 22:24 outchy Fixed in revision => 3695
2012-01-30 22:24 outchy Status feedback => resolved
2012-01-30 22:24 outchy Fixed in Version => Version 2.4 (Subversion repository/Daily zips)
2012-01-30 22:24 outchy Resolution open => fixed