~emersion/public-inbox

scfg: Specification: Fix directive-params v1 PROPOSED

Marten Ringwelski: 1
 Specification: Fix directive-params

 1 files changed, 1 insertions(+), 1 deletions(-)
Export patchset (mbox)
How do I use this?

Copy & paste the following snippet into your terminal to import this patchset into git:

curl -s https://lists.sr.ht/~emersion/public-inbox/patches/21307/mbox | git am -3
Learn more about email & git
View this thread in the archives

[PATCH scfg] Specification: Fix directive-params Export this patch

---
From the ABNF rcf 3.10 the operator precedence is the following (High to low):
      Rule name, prose-val, Terminal value

      Comment

      Value range

      Repetition

      Grouping, Optional

      Concatenation

      Alternative

So to my understanding the current rule for directive-params lets you choose between
'[word]' and 'word 1*( 1*WSP word)'.
This would mean that one word parameters are not allowed.
I rewrote the rule to mean: "One or more words seperated by whitespaces".

 README.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/README.md b/README.md
index 9ef4d95..6853fb2 100644
--- a/README.md
+++ b/README.md
@@ -43,7 +43,7 @@ config = *directive

directive = directive-name [ 1*WSP directive-params ] [ 1*WSP block ] newline
directive-name = word
directive-params = [ word ] / word 1*( 1*WSP word )
directive-params = word *( 1*WSP word )

block = "{" newline *directive "}"

-- 
2.30.2
Oh, indeed, that's simpler. Tweaked the commit message and pushed,
thanks!