Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Review and update the regex patterns in the measurement data protocol #18

Open
5 tasks
KGergo88 opened this issue Mar 19, 2019 · 1 comment
Open
5 tasks
Assignees
Labels
enhancement New, or improved functionality

Comments

@KGergo88
Copy link
Owner

KGergo88 commented Mar 19, 2019

Description

The project currently using the std::regex namespace for all the regular expression related tasks. Unfortunately this does not support more advanced regex functionalities, like named groups. It would make sense to review the regex related parts of the project and rewrite them with using the Qt regex features.

Todo

  • Review the regex related parts (mostly the protocol handlers)
  • Investigate the Qt regex features, like QRegularExpression.
  • Evaluate whether these could be made simpler by using Qt
  • Implement the changes
  • Extend the unit tests if necessary
@KGergo88 KGergo88 added the enhancement New, or improved functionality label Mar 19, 2019
@KGergo88 KGergo88 self-assigned this Mar 19, 2019
@KGergo88 KGergo88 changed the title Review and update the regex patterns Review and update the regex patterns in the measurement data protocol May 21, 2019
@KGergo88
Copy link
Owner Author

KGergo88 commented Jan 2, 2021

An example for the Qt regexes can be found in the CMDP class. See #36.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New, or improved functionality
Projects
None yet
Development

No branches or pull requests

1 participant