This is the mail archive of the kawa@sourceware.org mailing list for the Kawa project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Support CommonLisp in RunTestScript


On 07/02/2012 12:00 PM, Jamison Hope wrote:
Resending because I think the last one got lost in the GSOC thread.

Thanks - checked in.


I am still of the opinion that instead of hard-coding these
language-specific
tokens into RunTestScript, it should be able to query the Language
instance for
them (i.e. augment the Language API to include String methods
getBlockCommentStart(), getBlockCommentEnd(), etc).

I agree.


I recognize, though, that
this information is currently implicitly defined in several places in
the reader
classes, and so making such a change right now is too large and intrusive.

Well, I wouldn't expect to change the reader classes. Just add the methods to the Language classes, and then change RunTestScript. -- --Per Bothner per@bothner.com http://per.bothner.com/



Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]