~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 scfg] Specification: Fix directive-params

Details
Message ID
<20210320074911.27220-1-git@maringuu.de>
DKIM signature
missing
Download raw message
Patch: +1 -1
---
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
Details
Message ID
<CA24Q4GY3JR9.1ZUO7I9OBV45E@marten-xps>
In-Reply-To
<20210320074911.27220-1-git@maringuu.de> (view parent)
DKIM signature
missing
Download raw message
> This would mean that one word parameters are not allowed.

Ehh, that is of course not true but still I think the patch simplifies the grammar.
Details
Message ID
<cVcNajnMBn7wY5XGQF4fyxZ8Qfhv6blt46tJjbcqbBoJK70mirwJj5aLHQB2Kqx33qtzHXkNoLjcMW4ufsJeEKr0IsaNsZSwhx3AON8Wpqs=@emersion.fr>
In-Reply-To
<20210320074911.27220-1-git@maringuu.de> (view parent)
DKIM signature
missing
Download raw message
Oh, indeed, that's simpler. Tweaked the commit message and pushed,
thanks!
Reply to thread Export thread (mbox)