in reply to Re: regular expression
in thread regular expression

Some comments:
  1. /.+ is not needed. You have anchored your regex at the end of the string, so there is no need to check if there is anything before the part you are looking for.
  2. \[(.+)\] will work if there are no other ] characters after next ]. Your pattern is greedy and will go all the way to the last ]. I would have written this either \[(.+?)\] (non-greedy solution) or \[([^\]]+) (everything up to the first closing bracket and you can drop the next \] in your regex.)

CountZero

A program should be light and agile, its subroutines connected like a string of pearls. The spirit and intent of the program should be retained throughout. There should be neither too little or too much, neither needless loops nor useless variables, neither lack of structure nor overwhelming rigidity." - The Tao of Programming, 4.1 - Geoffrey James

My blog: Imperial Deltronics

In Section Seekers of Perl Wisdom