Re: [PATCH] Support CommonLisp in RunTestScript

[Jamison Hope <jrh at theptrgroup dot com>]

On Jul 2, 2012, at 3:07 PM, Per Bothner wrote:

> Well, I wouldn't expect to change the reader classes.  Just add the
> methods to the Language classes, and then change RunTestScript.

Well, maybe. That would improve RunTestScript in that it gains the
ability to automatically handle any and all current or future
Kawa-based languages, which is awesome. On the other hand, it
would mean that the "comment syntax descriptors" (for lack of a
better term) are still stored in two places. In the (or my) ideal
situation, there would be a single canonical owner of that data,
so either the Language would defer to a reader class, or the
reader class would defer to the Language, or both would defer
to some other helper class.

In this case, we aren't likely to suffer from any inconsistency, as
comment syntax is pretty fundamental and stable and not likely to
change any time soon. But it's still a nice goal to strive for in
general. I've encountered plenty of situations in my day job where
you've got some important system parameter (a velocity limit for
a robot, for instance), you change it and rerun, and nothing changes
because it turns out there are two copies of that info and you edited
the wrong one.

--
Jamison Hope
The PTR Group
www.theptrgroup.com