https://bugs.exim.org/show_bug.cgi?id=2400

            Bug ID: 2400
           Summary: PCRE c++ wrapper start_options missing LF option
           Product: PCRE
           Version: 8.43
          Hardware: All
                OS: All
            Status: NEW
          Severity: bug
          Priority: medium
         Component: Code
          Assignee: p...@hermes.cam.ac.uk
          Reporter: chasinglo...@gmail.com
                CC: pcre-dev@exim.org

The PCRE C++ wrapper was updated for 8.43 to use a start_options array to check
for all valid start options but the list does not include the (*LF) option from
https://www.pcre.org/original/doc/html/pcrepattern.html#SEC2

Obviously this is a small change but I have made a patch for it:

diff --git a/pcrecpp.cc b/pcrecpp.cc
index 77a2fed..70b84bd 100644
--- a/pcrecpp.cc
+++ b/pcrecpp.cc
@@ -91,6 +91,7 @@ static const char *start_options[] = {
   "(*LIMIT_RECURSION=",
   "(*LIMIT_MATCH=",
   "(*CRLF)",
+  "(*LF)",
   "(*CR)",
   "(*BSR_UNICODE)",
   "(*BSR_ANYCRLF)",


I could not find how to submit the patch upstream (and I'm not familiar with
subversion anyway) so feel free to just do this with no credit.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-- 
## List details at https://lists.exim.org/mailman/listinfo/pcre-dev 

Reply via email to