Pattern match failed

zed · 2018-02-22 19:34:27

Test code:

IsMatch('ab*.[ef]xyz', 'ab.exyz', True);
IsMatch('ab*.[ef]xyz', 'abcd.exyz', True);
IsMatch('ab*.[ef]xyz', 'ab.fxyz', True);
IsMatch('ab*.[ef]xyz', 'abcd.fxyz', True);

all failed!

mORMot version: 1.18.4332.
Compiler: Delphi 2007.

Last edited by zed (2018-02-22 19:37:16)

ab · 2018-02-22 22:13:28

Did you find out why?

Also the following fails:

  Check(IsMatch('ab.[ef]xyz', 'ab.exyz', True));
  Check(IsMatch('ab[ef]xyz', 'abexyz',true));

But this one works:

  Check(IsMatch('ab[ef]xyz', 'abexyz',false));

ab · 2018-02-23 11:24:56

Please check https://synopse.info/fossil/info/c35f1ecda7

zed · 2018-02-23 19:19:26

Yes, it helps.

But here is a new fail:

Assert(not IsMatch('ab[cd]e', 'abdde', False)); // ok
Assert(not IsMatch('ab[cd]ex', 'abddex', False)); // ok

Assert(not IsMatch('ab*.[cd]e', 'ab.dde', False)); // ok
Assert(not IsMatch('ab*.[cd]ex', 'ab.ddex', False)); // fail

ab · 2018-02-23 21:29:19

Another one....

Please check https://synopse.info/fossil/info/f27ae4bd52

Thanks a lot for the feedback!

zed · 2018-02-23 21:52:06

Ok, now it's work. Thank you!

Last edited by zed (2018-02-23 21:52:25)

edwinsn · 2018-02-24 09:34:08

This sounds like a simplified regular expression, cool! And I didn't know about it before.

PS, not a real issue, but I couldn't resist - I thought it should be placed in SynCommon.pas instead?

Junior/RO · 2018-02-24 18:19:32

Yeah, SynCommons is a better place

ab · 2018-02-24 19:01:19

It was in SynCommons.pas but the file is so big it became more than 65535 lines... which makes the Delphi compiler fails... Delphi 5 compiler, but to be honnest...

Check the explanation at the beginning of SynTable.pas.

mpv · 2018-02-24 19:25:58

Please, do not grow a SynCommons. It's already big enough. For example I can't even view it on github or give someone a HTTP link to the line of code when answer a question. I vote to split it into parts.

esmondb · 2018-02-24 21:46:50

+1 for splitting SynCommons

edwinsn · 2018-02-25 03:27:02

I didn't take into account the unit file size when I was thinking about it, just thought about it logically

mORMot Open Source

#1 2018-02-22 19:34:27

Pattern match failed

#2 2018-02-22 22:13:28

Re: Pattern match failed

#3 2018-02-23 11:24:56

Re: Pattern match failed

#4 2018-02-23 19:19:26

Re: Pattern match failed

#5 2018-02-23 21:29:19

Re: Pattern match failed

#6 2018-02-23 21:52:06

Re: Pattern match failed

#7 2018-02-24 09:34:08

Re: Pattern match failed

#8 2018-02-24 18:19:32

Re: Pattern match failed

#9 2018-02-24 19:01:19

Re: Pattern match failed

#10 2018-02-24 19:25:58

Re: Pattern match failed

#11 2018-02-24 21:46:50

Re: Pattern match failed

#12 2018-02-25 03:27:02

Re: Pattern match failed

Board footer