> On Feb. 21, 2015, 4:23 a.m., Michael Park wrote: > > This seems like a small subset of regex, can we just use > > `boost::regex_match` for now with the intention that we'll switch to > > `std::regex_match` later on? The only thing would be that we'll have to > > update our bundled `boost` to include `regex`.
I couldn't agree more. However in our code only stout has a dependency to boost and this dependency is header only. Using `boost::regex` will add a binary dependency, which is the reason I didn't do it. - Alexander ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31227/#review73392 ----------------------------------------------------------- On Feb. 20, 2015, 5:01 p.m., Alexander Rojas wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/31227/ > ----------------------------------------------------------- > > (Updated Feb. 20, 2015, 5:01 p.m.) > > > Review request for mesos, Joerg Schad, Niklas Nielsen, and Till Toenshoff. > > > Bugs: MESOS-2333 > https://issues.apache.org/jira/browse/MESOS-2333 > > > Repository: mesos > > > Description > ------- > > Adds a function `wildcardMatching` which takes a string to be match and a > pattern and returns true if the string matches the pattern. > > The patter is a string where the special characters '*' and '?' are usind as > wildcards. '*' is used as "any character, zero or more times" while '?' is > used as "any character exactly once". > > > Diffs > ----- > > 3rdparty/libprocess/3rdparty/stout/include/stout/strings.hpp > 46b79bc5c11cb1878ffccb4c0c8bb8a49ee9832d > > Diff: https://reviews.apache.org/r/31227/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Alexander Rojas > >
