Re^3: Suggestion for regular expression speed improvement.

Replies are listed 'Best First'.
Re^4: Suggestion for regular expression speed improvement. by bala.linux (Novice) on Jun 15, 2009 at 13:03 UTC
As I mentioned above, I would not be able to take that approach. Since, I want to enable support to match log lines of any format with grouping. If I go by CSV, then I would not be able to parse other formatted logs like syslogs and other proprietary logs.	[reply]
Re^5: Suggestion for regular expression speed improvement. by demerphq (Chancellor) on Jun 15, 2009 at 14:25 UTC
You need to think out of the box. Make your system handle the various cases. A regex, a split, an unpack, maybe even an xml parser.... Definitely doing it the way you are is wrong. If you are really insistent on ignoring our advice (unwise really), then at least use inversion and you might want to investigate the (?>...) construct "atomic matching". As well as adding anchors to your pattern. --- $world=~s/war/peace/g	[reply]
Re^6: Suggestion for regular expression speed improvement. by moritz (Cardinal) on Jun 15, 2009 at 14:34 UTC
Make your system handle the various cases. A regex, a split, an unpack, maybe even an xml parser.... I agree. The simplest solution is to make the caller supply a code reference, which you then call and which does the parsing/matching/unpacking/whatever.	[reply]
Re^6: Suggestion for regular expression speed improvement. by bala.linux (Novice) on Jun 15, 2009 at 14:46 UTC
Exactly. This was my approach but before taking that up I wanted to check with the community whether Im doing anything wrong with regular expression. May be I expected too much from the regular expression. Anyway, thanks for your inputs :)	[reply]
Re^7: Suggestion for regular expression speed improvement. by demerphq (Chancellor) on Jun 15, 2009 at 17:10 UTC
Re^8: Suggestion for regular expression speed improvement. by bala.linux (Novice) on Jun 15, 2009 at 18:11 UTC
Some notes below your chosen depth have not been shown here


Come for the quick hacks, stay for the epiphanies.
	PerlMonks