clp icon indicating copy to clipboard operation
clp copied to clipboard

core-clp: Rewrite wildcard matching method and add systematic unit tests (fixes #427).

Open kirkrodrigues opened this issue 8 months ago • 3 comments

Description

This PR:

  • rewrites clp::string_utils::wildcard_match_unsafe_case_sensitive both to fix #427, to slightly simplify the logic, and to add more comments to explain the algorithm;
  • completely rewrites the unit tests for wildcard matching so that they (hopefully) systematically test all cases;
  • replaces the wildcard performance test to be more realistic by matching against several lines (rather than a single line) from an example log file.

Validation performed

Validated unit tests passed.

kirkrodrigues avatar Jun 07 '24 00:06 kirkrodrigues