Mailing-List: contact harmony-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: harmony-dev@incubator.apache.org
Received-SPF: pass (hermes.apache.org: domain of weldonwjw@gmail.com
 designates 66.249.82.234 as permitted sender)
DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws;
        s=beta; d=gmail.com;
        h=received:message-id:date:from:to:subject:in-reply-to:mime-version:content-type:references;
        b=FB75hTS1f8vzajbdQQaub6h/co82lY7LODTZX9HvnzRlRQupVdOZLVlfynk8oVxihjDzxL2LIrXAWPSVaoGOhrW1m/xBbYSxO0Sr2YBm62uGvDqNt8Ba90VokPlB/BlxDXh71/6AWNKDt8ULSi2mWaJIbbX0/WWpBn4qJ20z0lU=
Message-ID: <4dd1f3f00609062031n57ff664ck976e424628ca65f5@mail.gmail.com>
Date: Wed, 6 Sep 2006 20:31:22 -0700
From: "Weldon Washburn" <weldonwjw@gmail.com>
To: harmony-dev@incubator.apache.org
Subject: Re: [drlvm] Cleaning insides of Class.h header
In-Reply-To: <e0f125db0609050325v75420bb0i7b3cdb0bcbf087d8@mail.gmail.com>
MIME-Version: 1.0
Content-Type: multipart/alternative;
	boundary="----=_Part_13526_23772423.1157599882527"
References: <e0f125db0607240324w15fba97bj3f1573826b3b27fb@mail.gmail.com>
	 <c3755b3a0607240337i26f10249i78cb9f29045b9f10@mail.gmail.com>
	 <e0f125db0607240402w28067574n28f7aab45f07e833@mail.gmail.com>
	 <44C4E656.4050101@pobox.com>
	 <e0f125db0607250208i26bcbd61h331410fc06f6ddc6@mail.gmail.com>
	 <e0f125db0609050325v75420bb0i7b3cdb0bcbf087d8@mail.gmail.com>

------=_Part_13526_23772423.1157599882527
Content-Type: text/plain; charset=WINDOWS-1252; format=flowed
Content-Transfer-Encoding: quoted-printable
Content-Disposition: inline

Pavel,
 In general I like the idea of cleaning up this code.  Maybe the best thing
to do is post some patches so that we have examples to discuss.
  Weldon

On 9/5/06, Pavel Pervov <pmcfirst@gmail.com> wrote:
>
> It's been long time this discussion stopped.
> This may mean three things:
> - first, everyone agrees this should be done and I'm ok to provide
> consecutive patches;
> - second, noone clearly understand the purpose of what is suggested to do=
;
> if this is the case, do not hesitate to ask (again?);
> - third, noone is really interested in making source code of DRLVM more
> readable and more understandable, and I should drop this activity.
>
> Meanwhile, I'd like to open jira and start posting patches there.
>
> Regards,
>    Pavel.
>
> On 7/25/06, Pavel Pervov <pmcfirst@gmail.com> wrote:
> >
> >  Geir,
> >
> > well, it is the argument at least for me to start thinking in this
> > direction and initiate this discussion.
> >
> > And there are places in VM core code where only definition of members o=
f
> a
> > class is required, but whole Class.h is included anyway. This is also
> > about localizing potential development in separate functional groups to
> > reduce recompilation when working intensively with these files.
> >
> > Hope, I answered, what you were asking about. :)
> >
> > Regards,
> >      Pavel.
> >
> > On 7/24/06, Geir Magnusson Jr <geir@pobox.com> wrote:
> > >
> > >
> > >
> > > Pavel Pervov wrote:
> > > > On 7/24/06, Alexey Petrenko < alexey.a.petrenko@gmail.com> wrote:
> > > >>
> > > >> 2006/7/24, Pavel Pervov <pmcfirst@gmail.com>:
> > >
> > > >> > First thing I would like to do is to split the file into a group
> of
> > >
> > > >> files,
> > > >> > each of which would contain only one entity (and some closely
> > > related
> > > >> > entities, if any). This would produce the following headers:
> > > >> > 1)       Class.h =96 constant pool and class
> > > >> > 2)       vtable.h =96 vtable
> > > >> > 3)       class_member.h =96 field and method entities descriptor=
s,
> > > >> exception
> > > >> > handler descriptor
> > > >> > 4)       cci.h =96 code chunk entity (part of compiled method co=
de)
> > > >>
> > > >> Will these header files be useful separately?
> > > >
> > > >
> > > > Yes, sure, they will be. This is one of the arguments for doing so.
> > > >
> > >
> > >
> > > To whom?
> > >
> > > geir
> > >
> > >
> > > ---------------------------------------------------------------------
> > > Terms of use : http://incubator.apache.org/harmony/mailing.html
> > > To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> > > For additional commands, e-mail: harmony-dev-help@incubator.apache.or=
g
> > >
> > >
> >
>
>


--=20
Weldon Washburn
Intel Middleware Products Division

------=_Part_13526_23772423.1157599882527--