From ce92d363f43815a8dd59cb88f8ae0eff61c50261 Mon Sep 17 00:00:00 2001 From: "Christian P. MOMON" Date: Thu, 25 Feb 2010 16:19:28 +0100 Subject: [PATCH] Replace recursive concatenations StringBuffer by append in a single StringBuffer. Goal is performance. --- .../classes/fr/devinsy/xid/DomPresenter.class | Bin 2550 -> 2549 bytes .../fr/devinsy/xid/FilePresenter.class | Bin 3380 -> 3248 bytes build/classes/fr/devinsy/xid/Presenter.class | Bin 18117 -> 18483 bytes .../fr/devinsy/xid/StringPresenter.class | Bin 3156 -> 3263 bytes demo/XidDemo.java | 39 ++++--- src/fr/devinsy/xid/DomPresenter.java | 8 +- src/fr/devinsy/xid/FilePresenter.java | 30 +----- src/fr/devinsy/xid/Presenter.java | 97 +++++++++--------- src/fr/devinsy/xid/StringPresenter.java | 4 +- 9 files changed, 84 insertions(+), 94 deletions(-) diff --git a/build/classes/fr/devinsy/xid/DomPresenter.class b/build/classes/fr/devinsy/xid/DomPresenter.class index 0a5a7e09cde3268d51eb0d4a04b039f3c3052a7c..b1eb8b03e4110fa3878a8141fc35b26d7b64bb5b 100644 GIT binary patch delta 362 zcmYL@J4*vW6otRpoi&>^N^~JaSTSk}AMt?-L9r7Hi$HvBjBgQ3iI~Phv{cY;{)4rm zpeXnQ1nvD1HUTZgyGg()<{r-d&dmAozkToR_xTl=VDHW=>6RFaij?_qQocu#jEBui zf;_8=!hm^q(x##(b*Ko;nH`u3AWf)#EypaT|Bb?GL-V{!hrJt9+9$1USD-dy`u5yElorsv)^Rfo%gxKvrw{v6I`@31 zXp}YYL;iKp$3$*`@>>3ZGYdX-94YD4FuB5J@F!+?x0wxYR#Hy)#kb<~~l&J@?$Zli5nU-+q1k0`#%I?T$Nf%@sw; z^y}1^%Som@rWsSjCo9v{;F8M)W;|w@QzR?%!R4umpmOo-#9Xx!3`Y!wogeBQbg2<| zbg&QtgT3W&^16r5qN0>H19qD^qaE|ZPMJ05kX=?}rO29}PSw7z$e48}6Voi4TKtsj zqUaC9!xwr+Ju|avk*g2AjC}-!|V9uX;XFJD0W_Ozv{^1M{U~VTZ&1F4u}% z!ikZRSSD#!-O|zq3%p~IgMytW5o*vyx8$%o!427$!==DYEJpXL4fg%T95%T{ljyea zmfPGBFCDaVS1^aud)1OCrm=rh#L+0tJv1#3wEUO8ww&ztSushkbEIA>ss^jWerVT< z?CuwNSQb)U);x;r9*MA4j>#efwIV}GI8DOnmOhkxY*xI^Tz%xYFOsF`1OH1JA-$C0 o*XFC&)f3+MZ#;HjfILs+{$`OsDp^T(3Jso`*5<5!E}g~NAI@@REC2ui delta 693 zcmYk3&1(};6vfY*$;?Y8v9V2P%}_N-{Roq(!MYMbP`lD33qf#EL2Xj0HchQFx_R&3%YDCl&V9|qmTi6e{bd`#7@mHz zE*gCnF5p@OS#Y%a0LE^5e7>SsGS*o6&E{$0bIPM(}R8r1~hFX)SMfmd% z8P`G$1YlCpJM}B{YRK8b-fSCJ3vQD&9fLg2BCc9ip}v6{hL}$v|3iJlVeI6;56y zv4K83qu1vc#!KI$LpEu+Np(P&Y-gO-1RJ`B1fM35qG=7Db(VYsLRVF z?)W*!+{3b33#W?pf-73M-_@h}JLXG5UbE>f_p|A1Ncy#Y9Pl+ndxJyFSGar4-^IiA zNGe?~j%G>h{@VY<5A69!VpE-pj2!g-=I?8L=Lux!Uikal&s$J|;k58by@(9U$ILW5 F`2*i#Y=r;- diff --git a/build/classes/fr/devinsy/xid/Presenter.class b/build/classes/fr/devinsy/xid/Presenter.class index 5108013373be0acb90ed675219737687479e0abc..ff98787a9b02f25ceb24032ef24b7714c1f23860 100644 GIT binary patch literal 18483 zcmcJ034B!5_5V5d&7McdLV!WQfDuC$!lDrp79j)}&4MHlmLg6v0~t+bFqsIkiyLmW z+Sa0#YOAO<#oE@Y4M}mKRl((FYd5M^tzE3$TWxI>f&9PczBiM}BysuwK0iLp%zbw~ z_uO;7XSpxWp7{P_L^M}@%}t7F;?|C`Q263Vw6nWxXCzd%x+B~fj>f|sZgMbXT@buD zSk@MdZY!&Az98HZXBr!*TVB0>OGEYQrE8Wp)~{jWDyA`2v1n&J7>%zBwsnOKEU<;i z)fU^fE!@H6uZeYRD+_K9wzP)JP?{ zX~HP$nlz5`n7rGw+B-dA2KE3pJrWGH9|%lPD9!##FmupgA?0WB2x)| zOLvn_r!$}eSRpyN5L9T%PqJ&UJsgt$wZZL`9y$}Gw}_Z!&;qJ-Qw5kXWagCaCM~2z zOeR*SlWM3#yQX~b&@86eBAZ-N&Oj(%V$xZ()SdyA_TI;V#9>oFE~Dt8#W-!JkjI>n z2Ka&rrfP8>Z5tRqCJOqKT*7lhJlJv}IHI-0O=l0m9BWfhvG#Dsw!vXoq@ghVxRaB1 z4wJf+rkNiw%uRKGsx)039P6W6S}mG;F4NQxT7Kp_>DpjYBdrBQK^r@|8^RFs>;${J z;*qv8RN-uzXoH*9gAw{15)CG8q)iwb(YO}_gpMCF*22SB2PW7|TikRW4rXwIM5{^X zQ&5gGyc4SrJ!}ikA$QO%OZWBKIEjL4AvlAY36i~4CU8OS&IxzRu3Als&;?LRRHaFv zJ!7gw*;O)En@R05mj?s2^vP^7leW`Em~DF(W}BZ1EcVb8X&Zfh4?!ZFo$>BAK!1R` zOuCqMfb8%^UBNaq<&7}$%#Ch>6`7DO{>!=|ZJ}_7%=1Z;K1H7f^j$1At$yo=@7GpB zAMK<|MN%$HTRWko5Rjd8g-KV^E|3}AzC9cT#FwU$ndq9r0ISL-uu`VD+N5hltv&IW z)hOa{ok^di>p^yFu(M8|JWS(8tP>AHiNNQRK0pW1tuxlu(E{}!XDzM8A|X-GU~e#? z>)2dMm!#yNkJ-}I(E+))VFuWpY7e`&Cv0NnNaGg{+p^tFyCF7T7GssMwn?gL#gSn- z4}F=5#Axj?>1O&0)287Z9m!~F%ui=Zs=*vi(ad#Tx}D%ceU<67G{_juXEEsxx)X=B zw5Ddun);>&reSC_+ECbwyWsLfLO!~O?se1G;WZ3fHEG~Y>Y;ByBXHUQ$W_^JF2GI< z8!qlaRjeZnOt=q6J7T+!_R;+&eT%*g`H<;{`(?!dsu9NycjB~ZdcdUb(u0=y)F&yY z0i~UYd4?DgRM$j01v4L}UN?OY6pYkv(thf*UAu5}Tf9}fh5-?a@0*lC2LOnLGdH2) zc6!vL$E346))9(CgKaYPL6d$!KSZa9HnMqzf#Gy$!k;whDLTZI0X_)LjgV`-R)ehd zwD?UBL02cdw8%~zduP&h^@z(NtR6uZEeX))02-ttKx$-$LIpf6w(O@nNJ5b278sl8 zQx)rqLj4(ja|XQTuznWz+^2g4qMmg;gW;niG+F$#-t}Ah`_|aN|fk8K%HVY#9NIKY;Z|=q<5wZ!^su z4nj6JK4ADh`nNdO?=Vdnjw-@9?Z|h+7S_^#O?sE!vs~O{Q<}-PjmPj6fv@jFQxI`h zL}Q^y=k~T>w}*~HSr*FK3}6!|4~-E=oy^`B*w4yjgB|evZ1yJ!1Hc8-7(Ybz(+*&8 z+{fepK6aVx7Kw5vcEw(RRDRiF?29vGUnWqfuxdC@>}_}$k*Zo)=H@IMLmFL!MpD4T z$74Ci^c(Dr)4|5$uS1 z7y+)MdS?Wr8BjEED-yDO7f-x3tm8)XKoidGjmz`buWGEV$)7W8c79{Z6lG=Ab<^{wx5ne!E6U1t?AS4D z$Gll$E*saBSu4t5yEts|>B7NsoEelMEEojnVV-Z&EGqMG1*mQbc82o>lxKm!wYAlC zja%yKmsh)aVVYS^BqTDpk{6r2gwKM#fyG3NHPtJs*HqV4RcpR@cnx+s*r{sy(#EC8p;a{o>g((t zjo|mfX`6J@CJ!S=V{G2M*u(3=(ZP|8)$1FrIR%(5m2=r-@+WvR{9zW)n!WoMn~>NWLM)eQ}Sx|Lf3bq$Sc)~3=4heR!0f&htIOb&^zyV|s) zj>RCVrxAdKG4W`nm` zBFx5)Fm{-T_#&~gKol`Bjwl>#PfOw?He2cAZvLd3FJUSaeB4>Kv%Rfsd$6N3+|ij# z{Mq5*ieL*wq1)t7F(TNEXn03J=Vv9bD}j`M)Cu&WmzsPTUk>}&sblvDyi%keaX^I| zk?R0eSDL&_I^?25U|TdM*~zL{yZC;c(tfqc*GT&*DebmyA#WTP7qJ!Yb|(_Vuo>Na zoyng~ELar_wuL)eFr&15&g2`U#gCb)cWw_xLt5Iq*M#AhhNH+(O0UnG{006ZlpOm` zu+L@;GMXP{`;~6~5)M7h+^yEkuI}iFb*#eDZO9YwO*GlZxA3iQM)W!DL-}g*9!6L& zR_ITeM?4%Y;&P^)INQ`lBy44g+fDu|f6WSQ8jwkE4?^CCYzB!6eK@gbnbiSGeJAWu zr|u09-^q80$-jG0^ip#BBWi%ohCm8#;jf!~FYka&(;|nB^EXW1Cz%EhG7Xwu;l|Y- zzF*TAm6Uy~!^4Ob-RlF(^JmYR?c)ddK{tOFE=L+`m&e+ZR}UsX#1F$v1Vf?Pa6H&3 z7J1Q7ZBCr(2=34@H3s`U+{=`29bbM+EV?zat;@Pl$PXv-;1&5k?&I&fxu0p)@N={V zLtF>PCKKf*9}rt3cC8jU^58bPg5XC@eoVXSVL50YO=rZGllTWrMZ=xp)QvQ@cEonb z)q$q`380_mZhojs6f3`B>H2&*1C)j4r-Ww2q1r+w#0Pzn)Oq-6n6^OukV)(bX8HJ8 z8UI`&3n)-jpBxI0%VK95LcunLnt}F3A00?w8bZy0fDa!5L^n0dY#oz9Y0_dHhTf3f z34>ggm_wCP))=l55YFV8GtExvD?AY_wPqjrsFTFm2c0lYBQYL*Q`k4iRe;N{Qu#dy z{ce60sxc^h!>AW~|7(+9=ih)yTO-lXs&G(WD(FZjJs*;0NpukH;okwd+o4zD4!vRW zANWm462S7r!%0tQ<_HoxG!HW98(k<9{n_Nd@LvIJ)-)5Et`XCciD&R8tyY$hC9s;eP{Jpv_`Hq}P8;{x83a%|H)VL^?X-I-dkL zBh9VTe2@W0O@5z`VOm^&hCAZR!drDx28rlY@^ts%HAC5AZvP|M+VsA`G!M$9x=UWP}l#5hFmS^BFt15y(|{$hG`JYK@6l;o9bL0XU1bSu^r(K z;gCl)KxY>=qbJfV0kzgt>r@k90C*kVDHlm2axmag8_;QCnLY4`+GMIvC`5MN&aP&t zG7P?41O||i9wzd#&K6UhFSIz&75BqqM8K0o$t;q05c;tSRNE@~wnp%dLd*(CMsiGs z;#|`4aw`yHU7x`@Sb<4WLKwG-05UX@cB4B%?rNz zPP@7bDH+qc>Pk$=XXCXFb){Sqn7AEKSDET+bq!7uR=CRwr)O?T_e4g$vBCY2y3SOe zl`Egj`nsA8TUG?tG&IW1k2HPGR5wUdZc0;QQ@y?zlJ?J=>I>37PPfZ-(H6WWv#Ne= z2Q;&z9b#8hju`#Tipz=WKcrtVhvp_tt%qowvhIi++@O6ZP3?za&Dk6>3VC& z7n})1;lAsML}L#v!> z7VW2#28YSoWDhZ|4m5?PqRvYvQ$D7dhOz@K(=`PF&@7a@i;7G7s9_0%Y8*m`?wZ*-(rk{|}s_UU!vBgg?9i-+8 zN0HyrPjG;GY1=^xR5<<4!*p_qU5OegT_}A6y%cTgp-&7p<#hB?d=Je@>3x89Zty$% z=#u^PnbeV&_tI6?_{k}~Q+q_4K-9HNUl|CO2Cj&T)k|gN6~%=Z#;;wuF)s>G<}Y4z!Nw{H=+bw z(I;~QPGM}abQ11X3(=~u!d|=t@_Dq(!+ri2=!>AEiY}p>aH>xF66O(!w%T^1P0nu$ zxxJp_l;@^eH*G#HuN5`p(3ZSM6OV!jxfMSRQx6sTb1+zh-V346xuuV8O&9wv482(I z#1eZiD73`htEG))ur2ob;yrx{=LRg10j-^1tQEjI=!^&HwhBiNHTa!5d$k%=xcsie zl&G&P@w>D(^wQlGZohl5Wt!jp1KPKT#`)a`X6Uy z`>5aV=%q(y-3Qd@1GPf2*a2ZgCqB=iYA!x|jB<}r?(LyvepjBipB`7#Pl&%Achb~d z9?tWiP*9X?D=F=x!#$LfRt*sr_g|>c$*3oTrKHj?s)4kBf9?0=N z>c;0zx}WZbuI;1m&;h)F@EAQvPvZ9f2oC%W)c=DXp||OAdI#Y1K3)eHLr-!RZu=+G z(>#lw=S6goSJDf7HvNoO(=WK0e#srU7T-ZH^Of`p-%P*Zd(if6`aOS_-sJuCCw_+h z!Y|QZ`8E0*{~4FE$LQZGgWgnm^k0NvN7Yn1rb_X9CX?2-w*`gZoZH%ll3+U;QL>DT zl%4?|7{Dcip2e;luo8)q)LV6%lcg~K%&zkw6c8KW1plh(F3ju#_lltl{{!q_2b23e z{Rn(+=jUL+UO?-`{6qQ){S>WtgSS6J$pikZwM+K`DSwVSFRo$htrFJ)5p^ASa|-tO(U_N7j44U{p$-D9*A(;Ovq9>8uqqx7T?od)kv zEXe#{&yy|IcT^&P=`Yi-(g7Ey$f*G?6PR8qp#DmQ!|xE&mf}zGc*wU`O+>$fV>*LDWuV_Iwc2~>F9+#GLG{Cw>vtBF`W=1r*IxQ(QxE^#@6pKr zgx@2`@AZ2Rlb0)eJv7$u?WKR^l=sqqEYKSRG&%|&;VXOx49u}!I?;s5G$mHKSdYNu z%rZHeU^NcYWWV>hd_`{1#%k6z9$_Zj6GJ@l&Io0rkczMLriV<~8CiwRN;H&xb zd>!A0i>SMCN%R2U%#Yww=P)j3UcmLtD|`q4k-v`IWRG(5H;|X>R+IVL>QsJE&Eki& zy4{Z#d7vN(bwkNkH$qnATt+Cq! z542xnx0fy?gNxAWgWq_URRX%cg*}#lw{vkhQ3}NNfxEwlRr@L6;9UB$9&7Ty;paXN z3QX-;uA>P9c%Rv;0eL|ZmNhmaD8T<01vvrC`;JitDl#iF#%1^>zenl>;iZnT40E>* ztx`J$pg2RlOI~qF@z(|E5CVUDs7rb{%_ZeWf$fidWS0~#;-xQPd8j_#jvI^CAEyAn*{)@O%1sDW1GiSYi2e z;?5%a@fG>CiWXU8@M53slD?~FtCiU-j~i1ZoNR#2A<8Q%_IvzZ>8_Quhn^BskpNnf z{pWH$CMe39A=+D%;K7NeqO3}(>7ko+r*oxKE;k4oNH4^%x|qvrEr7UCcWaVvViK$_ z`*}kjZ?rm`tvj4&&2xTY9?kI+O;&e{7k+s9-264lF3QT+M2Y8*RdYEkzj~T&m}byl ze2vZogT+WZjlZ~ghI09Nn!rD%dHhq{H~bvL{(`Fb2-Wh7cv=4?1c9#tJzk^R`E}aM zzr~xdze5c120hAe0!97~kM5sH&c8#y=KsR0dlzrE9i{(5wBLif@jjoxD`Nw6LLrTW zYXz@z@Hk+=1Z3&+fdi+hES{^fd4bB|h0xn-HHmB0NxUBTaK4(#EvkUqfD@Oi8N5rC z@D0F=n^hU#sb&L5=J5B_Jbpx-&QGXveoignpR2R93fu_O3_(u#i745AB1*QOh?3E>`u>-}$0+RYj)3%F6hPip7=rYpssaQydv}HhENS3LN+n$O>A81IQ<0R&iP+ z3xyXV)7Ri^AdaMHF3~cC+`^l-_fl7ywNON@bkKE>B4R5pNi(;Zr3qhwic2j^Ukar{ zM2!aR45UtCbLlf|RJ&=AAO;n$tQ&ck*sTL|$Au2<8p4vtWRfJZm zcB)e`TB|Oi&8mZ1RVT&N#mKmKYkm}2yGruo5}O~`sg?+ZwIZ4`RtCsR>KLsAte=J? zAu_S)j8Z!>p$m+9r;m5mmN{ztgx08QUeTC$%Xf86f)L#yMHAKLM-)$y6pJ0dG5iIOh7 zMj=^*E+E8H`uACW7A=U@=rhQ-k-C-~>Ux~t=O|O%K)LEhoZ%N~in^Jmt6On~x6vH+ zRXS7MLG|h`oZUS*yZh)u^=+Kncd1)FL^rC3>1Opk`m*Y!y{eDyQs1Zj>Hs~a9-)Kk zQF>NAPS2?y(Cg}lh#;S!KdC3_U+QUn=C|sk<2!UC{|_iy3%xAV$;mH68?4g7Wa&?^ zz*e1u;`}`S2nRkLnYthI3)oRHaPcQVN(WGMDgO+3=|uEg$iDz$DrB*8`Ipdm1JR?@ zy@{ropY zO@3E#KmSt^8vivyV~K8rw3hZW-Z}2)_Y7(pbUkK?!=S(eb%e6ii}1Z(q5}0Y%~HRj za`h@LRj<)G>etkuUZ-aD8+{h5EHZU^81%wp6{HoXpamY*a%|U$Ki)cy5X(Teu>lsO z3f%VN@8AZ2uTJD?mmv0%*j!)wTifSWjFatO$2GtrcpQr+e^X}vG(J{poUo$hcuXqN z{sepju&Snq)>%MSm#er`I}x5D)SiV}Nt!u~a;M*Mh)iLmB+r~ixvPgJ`5k%4rXe3D zlN28!)Q#EY;yhQLbML?#De3Au$^{L7Bc=WU1^6cz^bUmmKXkf!pDNXHszy+~5?XY& zVbB`GL2C^cZ8Y#_Xoi;}Mh0DMm~@FThCYqwYmA%}CSGTQn8K^T!Z2KD;gtX}SBffe zp(_qC=A<)px&kxfZ=4OZI`Kzr)06`xmo`nsgcNB3%ny$}CaKcEahfQx|1olME~rc7 zx^amL4$7;QHrGQ~NTBap$#hHp{xaJ})D|6rE7q%=J>)PJ8|983+E|q5=vA(AXP#4^ zOcllncFe$MK0XWanU2q5d`e4lmCy2o@=-)Ihj5efP|UQ7zLGouB_b#X=Q*bA9k{oo zS7oK%5on8HOvCD@kk>ekvW=NE)hMC@ql8L~Qd(`yq4maG+GLywM$f0M#v+Oui|Hbx zigu#k<;F_7+Neok_O&Wo=Xk9gwMgvM*u8aokWF1GN88e=us6A2tbt7G`I^NJ*q0hL z4t#QIX3wS!C^fQEFewn(BSdH6@LUoN6tmCgJ4!CF9M>yWBmr)i$cNb%Cx`$oIS=OC zB7zP9mAex!o_eTGa~dfFEu}qFk>`LcQifx(ZZ0m#g9JlhtvoPv#C0dq2@qoVr+sRC zjbm{SxxmpWdj_6Pk>9gHEE(&`FgB9U*hHDeW*TP%X}r-)exrq^86mKBD-|29RBA+M zp3zQ=j3})#wo`*~5!l=XHiu}7@u?Jc25s3j=scT=K5Ei(s%SiJZvAQ!nCPb!NQDX9 zPNT(YvXr(nPPqx$)zn>zCl^C__YS;?pR7SYWRgg`aUCS;da&UJnrM8U zPDZ)N*sZsgj|B+c$aE;4V9g9{Z6+4app&&MC@ld<7agZe94}`nfEZq2u;~k9xDx}% zijSlUO5sh7kfbv1Ak(-@uV*Ex#%j+K<7lj!q3=o8$6era<07hCd>v*fH zJvCe|n{oP&MXU*p31B`To*zs2p$AmShW%=mM(7?ottijgugVmhf!UC72LxNq#cQ$n znTPmGo!+#|$$7|ksWX`N4*VRDeIR)YR{@~9+wV-?!&&3?MVu<%Ls8KZ+{)qF4YzD+ z!5&(o7qIKZ|4KghszvB#-??E};?`NTI z<;R#KSEUS1Qode0_10x#v$%)GGZ3%m0nb083C0V+k)P0flvfx>U=I<18ZQGsUZIfj z8g&}4(^bZA=?BIi=r!Xn^sezTE8{KBG~VWHUAe{S_TBYQIa~y2t*&1&OXq-An)j;~z;u=IQAf3o2 z@75QPF3zL}tU8{8yF|H^bm^-|fdXr`TFhqk-wl|zdZDDDpo%h15Hk2j+=yL~e-lJ}T^$7ZYUD7;6ns!Ld8{Qac`A(ilh0A0PRz7|$< zO|NQPjNkS6Eq6>&a>M3OcUm?(yJ$+5>wGcJ8++AxsYX;^v^!vm9An7m7)v>hEIP?C zj%GN{C&gzY;sV z`AAWVcrv*(J{hG<6=*@Ce!QcUCOFEd#4(%ZIp)wAj=8kNQJ%8RrAi`x87+7!uQ;5M zj2I>)J^Yd(9==Ix-Jsj-G!x152`CR93sa9 zO?_(DV~_aGDZ>3T?wLLHa0Hu*9#q%j!I5~l9uLmM!;N@wCmz0t2bcA*;URU!1336B zP(E4h#*O|A2-9wO^Yaj6?1ucGMOFA+j`tB(U%C?g+$m2DPxse-7Bjo zxV2VWtRlZwaZ9aeU49NkEX57{xpYz6+G<;Et5(~pb!nBN`G3!SZzhvXt^dzYoSFOX zdhWUBe9v-Up8o9NLqs%FU2T$L8nv;lpdoy2Q)~OSf~`#r1!VCVgOv-*R&S^-TVArFq^4>G6PGd#DUGzYM?|O zFnO9In>K~pm;&XIwoL`0&7u0nZ~-b$KC7TyHx)CTGP$}oSW~)WLruwo^0LVr+a^2Y zOa?iVgAEHq(a_`|2nt1;BCXX?{DwDeV^ZsaOx`*5%}uRM(RoZp_Ow+@uF^6vKpW0NOZG=-)znVZ`p_2G6Ng z{k%Ow${?{F&5+DdgKu{P4qBgSNfdmj>xhQ6oH9)rU^o5QR-8gJXqHJcp%Efc@l%k^ zTXZs=0tLVdiOGfNY+HH~T|+J52I*fB+Fb0VQ?b2z5waATO~ob^f&5-GCv~@I4$Wn< zutKF&Lm65$oB%Sqh166I-=KD8x13Em|#6^FzIVJn4SsZtrne0 zAvwS)h)WbS*cXcH zIyRToB_TQJW7T)GwL$J3m;t^gTf=Re;})@apz(A1ZQ1Fjoe+mF^RUWL$0C(B;>fU^ zmu_Gpv01w*M>o@6lWt*} z+6Phw^Jpx(m2SgfmXw!oSW#75&D0OA1{(@{at9ourUpNC(p@I~06s;(RTFN3ML(pw zp)rt#AmpxKK)LHTT%3y1NLv_~aSx7uz;-|FqaRsxAKef6k?H$;ZMgul0mt@t@wDlB z(4zmLhivnzPg+bUlzJ5A>1Ak8UEb6#n7W_3OgaDx25PtHVLIqIe&N99qQN@p|D*3i@%YL=-Vx9ACa5}lf~5zfpG_NPM=e$=9;=zp0~ zzz3nZ26C_0s+P5$5x43lk(B2I2Y$*lv;Q$T&f1_tDB22tu4yZzraj>od&O&#lNvzP zZ2{89h%HD*VBo;#0S)ukJF3;!N0a?%X5(DqeM%!8tw0xsQ=S5sJglF^_xI}_!PbT_ z==2DDw9ho@cl~W`4{U{>ZfR|B!(H0Dhponvq~=Q-k;@;^nt3SO2g7>C4PdSMZzn;)K6YE|kyA6{yJ9b3 zBCB8?_Qihei&LP;*`0C=ZrqP0BAQsGU_>P_VFO zVbR}T>1yH@d0Acic`3ROHE$TCwfipWAVZ`DlPA*Yi_C!Hpr+&7B7z1 zS8r>FHI@1z4qCjFmm#3BY0yay#|mIueXPFIQ0(Oj*r=X8mz9?#w(R99$V5_GWlgYV z&4v|ai^^7%RhE`%zIYj7-PoQ^r3*`HN|56!tqE3DIz6hv?>Q6K>85pFMsR1WUq8>w zwcu#a$eOa%HTIkWOc%+ythIO@p8<~*iHax;Ga@Vql89fgoRh6AgBpEo5^$=a~Wr^AW;uoyGN{>z-!q>UbH)spf-ZXi(gJhPWgA2*bFh z)vPF~tPaYKOUgGu`mnFwRko-@(Iku)ebWM>8laY=dl}p0^M}BBH#-L}j*4^EUPk0M zaS+Rrtb<_Fgu%NxwEOBLaI0wxgRFQU)f6cRwj#6yU&Eo6KDm)Ng~fi}!rM%SZ;~yb zwzXhuOLM{IP+NPrtv!)tbE2+Ap?aL(HjB6Od9bss;VnU(^ptoh52E>HC(w&tVDW`~ z5sX^94vCxKjUhn?u4b=A3ISAIV)56dLk2noH?>A2e^(l55r3#%+ILudnY5pf)b1z* zlCe>7n;PMzwj=8bpx?rmTYN=)!O}>mIow{48Kvbai?5cJ0A?!Nx;fO^pha}siZH^N za4YhD((9WRU&G&mW@6uQ_Bo6}p76_Tzu4sKaOiz()N;-2vbMHJ+Y&6@j0lRarLlhA z#osmgdJOnNzFK?(-w2r&`jh4nw?xaOoM}7GHo37_xV(#RvUm?8n3Gg+HPX{9A;?>= z%^?2QhZAWnusc8#zYinQu6qM{xAHAwr}y@VUQ)t*z$C0rGC&G;@og60&K)pBTI8^C zzSH7P3E;g5;5EI%jh$Zpp{B7_^5Bs+FYf~dtAh)(PMUs_pYP@SO#Tu4gg&fY7->o5 z^DVxg9{{$88X79X(NK+;(Yd`{ocO5@;KK~yoUZcn{{WEeN0 za#5C_V|>8m{Y=yQpQAk(;@UGdk$$$gOALe`bSX78>wk*i)@j6JPzK=j{>qZce5HgM6t4}OIBye8K5jQKQ1&s;h43!_@GaM zIxioAJqlL!n#7r4x}Tqt@d*FyVboG%X~!1IPWK)@G| z01R82WsZ)?pgz)KABNtL(+Pt-wwObelGf<25)jVBnKPY~)K_>SSZdEc@KGm-a}GLg zPX=N<`X;fjhpPaW-z4+92l`EZ8LH7EeEp~wBmFyzf6spalQuTBHY^E;bgo{9C%xiJ zcvO8Xi8xt6gx7%FEzm3RVg6+CpZRqOj$wJC;e`7#Z2$@Foj&MsXwH&}{$lZ4{5F8i zo+uGz3>G#elPaB&iI~lQv-t0NXFx~s74`iA0!Dt<;`by;X-NZip*?CZzYk=AHj4p~ zULRWg5&sjLfgUbuYHN?`Gz@%$KE9adgADlC;!haSnuR;Xa9eaic%x26AitYTp6=ei zrZ+ouCxODmd3c4#QVk+Q*dg+cXqS>ydKJP}m-H6%+|i;*g#|;L*d*~(9ln&?QXXYO zvg;c|Z4&Lcki;qVDj!Nynxn;Dg^0^FWs{T;TDhl$S~eqs-9tMu4ZIPfJ&Lr30iO!m zSC0fzDmddDTn#d=kS3LBsSy(FromR~P@^VNEp`~g-Hyw`Yr!I&8|jlOaWW}x-6jN& zv($Js0UCyLDs^afX9r95eP>kXc+c;{+%Fre_CbJzfZ-NJ^u@mUb)vXIBT(7!2^KA* za_~ZuN+4JiOM(NwwduvVwMmxS6IAzOZx zLu2KN?r~EgM(lO@C;LfJ)9DTQnlC*g(z6RCb&915%krwzA$D^o!R=5B;C84|3DE{j57=IYSCwJh zB&?~g(R8}VPRrnM>?Ma|)b+d(;|BtySwpojU5E$}sqHzZOIyx1Y$%I_oXH?9ftMpfL2p|9gafJp;bnzJL(%3HHDiSv=d{J1^2O9 zUuX#{L0`ff*1jmX8El0|t@B5+@nQXLEKGGJ&Rnk%zaa&LBRp!`MiXId*oj&PNj{3)4(Q*@c!#nu4XM5z4(exd&)+ zMc#cVClio`-JGBa$g=xUHcdkxjG!FK#XKJL&%;g>RzT7THL|mx;n{`X+?*~tiK!y* zD0#WOlg8$y7k1HUiYoJ+8h?Hl0nf^FWobk2eG(=oOlD#_$!Dk0DInogY-Bdbo(lpN zXp-h(aV&wSQv43V=sa2o0wyF%(=>5XXVPRUqeXZQV7|o|VbBtcnnFf5Dhw0rn3$iA z(=e2~F}2j06x6lY9ITqXIlfCDGSO2x*h6kbCmqg3AqR#12dQEWPNb?LH%2SE2=5_P zcG8X5BCYd!9wH8EO!E{r1SRcAebAAC>K~#yOTaF#C z08eWuomSHryoYcioq@yu8qLDn1f{siUqTJ|w-I;jjdVLT(~r@9gxd7Utk5U;6n%rf ziTS`Ox&|c?7CEDD;uMA^N@M6Ar|v}Bi*>$*mRY#EzLu^79i_CLzKv6L(|0hBh_v0d z3vF_K;6jf|EQB)n(VrMlNTk`0E3GB0sj#?E#P-Je&nlc=X;%W{eQCD(8O{_ zzv6OJ^m2jE$L? zTDlWDje9=28!zTOKppfTT}ThnCHVY+4xqM+?x%zFIQ07n9inFeO@E{(=_Ay4(^IVI z8FtgNoPw+CarAS}qUSk}UgQPz8(xX}4fG1v(I2>xUgPcbC%%ea=Wo#)yodh6_alAt z1loQ|@9{6_1O7FA#Bb5R_!9(7pV24EPybd~@Ns9+2Wl=G>U8#~MeJ2;@LSKeU3_0~ zONHa=iIT0JQuj0928^&;uMLy|>uJ!>={6|l%tT3`z^)4?N-g}cQx0iMmhXb*hxI?0$lKoJ%Kk<6u47Ms;d4wI^=CMxydK z1x$KnR5v!;tB*-HYlHSF6`8c3KLvT5uhP{gbo-$jAmLA*+#V=n1VWBe8KyZ# z=-drU;T@zWMC1#m!5aXx)BnHc$rO7%C=o#D-_sxZA~cMbTO(m=PHtYl*nwAzTmhG; zQj!P6CsGP$YfEsl-8(L7@Us4d!vZr<`ZGjBT(ndzeg=?V3|&6BuiqEwyS8f<=pBNE zW;@oQ$PMIvU64CZ<9?C(0F4mqp8gjx?VcFfCjISvyFEtl9Hv(U{*O{dz@3vHa2=p` zH6HUT0k1X@M+06l5x#)$CP1RCB|umexop@lU@t>GE-ulp*Qec)UGfPJ+x%Arq1puxyIxRbdX+ z^9%cSq;O_Rrhjku_)fAQ3K^Ug<4L-3VvODnxO)))w^Rr`n1>@i9a(XB#<*rZ30JD~ zIYRyz4sToc@mT>20AXSlU^iSI#j^j>}swZ}Q8hVp(jlCMyc`Nt}s zA613?m{z-ecwYsI5?4Ev9JNEqQ9G0b_2JT?B&hEqpE^k^pqqxNY@Uib4+UUNvQaX@ zn`0Woy;KjoCK&Fcv&rCTX!XP6ywfgm0d}4Pto3mry#u@U60mtTELT2CDX?#Q^jM3J z!$W=n6j<6PT}30i;W+y=D9-K;jR+_3KO8X)(EE;43Mx{IQic!pkNt$yXEesds~DJY zl}Y=gb{xQQiu#y*;>_X={Jv0vsHB%O`*c5N7WUN(`zlf7mvLtCI$GZ~j3>Z7IJ5k| zmcZ$dzzjaAGJ|Ipxq1NX2tVzBRtBF8)QaPXgIqZWxKJF^Bb1IW{(ivYK+i68griw# zW}4!pUJg11RDJmDu%B6Ie;1#l&U3NTZ-%DlG|-cCn2Wj~ha{MIgr)?%U3@y8d{QX0 zy}x`NKcxVD99^{sEn({MX4#{sESD*D=qmHYeRROj5gGxt4R`}S8KZT!lb+X_90y=} zTu)J{^)zjYM#Em9npP}zowP^yTdwrmNsSE~=jfiRbWdS8NJFRny!rsI zv3peM9_yXCHYDbPO_8!az~;>u#0(tzzSxa4pbD_sEm4F(e+D;5LC>{O&((u7f27mE z^_QUwuh0V&i|sj z;MY9>y*miueggj8v&v1cDwF=C@G_c8r4Q9m>Q=+prC?3eaQ3N8PEjM+0xG1TZloH+ zC#dl}1qe}~rtqn1DwjgHm#Jx7iJWY$n!#tPncS>qafdpEx2j^kM9twF;5*)gxv=qD<1MgJ~z8ieFgU>>p2OPMP&jwPM@U!Q0GceT)4|_DXqSXhVdKgDg^3#X# zM+MWoaF?cPO!M(!==uw2^=d>}NLDwkBr^$1Y#`YK^bjm50#kACRdS3*d@6sRBB+xB zJefLe;&EDlP9I|?FW%z&Li~8QAAYDh_|V}mi662?Km6cKa7fZY7i!7c49SW|#rbJ- zLS}6N*U|-c#awTJ#ih~F}_)6 zgWk^1iSe!F((#VnGz`6@_yaHoZyDrf@Ld^v_W{1AlP;{Vhd+qnSmhzx_pHPK%>OW# zAEZfZ^7HodkALEHJZyJ->ObB{9>1S1DrpcP~G#f1l>3&?1@93z6R= zbql%FZ8*Q%DOKG;8R|});axON-9wYqeK^AhX@+`;PF2XosfTH$dW6=i$LMTz1h*ys zOWV{lbd7qJu2;{|4eF<~SN)uBRnOCY^#UDIFVbQ4OL|)Unx0X=0Y?3n-c&EsTj~|W zp|9#QzfmVSKcZ{+X;8EhDbhS-B^hzH-zgnVl-`2fwp(2kFep@WvIWATfIYR>OFXD|Deh0eVVR5phEQ#m8gHxGW9R2hC@_`m{t6O zB{rFe(Cx%@y+&X0AnJkGk^ykz@1p*V0L?%~q#8h33T_ARCuW}lbKS@b&xbXV$X;Lk z+uH}eD?RU(Tn;>fdop+I-I$Utz{k$T6ZG8nQbs5I2(YTWlUCVa2LGtrb~(H`s68FE zlAA-i3J%B-vV@V6qjMXDo=zGaaAkUQ+D#_OJwm7(vctofo=o@N?$?vjE`3odr- zL`BXKWNTx}=p>gh&nR?t(%PI%S4_DI-I;EEGNl+N*f9kkcq;f)NrIjc&BG@@FGFb; zr<1Z!^ox_zNtq~WqtQwJyi5Qk;w~3wy2kD8zB4bTQj_l_w8b!HWA$P%eLf8{N@%>X z0L))Vc}5v6H-fa*L|Vpj+Gwn#R$~>NW2~dC=y!>+fp!`VNzA?i{!Ttvo0ka*eoX5>Bo{j3xdl!oittSjsNvknKsm?Qr?-}c1qj|q78Fmg zW(u|@mxd`cLCb>D5`bjWr<980g`W%%!%GYfeUj3YT>nt=f5f?Tum_4ONr%y^WMk)b}0o^0(zO~i;gn0(?vIq8@y5I(^r64ET-n@>q;T~_FjbdP&A;G?W(^@4z0u2V3|)@Z&>jF#bvH#>aG-af}`{KBM2e4Eor3 zpOwqUsV+Ycb6GsjmCA*#G~VtS&JUpO5m!LV)@GdwITROLJ``6HKIAz3UO2C|!#|Ag zQH#`Ki0*LM$|V|aCev^gR7)X!D{=KH7oKivpk4aH(}Qb``|LVgr(AE>>8nqH0(-V) zn9c6L3ovi@LPX|}6SbJ&KVVg$fqAQjC9 zH^o5u;nZ(HNx_0lQ)gWIIxGOZI&p>2gqo_Q;8x@Nkj}I6CQqEFAv`$wJPDZr+Fcehip)z z=P--VQu~04tT{ri`)UuUa}GV?UzUUWVcY|ItKdP^<~*P}@ZgF+Y{i2+{%{^1%=p8F zc<|T{YaUdQ`@pd@5I+`y@i;Jg8h&$dC0~f&Q|WXn#qUD=jo)(o+TmLA8z}NK;%N<29IbIshCrZhoYl*RRvUGWu zZN@IS^f>tdAqomJ*4uX7)sF@^N>S0LN3F0+L*MCCl z@Uay}^p>wDI@WE5&9Bs{z6JKU?6A4aWZ&`kVqwrA_f10jw9{HnGDC_vGR%|Z6**p$ zXMqBXjId6T&rGn%6kANQE$kgC1jQkR;|@d%ha|e$DPbvI$_r8M;$fqfeXnIZ(*KK| zmELrjItc^p7jrk8VUT+wHsr|$?n}?)ftdH>J7ZUA#I#Yi&}_P(%04{14JnTNhsNQg z`HOntRq_1;Y~pgn&nG`zmQnOo|3URg7yX^8qTl%st0(%4zq`ZTqrNt&PsVsETN-iDv4J65_7sE&}=`K%EOC4Ir@7y delta 554 zcmYL_&r4KM6vsdJ{q$X5OhpcFq^OVyF=0@mNOB>xaFNnVs#Vs!$v5g$GdkmxHqfg5 zZRtX3(I!GE8YwYa6^YQIUHua++C}4;tO+jne9!rw?>XOl-)Fu?`t$F%JHRlFO}*y~ zYNomEb4w8w%DH@8O-hw|ZYjya0v_??~xxdE;KSJ*8%Tcr~^BD)C1xt$yO*ij_UH5fp=g z*JRFaadXm~*JIgu#K|}x=~5HVIrNU?dd6uoJ2AK+{Wizso}$OB>b^#S3NzFwvdkP$ znCB@a*0{?H?z71vO<}jG^MMt1c}$O}3+VP{PI5|O8=Vuik}jL9%&>UbK|39^EA<2R zlGI9cVK=zO(E}((Zphksp_*dxp3sh1fJuT2ev1glNBNZTxd<(eVErxpN&d45 bj|akai#RJ$NpuI^@X)M=$Nd$_EFSFxyFF=1 diff --git a/demo/XidDemo.java b/demo/XidDemo.java index 43f795c..ab72782 100644 --- a/demo/XidDemo.java +++ b/demo/XidDemo.java @@ -44,22 +44,24 @@ class XidDemo datas = new Data (); datas.setContent ("name", "Superman"); - StringBuffer html; + String htmlSource = "
a name
"; + StringBuffer htmlTarget; try { - html = StringPresenter.doXid ("
a name
", datas); + htmlTarget = StringPresenter.doXid (htmlSource, datas); } catch (Exception exception) { System.out.println(exception.getMessage()); - html = null; + htmlTarget = null; } System.out.println ("datas = new Data ();"); System.out.println ("datas.setContent (\"name\", \"Superman\");"); System.out.println ("+"); System.out.println ("
a name
"); System.out.println ("=>"); - System.out.println (html); + System.out.println (htmlTarget); + System.out.println ("[" + htmlSource.length() + "] => [" + htmlTarget.length() + "]"); } @@ -73,15 +75,16 @@ class XidDemo datas.appendAttribute ("lastname", "style", "foreground: red;"); datas.setAttribute ("lastname", "class", "nameClass"); - StringBuffer html; + String htmlSource = "
a last name
"; + StringBuffer htmlTarget; try { - html = StringPresenter.doXid ("
a last name
", datas); + htmlTarget = StringPresenter.doXid (htmlSource, datas); } catch (Exception exception) { System.out.println(exception.getMessage()); - html = null; + htmlTarget = null; } System.out.println ("datas = new Data ();"); @@ -92,7 +95,8 @@ class XidDemo System.out.println ("+"); System.out.println ("
a last name
"); System.out.println ("=>"); - System.out.println (html); + System.out.println (htmlTarget); + System.out.println ("[" + htmlSource.length() + "] => [" + htmlTarget.length() + "]"); } @@ -108,15 +112,16 @@ class XidDemo datas.setContent ("words", 4, "echo"); datas.setContent ("words", 5, "fox"); - StringBuffer html; + String htmlSource = "
    \n
  • a word
    • \n
"; + StringBuffer htmlTarget; try { - html = StringPresenter.doXid ("
    \n
  • a word
    • \n
", datas); + htmlTarget = StringPresenter.doXid (htmlSource, datas); } catch (Exception exception) { System.out.println(exception.getMessage()); - html = null; + htmlTarget = null; } System.out.println ("datas = new Data ();"); @@ -131,7 +136,8 @@ class XidDemo System.out.println ("
  • a word
    • "); System.out.println (""); System.out.println ("=>"); - System.out.println (html); + System.out.println (htmlTarget); + System.out.println ("[" + htmlSource.length() + "] => [" + htmlTarget.length() + "]"); } @@ -154,15 +160,15 @@ class XidDemo source.append (""); String htmlSource = source.toString (); - StringBuffer html; + StringBuffer htmlTarget; try { - html = StringPresenter.doXid (htmlSource, datas); + htmlTarget = StringPresenter.doXid (htmlSource, datas); } catch (Exception exception) { System.out.println(exception.getMessage()); - html = null; + htmlTarget = null; } System.out.println ("datas = new Data ();"); @@ -176,7 +182,8 @@ class XidDemo System.out.println ("+"); System.out.println (htmlSource); System.out.println ("=>"); - System.out.println (html); + System.out.println (htmlTarget); + System.out.println ("[" + htmlSource.length() + "] => [" + htmlTarget.length() + "]"); } diff --git a/src/fr/devinsy/xid/DomPresenter.java b/src/fr/devinsy/xid/DomPresenter.java index cd678a4..a79d0da 100644 --- a/src/fr/devinsy/xid/DomPresenter.java +++ b/src/fr/devinsy/xid/DomPresenter.java @@ -116,7 +116,8 @@ public class DomPresenter extends Presenter else { // Build the web page. - result = Presenter.doXid (this.doc, datas); + result = new StringBuffer (); + Presenter.doXid (result, this.doc, datas); } // @@ -127,11 +128,12 @@ public class DomPresenter extends Presenter /* * Xid a file with data. */ - static public StringBuffer doXid (Document doc, IdsDataById datas, String webappPath) throws Exception + static public StringBuffer doXid (Document doc, IdsDataById datas) throws Exception { StringBuffer result; - result = Presenter.process (doc, datas, webappPath); + result = new StringBuffer(); + Presenter.process (result, doc, datas); // return (result); diff --git a/src/fr/devinsy/xid/FilePresenter.java b/src/fr/devinsy/xid/FilePresenter.java index a3e4eda..9bf879b 100644 --- a/src/fr/devinsy/xid/FilePresenter.java +++ b/src/fr/devinsy/xid/FilePresenter.java @@ -113,7 +113,8 @@ public class FilePresenter extends DomPresenter } // Build the web page. - result = Presenter.doXid (doc, datas); + result = new StringBuffer (Presenter.estimatedTargetLength(source.length())); + Presenter.doXid (result, doc, datas); // return (result); @@ -123,32 +124,7 @@ public class FilePresenter extends DomPresenter /* * Xid a file without data. */ - static public StringBuffer doXid (String fileName) throws Exception - { - StringBuffer result; - - Document doc = Presenter.fileToTree (fileName); - - if (doc == null) - { - result = null; - } - else - { - Presenter.addMetaTag (doc, "generator", "XID 0.0"); - - result = Presenter.doXid (doc, null); - } - - // - return (result); - } - - - /* - * Xid a file without data. - */ - static public StringBuffer doXid2 (String filePathname) throws Exception + static public StringBuffer doXid (String filePathname) throws Exception { StringBuffer result; diff --git a/src/fr/devinsy/xid/Presenter.java b/src/fr/devinsy/xid/Presenter.java index c12bf52..19c24e6 100644 --- a/src/fr/devinsy/xid/Presenter.java +++ b/src/fr/devinsy/xid/Presenter.java @@ -25,14 +25,9 @@ public class Presenter /* * Xid a file with data. */ - static public StringBuffer doXid (Document doc, IdsDataById datas) throws Exception + static public void doXid (StringBuffer result, Document doc, IdsDataById datas) throws Exception { - StringBuffer result; - - result = Presenter.process (doc, datas); - - // - return (result); + Presenter.process (result, doc, datas); } @@ -117,27 +112,20 @@ public class Presenter /* * */ - static protected StringBuffer processChildren (Node node, IdsDataById datas) throws Exception + static protected void processChildren (StringBuffer result, Node node, IdsDataById datas) throws Exception { - StringBuffer result; - - result = processChildren (node, datas, ""); - - // - return (result); + processChildren (result, node, datas, ""); } /* * */ - static protected StringBuffer processChildren (Node node, + static protected void processChildren (StringBuffer result, + Node node, IdsDataById datas, String suffix) throws Exception { - StringBuffer result; - result = new StringBuffer (); - // Get the iteration strategy. IdData.IterationStrategy strategy; @@ -189,12 +177,12 @@ public class Presenter lineCounter += 1; if (lineCounter == 1) { - result.append (process (children.item (childIndex), datas, suffix)); + process (result, children.item (childIndex), datas, suffix); } } else { - result.append (process (children.item (childIndex), datas, suffix)); + process (result, children.item (childIndex), datas, suffix); } } break; @@ -209,12 +197,12 @@ public class Presenter if ((lineCounter == 1) || (lineCounter == 2)) { - result.append (process (children.item (childIndex), datas, suffix)); + process (result, children.item (childIndex), datas, suffix); } } else { - result.append (process (children.item (childIndex), datas, suffix)); + process (result, children.item (childIndex), datas, suffix); } } break; @@ -229,12 +217,12 @@ public class Presenter if ((attrs2 != null) && (attrs2.getNamedItem ("id") != null)) { - result.append (process (children.item (childIndex), datas, suffix)); + process (result, children.item (childIndex), datas, suffix); } } else { - result.append (process (children.item (childIndex), datas, suffix)); + process (result, children.item (childIndex), datas, suffix); } } break; @@ -248,12 +236,12 @@ public class Presenter if ((attrs2 == null) || (attrs2.getNamedItem ("id") == null)) { - result.append (process (children.item(childIndex), datas, suffix)); + process (result, children.item(childIndex), datas, suffix); } } else { - result.append (process (children.item (childIndex), datas, suffix)); + process (result, children.item (childIndex), datas, suffix); } } break; @@ -261,13 +249,10 @@ public class Presenter case ALL_ROWS: for (int childIndex = 0; childIndex < childrenCount; childIndex++) { - result.append (process (children.item(childIndex), datas, suffix)); + process (result, children.item(childIndex), datas, suffix); } break; } - - // - return (result); } @@ -370,7 +355,7 @@ public class Presenter if ((data == null) || (data.display () == null)) { - result.append (processChildren (node, datas, suffix)); + processChildren (result, node, datas, suffix); } else { @@ -417,7 +402,7 @@ public class Presenter // Insert data. if ((data == null) || (data.display () == null)) { - result.append (processChildren (node, datas, suffix)); + processChildren (result, node, datas, suffix); } else { @@ -455,14 +440,9 @@ public class Presenter /** * */ - static protected StringBuffer process (Node node, IdsDataById datas) throws Exception + static protected void process (StringBuffer result, Node node, IdsDataById datas) throws Exception { - StringBuffer result; - - result = Presenter.process (node, datas, ""); - - // - return (result); + Presenter.process (result, node, datas, ""); } @@ -470,16 +450,13 @@ public class Presenter * Recursive method that processes a node and any child nodes. * */ - static protected StringBuffer process (Node node, IdsDataById datas, String suffix) throws Exception + static protected void process (StringBuffer result, Node node, IdsDataById datas, String suffix) throws Exception { logger.debug ("process - started"); String TRANSITIONAL_DTD = "xhtml1-transitional.dtd"; String TRANSITIONAL_DOCTYPE = "\n"; - StringBuffer result; - result = new StringBuffer (); - // Is there anything to do? if (node != null) { @@ -514,7 +491,7 @@ public class Presenter // Log.write(Log.TRACE,"systemId = " + systemId); } - result.append (Presenter.process (((Document) node).getDocumentElement(), datas, suffix)); + Presenter.process (result, ((Document) node).getDocumentElement(), datas, suffix); break; } @@ -596,7 +573,6 @@ public class Presenter // //logger.info ("result=" + result); logger.debug ("process - ended"); - return (result); } @@ -644,7 +620,7 @@ public class Presenter { result.append('>'); - result.append (processChildren (node, datas, suffix)); + processChildren (result, node, datas, suffix); result.append("