Mailing-List: contact rat-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: rat-dev@incubator.apache.org
Received-SPF: pass (athena.apache.org: domain of alexei.fedotov@gmail.com
 designates 209.85.220.224 as permitted sender)
DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=gamma;
        h=mime-version:in-reply-to:references:date:message-id:subject:from:to
         :cc:content-type:content-transfer-encoding;
        b=utZWcf9hEkwEe24xBh1yhvagJFwXTKlZz8ob6zmD0FpZyXWmqFt4Lta+6wp8FEncKz
         Gh23QOPhu7hQkfVb+DoRpk8FEDZMJf1y8ua2SKkJ9OEM9wtch4G4Ky1lMxvlaHrxAUja
         z1KnEtVXAX/ioCnCmVv9CQlTw2Iy/HtRaIJBY=
MIME-Version: 1.0
In-Reply-To: <5b553b550908251320l3c612628lcc0c084faaff5941@mail.gmail.com>
References: <176050.13928.qm@web54406.mail.yahoo.com>
	 <B32847F7-AB99-45AD-9634-43AA3EB14C14@Sun.COM>
	 <c30d70960908250931v42488de0l4ade67c0df4ec755@mail.gmail.com>
	 <5b553b550908251320l3c612628lcc0c084faaff5941@mail.gmail.com>
Date: Wed, 26 Aug 2009 14:41:08 +0400
Message-ID: <c30d70960908260341j1d825ef5x7627a91afff02a62@mail.gmail.com>
Subject: Re: MODERATE for rat-dev@incubator.apache.org
From: Alexei Fedotov <alexei.fedotov@gmail.com>
To: =?ISO-8859-2?Q?Marija_=A9ljivovi=E6?= <maka82@gmail.com>
Cc: rat-dev@incubator.apache.org
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

Marija, thanks!


2009/8/26 Marija =C5=A0ljivovi=C4=87 <maka82@gmail.com>:
> Hi,
>
> Egor and me solved this issue by creating new mailing-list on Google
> Groups and redirecting all commits and code-review mails to this
> address [1].
> We are currently doing some code-review there, so anybody interested
> in can join us with his/hers comments.
>
> I would like to thank all of you for suggestions and advices given to
> me during this open source program.
> I would stay with you to improve apache-rat-pd together :)
>
> [1] http://groups.google.com/group/apache-rat-pd-reviews
>
> Best regards,
> Marija
>
> 2009/8/25 Alexei Fedotov <alexei.fedotov@gmail.com>:
>> Marija,
>> It seems my idea of redirecting commit notifications to the list was
>> not quite good. Could you please revert googlecode notifications to a
>> smaller alias?
>>
>> To address this issue properly I suggest attaching the source 0.1.0
>> release to [1] so RAT committers can consider committing the project
>> to the RAT code base. I believe I cannot do that myself both
>> technically and without proper approval because I'm committer of a
>> different Apache project.
>>
>> BTW, thanks for creating a nice tool! I'm currently trying to apply it
>> to a real project. Hope you will remain with Apache even after GSoC
>> completes.
>>
>> [1] http://issues.apache.org/jira/browse/RAT-45
>>
>> On Wed, Aug 19, 2009 at 8:27 PM, Craig L Russell<Craig.Russell@sun.com> =
wrote:
>>> I'm afraid I can't moderate these messages through any more. Please rem=
ove
>>> rat-dev from the subscribers for these code reviews.
>>> I like the convenience and the community building but some other techni=
que
>>> needs to be used. For example, the person receiving the code review can
>>> forward it to rat-dev. But I don't have the time.
>>> Sorry,
>>> Craig
>>>
>>> Begin forwarded message:
>>>
>>> From: Joe Schaefer <joe_schaefer@yahoo.com>
>>> Date: August 19, 2009 8:53:17 AM PDT
>>> To: Craig L Russell <Craig.Russell@Sun.COM>, infrastructure@apache.org
>>> Subject: Re: Fwd: MODERATE for rat-dev@incubator.apache.org
>>> Google apps aren't compatible with ezmlm, sorry.=C2=A0 That email addre=
ss
>>> should be told to disable whatever they're doing until they set up
>>> and use a remailing address which masks the sender munging Google does.
>>>
>>> From:=C2=A0Craig L Russell <Craig.Russell@Sun.COM>
>>> To:=C2=A0infrastructure@apache.org
>>> Sent:=C2=A0Wednesday, August 19, 2009 11:45:07 AM
>>> Subject:=C2=A0Fwd: MODERATE for=C2=A0rat-dev@incubator.apache.org
>>>
>>> This is the fifteenth moderation request from the same email address. A=
re
>>> they on the permanent "require moderation" list?
>>> Craig
>>>
>>> Begin forwarded message:
>>>
>>> From:=C2=A0rat-dev-reject-1250664693.1349.kjfkfkaaeiggnegpceeg@incubato=
r.apache.org
>>> Date:=C2=A0August 18, 2009 11:51:33 PM PDT
>>> To:=C2=A0Recipient list not shown: ;
>>> Cc:=C2=A0rat-dev-allow-tc.1250664693.agflkbhpleccelchmjin-32KCLShAKAE8t=
5uv9zAv-458v62Fx55x2v.t53=3Dcodesite.bounces.google.com@incubator.apache.or=
g
>>> Subject:=C2=A0MODERATE for=C2=A0rat-dev@incubator.apache.org
>>> Reply-To:=C2=A0rat-dev-accept-1250664693.1349.kjfkfkaaeiggnegpceeg@incu=
bator.apache.org
>>>
>>> To approve:
>>> =C2=A0=C2=A0rat-dev-accept-1250664693.1349.kjfkfkaaeiggnegpceeg@incubat=
or.apache.org
>>> To reject:
>>> =C2=A0=C2=A0rat-dev-reject-1250664693.1349.kjfkfkaaeiggnegpceeg@incubat=
or.apache.org
>>> To give a reason to reject:
>>> %%% Start comment
>>> %%% End comment
>>>
>>>
>>> From:=C2=A0codesite-noreply@google.com
>>> Date:=C2=A0August 18, 2009 11:51:04 PM PDT
>>> To:=C2=A0rat-dev@incubator.apache.org
>>> Subject:=C2=A0Comment on revision r45 in apache-rat-pd
>>>
>>>
>>> egor.pasko commented on revision r45 in project apache-rat-pd.
>>> Details are at=C2=A0http://code.google.com/p/apache-rat-pd/source/detai=
l?r=3D45
>>>
>>> Score: Positive
>>>
>>>
>>> Line-by-line comments:
>>>
>>> File: /trunk/src/main/java/org/apache/rat/pd/core/PlagiarismDetector.ja=
va
>>> (r45)
>>> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D
>>>
>>> Line 102:=C2=A0 pdCommandLine, out);
>>> -----------------------------------------------------------------------=
--------
>>>
>>>>this probably looks better if wrapped after '=3D 'like this:
>>>
>>>>final List<IHeuristicChecker> algorithmsForChecking =3D
>>>
>>>> =C2=A0=C2=A0=C2=A0configureHeuristicCheckers(pdCommandLine, out);
>>>
>>> I totally agree. :)
>>>
>>> thanks for fixing this in some next commit
>>>
>>> I use default Eclipse built-in formatter. If it is better to use any
>>> different formatter, lets do it!
>>>
>>> This was surprising to me. I never really used default formatting in
>>> Eclipse, so I did not know. Here is how to fix:
>>> Window -> Preferences -> Java -> Code Style -> Formatter -> Select a Pr=
ofile
>>> (Eclipse [built-in]) Show ... -> Indentation -> Tab policy -> Spaces on=
ly ->
>>> Apply
>>>
>>> name it something like "Eclipse [spaces]"
>>>
>>> Then use Source -> Format to reformat individual files. Worked for me :=
)
>>>
>>> Do you like that?
>>>
>>> To reformat everything a quick replacement script can be run. A usual
>>> request in such cases is not to intermix formatting changes with other =
work
>>> in a single commit.
>>>
>>> Line 130:=C2=A0 =C2=A0* @param pdCommandLine
>>> -----------------------------------------------------------------------=
--------
>>>
>>> It is nice that you notice that. It is already done.
>>>
>>> thanks for doing this! it makes the code cleaner
>>>
>>> Line 135:=C2=A0 List<ISearchEngine> toret =3D new ArrayList<ISearchEngi=
ne>();
>>> -----------------------------------------------------------------------=
--------
>>>
>>>>Comment by egor.pasko, Aug 05 (5 days ago):
>>>
>>>>hm, 'ret' is more natural to me than 'toret', 'toRet' is comprehensible=
 too
>>>> .. java people > just love >CamelCase
>>>
>>> This is correct. I like CamelCase, too! :)
>>>
>>> hey :) you could give a link on the revision where you fix this. I chec=
ked
>>> the last revision. Sounds to be fixed. Thanks for that!
>>>
>>> Line 154:=C2=A0 PdCommandLine pdCommandLine, PrintStream out) throws IO=
Exception
>>> {
>>> -----------------------------------------------------------------------=
--------
>>>
>>> Actually, if we want to use logging this way, PrintStream can be passed=
 by
>>>
>>> [1]setter method
>>>
>>> [2]through class constructor
>>>
>>> [3]like public static class member
>>>
>>> I prefer first two approach . I think that situation in this class will=
 be
>>> much better if all
>>>
>>> methods are not static anymore. PrintStream will be then just a class
>>> member. main
>>>
>>> function will be begin with:
>>>
>>> PlagiarismDetector pd =3D new PlagiarismDetector();
>>>
>>> .................
>>>
>>> final PrintStream out =3D pd.getProperPrintStream(pdCommandLine);
>>>
>>> I am OK with this approach.
>>>
>>> However, to make code shorter you could make a public setter with a
>>> protected getter in a superclass of all heuristics. And then you do not=
 need
>>> to define this thing in every heuristic. And cleaning up is easier.
>>>
>>> Just a thought.
>>>
>>> Line 336:=C2=A0 new ByteArrayOutputStream());
>>> -----------------------------------------------------------------------=
--------
>>>
>>> This is nice hack. :) Thanks.
>>>
>>> I am happy to help you, and this is not a hack :)
>>>
>>> Respond to these comments
>>> at=C2=A0http://code.google.com/p/apache-rat-pd/source/detail?r=3D45
>>> --
>>> You received this message because you starred this review, or because
>>> your project has directed all notifications to a mailing list that you
>>> subscribe to.
>>> You may adjust your review notification preferences at:
>>> http://code.google.com/hosting/settings
>>>
>>>
>>>
>>>
>>> Craig L Russell
>>> Architect, Sun Java Enterprise System=C2=A0http://db.apache.org/jdo
>>> 408 276-5638=C2=A0mailto:Craig.Russell@sun.com
>>> P.S. A good JDO? O, Gasp!
>>>
>>>
>>>
>>> Craig L Russell
>>>
>>> Architect, Sun Java Enterprise System http://db.apache.org/jdo
>>>
>>> 408 276-5638 mailto:Craig.Russell@sun.com
>>>
>>> P.S. A good JDO? O, Gasp!
>>>
>>
>>
>>
>> --
>> With best regards / =D1=81 =D0=BD=D0=B0=D0=B8=D0=BB=D1=83=D1=87=D1=88=D0=
=B8=D0=BC=D0=B8 =D0=BF=D0=BE=D0=B6=D0=B5=D0=BB=D0=B0=D0=BD=D0=B8=D1=8F=D0=
=BC=D0=B8,
>> Alexei Fedotov / =D0=90=D0=BB=D0=B5=D0=BA=D1=81=D0=B5=D0=B9 =D0=A4=D0=B5=
=D0=B4=D0=BE=D1=82=D0=BE=D0=B2,
>> http://www.telecom-express.ru/
>> http://harmony.apache.org/
>> http://code.google.com/p/openmeetings/
>>
>


--=20
With best regards / =D1=81 =D0=BD=D0=B0=D0=B8=D0=BB=D1=83=D1=87=D1=88=D0=B8=
=D0=BC=D0=B8 =D0=BF=D0=BE=D0=B6=D0=B5=D0=BB=D0=B0=D0=BD=D0=B8=D1=8F=D0=BC=
=D0=B8,
Alexei Fedotov / =D0=90=D0=BB=D0=B5=D0=BA=D1=81=D0=B5=D0=B9 =D0=A4=D0=B5=D0=
=B4=D0=BE=D1=82=D0=BE=D0=B2,
http://www.telecom-express.ru/
http://harmony.apache.org/
http://code.google.com/p/openmeetings/