[Joe-task-developer] =?iso-8859-1?Q?R=E9f=2E_=3A_[Joe-task-developer]_AvalonCodeWriter?=

[<loi...@bn...>]

Hello T=F5nis,

> Hello all!
>
>
> I plan to finish with AvalonCodeWriter this week but some problems still
> remain.
>
> 1)  Writing m_ prefix to all class variable references. This is quite
> tricky, information needs to be collected first. Symbol tabels there
> mentioned?

Indeed, this part of the task will be made later since it needs the symbol =
table!

>
> 2) Indent comments on an 80 column basis and the code on a 100 column, us=
ing
> a two more indents when a line must be wrapped.
>
>  Well, there are problems with comments output, it looks like sometimes t=
hey
> appear not exactly where they should  ;)  Use test5.java to see this. It
> seems to be parser problem.

Hmmm in fact, comments are attached to the following JJTree node so it shou=
ld be ok
but the way they are written is not valid. In fact, one method call should =
be added per
writeXXX method to manage the comments according to the nodes.

>  Doing comment line wrapping is easy, code line wrapping is much tougher.

Yes, totally agree :-(

> If you wanna indent intelligent way like Sun coding conventsions suggest,=
 you
> need more complete information. Simple line wrapping may be done in write=
XXX
> functions with some additional computing and tricks, I guess.

I suggest you to finish all the rest except the "m_" attribut prefix and th=
en we will
think about this, I think we could look the code of open source java code f=
ormater
proprams ;-)

>
> 3) Talking about tricks, I actually didn't know that other CodeWriter's a=
re
> using AvalonCodeWriter as base class. So I changed code a little bit and
> don't know how it will affect others work?

In fact, the JBoss and Sun code writer extends the AvalonCodeWriter thus one
will only overrides the methods that are different (in fact copy the Avalon=
 one and manage
the new lines, tab...) For example the JBoss code writer is quite simple to=
 implement since
the brackets' positions are the same than in the Avalon code writer.

I hope my change didn't add any conflict to your work. :-)

About the symbol table, I'm awaiting informations from Jason, but I think h=
e has got
too much work to begin to think on it :-( But I'll begin to design it next =
week, we really need it
because it'll reduce the complexity of the Assinment operator COP for examp=
le :-)

At last I need you to commit your last version of AvalonCodeWriter today be=
cause I'll
make the 0.2a package this evening (that's to say at about 23:00 GMT+1:00).

Thanks,
Lo=EFc




This message and any attachments (the "message") is
intended solely for the addressees and is confidential.=20
If you receive this message in error, please delete it and=20
immediately notify the sender. Any use not in accord with=20
its purpose, any dissemination or disclosure, either whole=20
or partial, is prohibited except formal approval. The internet
can not guarantee the integrity of this message.=20
BNP PARIBAS (and its subsidiaries) shall (will) not=20
therefore be liable for the message if modified.=20

                ---------------------------------------------

Ce message et toutes les pieces jointes (ci-apres le=20
"message") sont etablis a l'intention exclusive de ses=20
destinataires et sont confidentiels. Si vous recevez ce=20
message par erreur, merci de le detruire et d'en avertir=20
immediatement l'expediteur. Toute utilisation de ce=20
message non conforme a sa destination, toute diffusion=20
ou toute publication, totale ou partielle, est interdite, sauf=20
autorisation expresse. L'internet ne permettant pas=20
d'assurer l'integrite de ce message, BNP PARIBAS (et ses
filiales) decline(nt) toute responsabilite au titre de ce=20
message, dans l'hypothese ou il aurait ete modifie.