fenfire-dev
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Fenfire-dev] Re: Viikontakainen päivitys


From: Tuomas Lukka
Subject: [Fenfire-dev] Re: Viikontakainen päivitys
Date: Mon, 22 Sep 2003 11:35:18 +0300
User-agent: Mutt/1.5.4i

On Mon, Sep 22, 2003 at 08:56:05AM +0300, Asko Soukka wrote:
> Hei,
> 
> Toimisivatkohan nuo sen viime viikolla tekemäni kolmannen
> patchin jälkeen? Tosin kai tuossa patch-2:ssa on edelleen 
> se konflikti, vaikka patch-3:ssa olisikin korjattu. No,
> kai tiedät, kuinka ne pitäisi mergettää :)

Siis voin tehdä star-mergen uusimpaan patchleveliin.

>    address@hidden
>    sftp://himalia.it.jyu.fi/home/atsoukka/{archives}/address@hidden
>    fenfire--makondo--0.1--patch-1
>    fenfire--makondo--0.1--patch-2
>    fenfire--makondo--0.1--patch-3

Merge meni puhtaasti mutta haluaisin vielä pari korjausta:

+++ mod/org/fenfire/functional/Functional.java
+                   if (dbg) p("DISMATCH: "+givenParamType+", "+ptypes[j]);

"NO MATCH" on parempi kuin epäsana ;)

+++ mod/org/fenfire/functional/FunctionalTest.java
+    /** Helper class to test mapping parameters
+     * <pre>
+     * integer -> float
+     * double -> float
+     * </pre>
+     */
+    static public class floatParamClass implements Function {
+       public floatParamClass(float f) { }
+       public Object f(Object o) {  return o; }
+    }


1) javadoc: huom. eka lause pitää päättää pisteellä, muuten tulee outoja
käännetyssä versiossa (suosittelen tarkistamaan)

2) kannattaa sanoa, mikä luokan pointti on eikä mitä mäppäyksiä testataan:

"A Function that takes a float as a constructor parameter". Heti selvää, 
mikä tämä on. Sitten "Useful for testing that parameters to constructors
are mapped correctly ..."

3) luokan nimi: aina isolla alkukirjaimella. Jos haluat korostaa, että
parametri on float eikä Float-olio, niin sano esim. PrimitiveFloatParamClass

Tai ConstructorParam_float voisi olla hyvä

        Tuomas




reply via email to

[Prev in Thread] Current Thread [Next in Thread]