You're viewing a comment by Jaanis and its responses.

Jaanis Permalink
April 19, 2013, 01:46

Regarding 109: in general this is nasty approach to match something between start and end of the string as you may run in funny problems with it. I'd recommend to use \b which matches word boundary around (please look into perl regexp specs for it).

Regarding 112: Although criticized a lot there is one significant problem - you match .+ before @ which means "everything before @". Please note "everything" may include spaces and other things you do not want to see.

117: Important to note you are matching a single character this way, not characters. :)

Did not check others in details but well - good idea of collecting this kind of stuff and good luck!

Jaanis Kruumins

Reply To This Comment

(why do I need your e-mail?)

(Your twitter name, if you have one. (I'm @pkrumins, btw.))

Type the word "browser": (just to make sure you're a human)

Please preview the comment before submitting to make sure it's OK.