On Mon, Nov 22, 2004 at 07:27:18PM +0100, Jerome Quelin wrote: > On 04/11/22 07:14 -0700, Patrick R. Michaud wrote: > > Looks great to me -- I'll switch things around to (pardon the pun) match. > > They'll become "P6GE::compile" (compiler sub) and "P6GE::Match", with > > a note that P6GE::compile is still subject to change. > > Well, since we're nitpicking conventions and consistency, what about: > PGE::compile > and PGE::match (instead of Match)
As Luke mentioned, "Match" is uppercase because it's a class. However, I ended up naming the compilation routines based on what they compile -- i.e., "p6rule", "p5re", and "glob". Note that this may or may not eventually correspond to the string name that is used as part of the C<compreg> opcode -- that's still to be worked out. Pm
