-
-
Notifications
You must be signed in to change notification settings - Fork 38
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[150] 优化指令参数说明的格式 #844
Comments
我反对增加语法糖的修改。我的理由是:增加语法糖而不禁止旧的写法只是在增加用户需要记忆的语法规则量。 我希望修改者能按 ta 的想法重新设计语法,为以下各个情况分别定义唯一的写法:
作为参考,在当前语法下,以上 10 种情况分别表述为:
|
我认为这一糖是对已经存在的规则5的延伸,如果认为5的 [A|B] 能展开是自然的,那 <A|B> 不予展开与否就有些奇怪? |
在当前规范中,
那么我希望你能做到一步到位地替换旧写法,不要让两种写法共存,以免增加记忆成本 |
认可同一个含义不应该存在两种表达。 |
E <- ((expr / or / wrap / orExt / replacedExt) sp)+ // 入口 identifier <- ... // 总之就是非符号unicode字符串 replacedName <- '<' identifier '>' expr <- replacedName / keywords or <- '[' expr ']' orExt <- '[' (expr '|')+ ']' sp <- ' ' // 空格 |
最终在文档和帮助中,只会看到一种写法。只是恰好与旧语法没有语义冲突。
只有6和8写法变化了。 |
对立解决方案:如果在下个版本发版前仍没有将使用现行规范的说明统一修改为新提议规范(包括 sealdice/sealdice-manual-next#209,以及截至发表此评论仍暂缺的 core 中指令帮助修改,后者可以参照 #841),则应该视为该新提议是难以实行的,建议将 #842 (comment) 中被标记的位置视为格式不符合现行规范,采纳建议的修改 |
Originally posted by @fy0 in #842 (comment)
按照群内讨论,我认为可以借鉴文档中所示的这一规则:
即
[A|B] = ([A] | [B])
简化
(<角色名>|<序号>)
为<角色名|序号>
即
<A|B> = (<A>|<B>)
这样的好处为:
.pc rename <角色名|序号> <新角色名>
比.pc rename (<角色名>|<序号>) <新角色名>
更加紧凑The text was updated successfully, but these errors were encountered: