Re: [Jazzlib-developers] Patched latest version to fix various bugs

[Jochen H. <hoe...@gm...>]

Hello Christian,

On Mon, 21 Feb 2005 13:17:12 +0100, Christian Schlichtherle
<chr...@sc...> wrote:
> After zipping and unzipping some Gigabytes of data with 15.000 files, I have
> modified the sources to support/fix the following: 
> 
> + Added new constructors to support arbitrary character sets for entry names
> and comments for international ZIP files. As an example, CP437 may be used
> which seems to be most widely used. 

I'm not sure whether we are allowed to add new constructors.  Jazzlib is used
in classpath as java.util.zip implementation, that should be compatible with
Sun's java class library. OTOH the current hard coded solution is certainly
suboptimal and an additional public constructor does not break binary
compatibility.

> + Fixed bug in date/time conversion. 
>
> + Enabled support for ZIP files up to 4 GB. The current version only
> supports 2GB due to erratic signed int usage. 
> 
> If this sounds interesting, then I would be more than happy to provide my
> modified sources for inclusion in the standard distribution. 

Yes it sounds interesting

> Now the bad news: I never worked with CVS! So how could I contribute? 

The best way is to create a diff and send it to this list, or add it
to the patch
tracker. The patch needs to be applied to classpath, too, so it doesn't make
sense if you get write access to the CVS without write access to classpath
CVS.

If you work on a checked out CVS repository, you can get easily get a diff.
E.g. if you use the cvs command line tool that comes with cygwin, you can
just do "cvs diff -u" in the repository.  I think the graphical cvs tools also 
have something that does this.  Please use unified (-u) diffs, as they are the
best to read.

You can also use the diff utility (from diffutils) to compare your modified 
directory with an original one: "diff -ur jazzlib-orig jazzlib-modified"

If your contribution is large, we need you to assign copyrights to FSF due 
to legal reasons.  This is because the patches should go to GNU classpath,
too.

Regards,
  Jochen

-- 
Jochen Hoenicke, University of Oldenburg, 26111 Oldenburg, Germany
Email: hoe...@in...  Tel: +49 441 798 3124