On 2014-01-06 04:25, Kinkie wrote:
Hi,
the attached patch expands on the already-merged CharacterSet,
adding:
- operator+
- addRange
- range-based constructor
- a number of predefined sets, mostly taken from the HTTP specs
- unit test
It's a candidate for merging to trunk.
in src/base/CharacterSet.cc
* We should avoid defining CharacterSet::CRLF since that is a specific
ordered pattern rather than unordered set of characters.
* Please add an extra comment noting why QDTEXT and OBSTEXT are elided
... because they require C++11 constructors.
+1 once those are fixed.
Amos
