dialogos-project / dialogos

The DialogOS dialog system.
https://www.dialogos.app
GNU General Public License v3.0
21 stars 8 forks source link

Plus operator in DirectGrammars #121

Closed alexanderkoller closed 6 years ago

alexanderkoller commented 6 years ago

According to our analysis (see #109), the patterns of a DirectGrammar can be any RHS of an ABNF rule (see #120). We have confirmed that the Kleene star (*) works correctly, but the plus (+) operator doesn't seem to work. Instead, the pattern behaves as if the + wasn't there.

We should figure out what's going on and, if necessary, submit a bug report to Sphinx. Some things to do:

timobaumann commented 6 years ago

works nicely in Sphinx, see test in eaac4605c54772a2e89e7fc29cb1b6f778ccec3a

timobaumann commented 6 years ago

I traced this to the URL-encoding that we use to pass the grammar to Sphinx, looking for the best solution to url-encode a +.

timobaumann commented 6 years ago

fixed in c5e2d8199c64b142cd7b78ad5c78bbe07fc895a9

alexanderkoller commented 6 years ago

Beautiful!

Record it in the CHANGELOG, too?