search

kieker-monitoring / kieker

Kieker's main repository
Apache License 2.0
70 stars 41 forks source link

[KIEKER-880] The class PatternParser should get a thorough review and cleanup #1050

Open rju opened 2 weeks ago

rju commented 2 weeks ago

JIRA Issue: KIEKER-880 The class PatternParser should get a thorough review and cleanup Original Reporter: Jan Waller

The class [PatternParser|source:src/monitoring/kieker/monitoring/core/signaturePattern/PatternParser.java] should get a thorough review and cleanup. It is currently quite complicate to read and understand this class.

Checklist:

rju commented 2 weeks ago

author André van Hoorn -- Mon, 3 Jun 2013 17:06:21 +0200

Review (Structured Walkthrough) in one of the next dev meetings

rju commented 2 weeks ago

author André van Hoorn -- Mon, 29 Jul 2013 16:16:17 +0200

Could make sense to implement as text replacement. Another option could be ANTLR, but maybe overkill.

Quite a comprehensive test suite exists; hence, probably becomes relevant if extensions to expressions needed.