Re: testsuite: prefix handling

Tom Tromey
2012-02-21T20:59:24Z

>>>>> "Pedro" == Pedro Alves <palves@redhat.com> writes:

Pedro> I really dislike mixing in a lot of formatting changes with
Pedro> code changes, so I was really avoiding it.

Yeah, I totally blanked out about that.
I completely agree.

Pedro> Honestly, I kind of liked the current formatting -- I was kind of
Pedro> reading it as a function attribute. But I don't mind seeing this
Pedro> re-indented either. But that's a lower priority to me.

If you don't object I will do it.

There's also the defining style (untested):

proc prefix_proc {name arglist substitution body} {
set delayed "\[[list subst $substitution]\]"
proc $name $arglist [list with_test_prefix $delayed $body]
}

prefix_proc test_something {name} {hi: $name} {
blah blah
}

Ok, I wouldn't really want to use it anyway...

Tom