Return-Path: Delivered-To: apmail-harmony-dev-archive@www.apache.org Received: (qmail 23770 invoked from network); 31 Mar 2008 11:30:39 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 31 Mar 2008 11:30:39 -0000 Received: (qmail 46254 invoked by uid 500); 31 Mar 2008 11:30:37 -0000 Delivered-To: apmail-harmony-dev-archive@harmony.apache.org Received: (qmail 46227 invoked by uid 500); 31 Mar 2008 11:30:37 -0000 Mailing-List: contact dev-help@harmony.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@harmony.apache.org Delivered-To: mailing list dev@harmony.apache.org Received: (qmail 46218 invoked by uid 99); 31 Mar 2008 11:30:37 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 31 Mar 2008 04:30:37 -0700 X-ASF-Spam-Status: No, hits=-0.0 required=10.0 tests=SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of alexey.a.petrenko@gmail.com designates 209.85.162.181 as permitted sender) Received: from [209.85.162.181] (HELO el-out-1112.google.com) (209.85.162.181) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 31 Mar 2008 11:29:57 +0000 Received: by el-out-1112.google.com with SMTP id m34so314412ele.13 for ; Mon, 31 Mar 2008 04:30:08 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=beta; h=domainkey-signature:received:received:message-id:date:from:to:subject:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references; bh=oG1hOHmReVOV7YwyiyX5s3ycYv9mJ6BzcH7LsTlk+qM=; b=DXLziXZ+MYPi4l4YmpndoikPwDZNnpf9qVO7DM+xRFEJRgUVT6ZI9E6/1o0jW8ZEJVFnDvDr+gf6KGMW6e1kKm+pAWJ34e6bNTV8RU0Uy5uy3V9g3nfscwzBH+L14uAVEcsl9YosdSykppquVstj8M5Uy+NHn4EDr0QMbGN1NQk= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=beta; h=message-id:date:from:to:subject:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references; b=gbrNCxqf7CviNR043vLaaP+f/riLJu6bxylm+2HpnLt1XXyPWkHEAG4HAi7oziR2q7bH7T71MRnIqm/OZLmOvutWeJ9/bSdtOzq0nO3aVGY4cAQWAdPBSefT62yUrEOxgtCQ/wzuGeYvBKKBHjh9fRO5bYTxa9anaCIbEPhIOhI= Received: by 10.150.205.13 with SMTP id c13mr3266080ybg.135.1206963008883; Mon, 31 Mar 2008 04:30:08 -0700 (PDT) Received: by 10.151.6.15 with HTTP; Mon, 31 Mar 2008 04:30:08 -0700 (PDT) Message-ID: Date: Mon, 31 Mar 2008 15:30:08 +0400 From: "Alexey Petrenko" To: dev@harmony.apache.org Subject: Re: [GSOC2008] Application for harmony-swing-rtf In-Reply-To: <2c9597b90803302224k1f20b006y5c3fa9a9587b98fe@mail.gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <4c4c40460803300528g3a0a5cd6gdd97e43e68689438@mail.gmail.com> <4c4c40460803300813t6c5e60b4nc04f6cc9efca1e33@mail.gmail.com> <2c9597b90803302224k1f20b006y5c3fa9a9587b98fe@mail.gmail.com> X-Virus-Checked: Checked by ClamAV on apache.org And do not forget that we got rtf package contribution. It just uses JavaCC grammar with not acceptable license. So I would say that we need to create JavaCC grammar from scratch. SY, Alexey 2008/3/31, Alexei Zakharov : > Just a few words in addition to that Alexei has already said. > > > Implementation will fully cover functionality of Sun implementation of RTF parser, and, if possible, supersede it adding support of latest versions of RTF specification. > > Sun implementation most likely supports (or is a subset of) RTF v1.7 > or below. So it probably make sense to set this spec. version as an > intermediate goal. Of course it depends on differences between 1.7 and > 1.9 (222 pages of text vs. 283 pages). If differences are minimal then > we may probably avoid this step. > > I also support Alexei's point 5: it would be nice if you can find some > RTFEditorKit demo somewhere. I heard that one of the Sun's standard > demos uses it but I was unable to find anything like this in JDK 1.5 > demos. May be you'll be more lucky. > > As for the rest of your proposal - IMO it looks good. So I'm +1 for submit. > > Regards, > Alexei > > > 2008/3/30, Aleksey Lagoshin : > > Thanks for comments. I've taken them into account. > > > > 2008/3/30, Alexei Fedotov : > > > > > > > > Aleksey, > > > Great job. Here are few comments: > > > > > > 1. > > > > at worst, cover the functionality of Sun's RTF parser, the first goal > > > will be to conduct black-box tests on Sun's implementation of RTFEditorKit. > > > I believe "worst" is not good for application, let's be positive. I > > > suggest we set a goal "to conduct black box testing of Sun > > > implementation, create a grammar under Apache license for one of the > > > popular parser generators, and create RTFEditorKit toolkit > > > implementation based on that grammar. The side goal of exposing such > > > grammar under open source license would help any java program to > > > conduct RTF parsing." > > > > > > 2. > > > > Test cases for all created Java classes and documentation. > > > I believe testing classes is enough. > > > > > > 3. > > > Please, add the parser from Eclipse team to you list. While we may not > > > want the third parser in Harmony, but this consideration might be weak > > > compared to the advantages of that parser advantages. BTW I believe we > > > are using ecj already for hdk. > > > > > > 4. > > > It seems that "" around URLs are not needed to get them work on Apache > > > Wiki. > > > > > > 5. May be replace "mid-term evaluation" with "enabling > > > with RTF editor"? I've heard Sun has some RTF demo. > > > > > > Thank you for your effort! > > > > > > > > > On Sun, Mar 30, 2008 at 3:28 PM, Aleksey Lagoshin > > > wrote: > > > > Here is my application for harmony-swing-rtf project. It would be great > > > if > > > > someone read it and gave advice on how to improve it. > > > > > > > > > > > http://wiki.apache.org/general/AlekseyLagoshin/GSoC2008/harmony-swing-rtf > > > > > > > > Many thanks. :) > > > > -- > > > > Thanks, > > > > Aleksey > > > > > > > > > > > > > > > > > > > -- > > > With best regards, > > > > > > Alexei > > > > > > > > > > > > > -- > > Thanks, > > > > Aleksey > > >