From harmony-dev-return-18004-apmail-incubator-harmony-dev-archive=incubator.apache.org@incubator.apache.org Thu Nov 02 18:01:58 2006 Return-Path: Delivered-To: apmail-incubator-harmony-dev-archive@www.apache.org Received: (qmail 81856 invoked from network); 2 Nov 2006 18:01:57 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 2 Nov 2006 18:01:57 -0000 Received: (qmail 92577 invoked by uid 500); 2 Nov 2006 18:01:52 -0000 Delivered-To: apmail-incubator-harmony-dev-archive@incubator.apache.org Received: (qmail 92524 invoked by uid 500); 2 Nov 2006 18:01:51 -0000 Mailing-List: contact harmony-dev-help@incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: harmony-dev@incubator.apache.org Delivered-To: mailing list harmony-dev@incubator.apache.org Received: (qmail 92472 invoked by uid 99); 2 Nov 2006 18:01:51 -0000 Received: from herse.apache.org (HELO herse.apache.org) (140.211.11.133) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 02 Nov 2006 10:01:51 -0800 X-ASF-Spam-Status: No, hits=1.4 required=10.0 tests=SPF_HELO_PASS,SPF_NEUTRAL X-Spam-Check-By: apache.org Received-SPF: neutral (herse.apache.org: 216.86.168.178 is neither permitted nor denied by domain of geir@pobox.com) Received: from [216.86.168.178] (HELO mxout-03.mxes.net) (216.86.168.178) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 02 Nov 2006 10:01:35 -0800 Received: from [192.168.1.104] (unknown [67.86.14.213]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by smtp.mxes.net (Postfix) with ESMTP id 24A9C519DA; Thu, 2 Nov 2006 13:01:14 -0500 (EST) Message-ID: <454A3272.3070303@pobox.com> Date: Thu, 02 Nov 2006 13:01:22 -0500 From: "Geir Magnusson Jr." Reply-To: geir@pobox.com User-Agent: Thunderbird 1.5.0.7 (Macintosh/20060909) MIME-Version: 1.0 To: Rana Dasgupta CC: harmony-dev@incubator.apache.org Subject: Re: [drlvm] more self-dependent VM tasks, newbies welcome References: <4549D404.9000504@pobox.com> <4549D82D.7010505@pobox.com> <51d555c70611020650k14e04b5bwaa6f77ccb415828c@mail.gmail.com> <454A0720.7080907@pobox.com> <51d555c70611020705g16038ce0ue7be793129d81a78@mail.gmail.com> In-Reply-To: <51d555c70611020705g16038ce0ue7be793129d81a78@mail.gmail.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-Virus-Checked: Checked by ClamAV on apache.org Sure, so use wiki as a community collaboration tool, and then point to the JIRAs... geir Rana Dasgupta wrote: > > They serve different purposes. The Wiki is just a broad reminder to > even a visitor to Harmony of what remains to be done. I think that the > JIRA is more specific, maybe even more technical. The TODO page as it is > today, with sublinks, also seems well suited to capture this info. But > it's not a big deal. > > > On 11/2/06, *Geir Magnusson Jr.* > wrote: > > > > Rana Dasgupta wrote: > > No problem with them being both on the Wiki and the JIRA, I think. > > Other than getting out of sync.... it just makes sense to use the > "issue tracking system" for... "issue tracking" :) > > geir > > > > > > On 02 Nov 2006 17:57:29 +0600, Egor Pasko < egor.pasko@gmail.com > > wrote: > >> > >> On the 0x215 day of Apache Harmony Geir Magnusson, Jr. wrote: > >> > Alexey Varlamov wrote: > >> > > 2006/11/2, Geir Magnusson Jr. < geir@pobox.com > >: > >> > >> Put them in as JIRAs > >> > > Done: HARMONY-2051, 2052, 2053. > >> > > >> > Thanks - that just makes it easy for people to grab them and get > >> going... > >> > >> maybe, put the list of JIRAs on wiki? > >> > >> > > > >> > >> Alexey Varlamov wrote: > >> > >> > Below is a list of isolated development tasks which do > not require > >> > >> > advanced knowledge of VM and could be a nice start for > newbies to > >> get > >> > >> > acquainted with the code. All items are targeted for > better code > >> > >> > sharing. > >> > >> > > >> > >> > 1) Eliminate duplicate implementation of > j.l.Runtime.Process in > >> kernel > >> > >> > classes of DRLVM [1]. The classlib provides neat > portlib-based > >> > >> > reference implementation [2], which should be reused. These 2 > >> impls > >> > >> > are roughly identical, so one needs to made more scrupulous > >> comparison > >> > >> > and squeeze some features/fixes of [1] which may be > missing in > >> [2], > >> > >> > then employ [2] in j.l.Runtime of DRLVM and drop [1]. > >> > >> > > >> > >> > 2) Improve/re-implement a parser of generic signatures in > DRLVM > >> kernel > >> > >> > classes [3], and move this functionality to classlib > (luni ?), so > >> > >> > other VMs could reuse it for 1.5 support. The current impl is > >> somewhat > >> > >> > messy and half-baked, one need to invent more shaped and > modular > >> API > >> > >> > to the parser. One more possible issue is parser's > dependency on > >> > >> > antlr, which may be considered overkill for this duty. I > think > >> antlr > >> > >> > has its pros, like more illustrative code with clear > >> correlation to > >> > >> > formal grammar [4]; unfortunately this is not the case > with the > >> impl > >> > >> > in question. OTOH minimizing number of dependencies for VM is > >> always > >> > >> > good. > >> > >> > > >> > >> > 3) Classlib's j.u.TimeZone expects "user.timezone" > property value > >> > >> > initialized during VM startup (BTW I did not find explicit > >> statement > >> > >> > in VMI docs for that, only indirect reference in kernel > stub for > >> > >> > j.l.System). I believe this action should be done by hyluni > >> natives > >> > >> > during JNI_OnLoad, no reason to burden VM with it. > Therefore I > >> suggest > >> > >> > to move "port_user_timezone()" function [5] from DRLVM to > classlib > >> > >> > (luni/port), and fix DRLVM & hyluni accordingly. > >> > >> > > >> > >> > [1] > >> > >> > >> > working_vm\vm\vmcore\src\kernel_classes\javasrc\java\lang\Runtime.java > >> > >> > + > >> working_vm\vm\vmcore\src\kernel_classes\native\Runtime_[lnx|win].cpp > >> > >> > [2] > >> > >> > > >> > >> > >> > working_classlib\modules\luni\src\main\java\org\apache\harmony\luni\internal\process\* > > >> > >> > >> > > >> > >> > + > >> working_classlib\modules\luni\src\main\native\luni\shared\process.c > >> > >> > [3] > >> > >> > > >> > >> > >> > working_vm\vm\vmcore\src\kernel_classes\javasrc\org\apache\harmony\lang\reflect\** > >> > >> > >> > > >> > >> > [4] > >> > >> > > >> > >> > >> > http://java.sun.com/docs/books/vmspec/2nd-edition/ClassFileFormat-Java5.pdf > >> > >> > >> > Para 4.4.4 > >> > >> > [5] working_vm\vm\port\src\misc\[win|linux]\timezone.c > >> > >> > > >> > >> > Comments? Should I put this somewhere on the Wiki? > >> > >> > > >> > >> > >> > > > >> > > >> > >> -- > >> Egor Pasko, Intel Managed Runtime Division > >> > >> > > > >