[Octave-patch-tracker] [patch #10292] Switch from PCRE to PCRE2

[John W. Eaton]

Follow-up Comment #1, patch #10292 (project octave):

Thanks for providing these changes.

Octave's public header files should not depend on macros like
PCRE2_DATA_WIDTH, PCRE2_CODE_UNIT_WIDTH, or include the pcre2.h header
directly.  Using PCRE (or any library) is an implementation detail that
shouldn't be exposed to someone just wanting to use the regexp class.  That's
why we used the void pointer previously and only included the header files in
.cc files.

If we need a change for OCTAVE_CHECK_LIB, that should probably be discussed
separately.

How do we know that a buffer size of 128 is sufficient for the call to
pcre2_get_error_message?  Should we try to adapt that in case the message ends
up being longer and pcre2_get_error_message returns a failure code?


    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/patch/?10292>

_______________________________________________
Message sent via Savannah
https://savannah.gnu.org/