Mailing-List: contact dev-help@harmony.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@harmony.apache.org
Received-SPF: pass (athena.apache.org: domain of sianjanuary@googlemail.com
 designates 209.85.146.178 as permitted sender)
DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=googlemail.com; s=gamma;
        h=message-id:date:from:to:subject:in-reply-to:mime-version
         :content-type:references;
        b=mx3nkP10mSmKGwYC9n4st33+lrApzq9eAvveA5ErukayHM72cnP2Rj97CkI6ZAPhBn
         J+EjnlCI0wzCALY9QDZy7vORO16/CCuZ/WL9e+RXg/VWsSZh5UyOsah/WuLMSVhHJnB7
         dm1sshsHe3KDOdjcx2LGq5pbfPy8ZWjfP6reI=
Message-ID: <fc2fc95c0807180629u39656be1t9119c495da4dcf86@mail.gmail.com>
Date: Fri, 18 Jul 2008 14:29:07 +0100
From: "Sian January" <sianjanuary@googlemail.com>
To: dev@harmony.apache.org
Subject: Re: [classlib][pack200] Decoupling I/O and processing for unpacking
 scenario
In-Reply-To: <4bebff790807180624i2f95a703t5503f38e623e6b95@mail.gmail.com>
MIME-Version: 1.0
Content-Type: multipart/alternative;
	boundary="----=_Part_49665_11663180.1216387748069"
References: <4bebff790807170935y1640f01am9eeb5decdec1c36@mail.gmail.com>
	 <fc2fc95c0807180216u1187ecch6ecf4897a49958e7@mail.gmail.com>
	 <4bebff790807180233o6b487449l91264f162bd43cf7@mail.gmail.com>
	 <fc2fc95c0807180616g5c54855amb7d96e8a8d0ae783@mail.gmail.com>
	 <4bebff790807180624i2f95a703t5503f38e623e6b95@mail.gmail.com>

------=_Part_49665_11663180.1216387748069
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

On 18/07/2008, Aleksey Shipilev <aleksey.shipilev@gmail.com> wrote:
>
> Sian,
>
> On Fri, Jul 18, 2008 at 5:16 PM, Sian January
> <sianjanuary@googlemail.com> wrote:
> > There is an optional field in the segment header - archive_size that can
> > give the size in bytes or be 0 if the compressor chooses not to specify
> the
> > size.  I've just checked and Sun's compressor does specify this, and we
> can
> > do in ours when it's written, although we will also have to allow for the
> > possibility that it will be 0.
> Awesome! Am I understanding correctly: this value determines the size
> of segment? If yes, can you point me how to access this value? Is
> there API in current implementation?


Yes - use SegmentHeader.getArchiveSize()

I think we can just check this field in Archive and:
> a. if the field != 0, then synchronously copy out entire segment from
> inputStream to Segment's internal and do the unsynchronized
> Segment.read() against internal buffer.
> b. if the field == 0, then synchronously do the Segment.read() agains
> the inputStream


Sounds like a good plan.

Thanks,
> Aleksey.
>


-- 
Unless stated otherwise above:
IBM United Kingdom Limited - Registered in England and Wales with number
741598.
Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU

------=_Part_49665_11663180.1216387748069--