[Top][All Lists]
[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
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- [Fenfire-dev] Re: Viikontakainen päivitys,
Tuomas Lukka <=