trushworth

Results 15 comments of trushworth

A lot of these changes seem to have ended up in the python directory (as opposed to your python3 directory) before I started confusing things :). I inadvertently added even...

I think I've managed to hand merge all these changes into the master branch. I hope I've given credit correctly in the commit message Thank you for the improvements!

I've merged this by hand, partly because I've taken a slightly different approach to the string constant encoding/locale in tests/retest.c, and partly because the master branch has moved ahead (via...

Merged by hand (together with PR#57) in commit 3702f071. Thank you!

awilfox's suggested one-line fix: > The fix is pretty simple: replace `AC_CONFIG_AUX_DIR(utils)` with `AC_CONFIG_AUX_DIR([utils])` in `configure.ac`. has been commited to laurikari/tre if you want to use the latest.

This looks reasonable, but I want to test it offline first and it will be a week or two before I can do that. Thanks for this (in advance)!

Hi, Reply is in-line below. On Sat, Jun 03, 2023 at 08:33:26AM -0700, Ben Bolker wrote: > The [TRE documentation](https://laurikari.net/tre/documentation/regex-syntax/) defines a range as > > > Two characters separated...

Hi, On Sun, Jun 04, 2023 at 02:03:02PM -0700, Ben Bolker wrote: > For what it's worth there's an [extensive Stack Overflow answer](https://stackoverflow.com/questions/11925537/should-we-consider-using-range-a-z-as-a-bug/44228789#44228789) demonstrating that many tested regex implementations use...

I'm hoping to get a look at this (and a couple of other outstanding issues) later this week. I've retired (and don't have to deal with windows any more) but...

My apologies if I didn't follow the established style, I thaought I was doing so. On reviewing the code the indentation seems to be correct (multiples of 2 spaces), is...