>>>>> "Angus" == Angus Leeming <[EMAIL PROTECTED]> writes:

Angus> I've rewritten ReplaceEnvironmentPath in a sane manner and got
Angus> rid of lstrings.[Ch] regexMatch because nothing was using it.

This is very nice. The only possible improvement would be to allow the
omission of curly brackets in cases where there is no ambiguity.

That would mean that, beside this case:
+       // A valid environment variable is defined as
+       // $\{[A-Za-z_][A-Za-z_0-9]*\}
we should handle a case like
  $[A-Za-z_][A-Za-z_0-9]*

JMarc

Reply via email to