~emersion/public-inbox

This thread contains a patchset. You're looking at the original emails, but you may wish to use the patch review UI. Review patch
2 2

[PATCH go-sieve] Fix tokenization of numbers

Details
Message ID
<20240924200027.4169411-1-andy@balholm.com>
DKIM signature
pass
Download raw message
Patch: +1 -1
---
 tokenizer.go | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tokenizer.go b/tokenizer.go
index 406225a..d1aded6 100644
--- a/tokenizer.go
+++ b/tokenizer.go
@@ -226,7 +226,7 @@ func (t *tokenizer) number() string {
			break
		}

		if !isAlpha(c) {
		if !isDigit(c) {
			if sb.Len() > 0 && isQuantifier(c) {
				sb.WriteByte(c)
			} else {
-- 
2.46.2
Details
Message ID
<bdec930f-51b6-4c81-bd24-8e2a37624f8a@balholm.com>
In-Reply-To
<20240924200027.4169411-1-andy@balholm.com> (view parent)
DKIM signature
pass
Download raw message
Currently, numbers always give a syntax error, because the tokenizer is 
expecting them to be made up of letters.

Andy
Details
Message ID
<RtDzvkyEcuySZmUoxF2FrnLhj2y3lZtzOAuuDvALs0FDAMkPZY1l9nbWFDnNaQrnc1ZAzyxtkDBTPIkQNpcJqo88MB6z5fVEVj1GM6DeuYI=@emersion.fr>
In-Reply-To
<bdec930f-51b6-4c81-bd24-8e2a37624f8a@balholm.com> (view parent)
DKIM signature
pass
Download raw message
Pushed, thank you!
Reply to thread Export thread (mbox)