sebkirche / pbniregex Goto Github PK

By adding a method string group( long matchId, string groupName ) so we could simplify interpretation of complex regex:

lrx_extract.initialize( "(?x) (?| \bfile:? (.+) \s at \s line \s (\d+) | \s at \s line \s (\d+) | \bfile:? (.+) )", false, false )

Here we have to check if the group #1 is like a number.

Then, this could be rewrited as :

lrx_extract.initialize( "(?x) \bfile:? (?<file>.+) \s at \s line \s (?<line>\d+) | \s at \s line \s (?<line>\d+) | \bfile:? (?<file>.+)" , false, false )

So after we can just do:

ls_file = lrx_extract.group( 1, "file" )
ll_line = long( lrx_extract.group( 1, "line") )

Using replace() with an empty string as search argument runs into an infinite loop.

We could prevent that situation returning the input string unmodified as result instead of stalling.

PbniRegex is built upon PCRE version 5.8 (compiled on 2008-09-05).
At the time of this writing, PCRE is now in version 8.33.

The lib used by pbniregex [c|s]hould be upgraded.

Starting at PB11, the documentation states that when using IPB_Session::GetString() a corresponding ReleaseString() must be invoked to free the memory allocated.

The code misses the required ReleaseString() calls and should suffer memory leaks :^S

The pcre_study() function that makes some analysis on a given regexp may return a pointer to a block of data that is reused by pcre_exec() (or it may return NULL).

When such a block is returned, it must be freed by pcre_free() (or pcre_jit_free() since version 8.20) but pbniregex never frees it.

version pb 2017 r3 ?

I have just fixed in bedb95e a problem with a POST LocalFree() in n_tooltip in the 11.5 library, the fix must be ported to the other versions.