Hi all,
Thanks for the C&C on my regex. I will definitely avoid the tentpole
syndrome in the future, as well as add the option of having no spaces
around the hint words (although I need to check to see if this condition
*should* be allowed on Oracle's end), and removing some greediness in
looking for the hint keywords themselves. Here is a revised version:
if ( $query =~ m#/\*\+\s+?(+*?)\s+?\*# ) {
$hint = $1;
}
With this particular regex, it's not exactly the centerpiece of the
script, so I don't have a need to expand it readability-wise. This code
is purely for producing some diagnostic text in the context of the main
program, and is thus largely throw-away. However, it offered me a
welcome diversion to brush up on my regex game. :-)
Thanks again, all.
Peter
--
Peter R. Wood (Lists) - [EMAIL PROTECTED] - http://prwdot.org/
