跳到内容
新问题

对这个项目有疑问吗?注册一个免费的GitHub帐户以打开一个问题,并联系其维护者和社区。

单击“注册GitHub”,表示您同意我们的服务条款隐私声明。我们偶尔会向您发送与帐户相关的电子邮件。

已经在GitHub上了?登录到您的帐户

将ctags解析器与Geany进行小更改同步 #2990

已合并
将7个提交合并到
2021年12月2日

对话

技术人员
复制链接
成员

这个pull请求从解析器中删除了geany_前缀,我们可以很容易地与ctags同步,它只包含对这些解析器的小更改和来自uctag的副本(来自标签p5.9.20211031.0的版本,如#2984). 在提交消息中可以找到更多详细信息。

我也计划对其他解析器执行同样的操作,这个pull请求只包括那些没有包含太多更改的解析器,以使这个pull要求易于查看。此拉请求编译良好,与是否#2984是否应用。

这将允许update-ctags.py脚本替换这些解析器使用来自uctags的解析器。
这个补丁在不带gutags前缀的解析器上复制标签p5.9.20211031.0。接下来,当有一个改变。ABC映射中删除的种类从未由ABC解析器,已在上游版本中删除。新的“C”和“d”将各种HAXE解析器映射到tm_tag_undef_t。最后,txt2tags使用不同的上下文分隔符(“\”而不是\x03)因此tm_parser_context_sepaator()得到了相应的更新单元测试。同时,我从中删除了TM_PARSER_ASCIIDOCtm_parser_context_sepaator(),因为解析器不使用上下文完全是分隔符,在那里没有必要。
@库格尔-
复制链接
成员

我还没有详细查看update-ctags.py,但我认为它试图保留我们的小更改?

@技术人员
复制链接
成员 作者

我没有详细查看update-ctags.py,但我认为它试图保留我们的小更改?

ctags中唯一的微小变化主要的是什么ctags_changes.patchdoes-该脚本只是从ctags主服务器复制文件并应用此修补程序。

另一个例子是,除了这个pull请求中的解析器外,解析器的变化相当大。脚本忽略以开头的分析器杰尼_(这些必须由我们维护),并且只替换没有这个前缀的解析器。这也是这个拉取请求所做的事情-删除杰尼_前缀,以便下次运行更新标记.py,这些解析器的上游版本将自动复制,我们不必再担心它们。

@elextr公司
复制链接
成员

同意@技术人员其目的是能够简单地自动更新ctags中的更改。对于那些有价值更改的解析器,我们应该将其推到上游,这无论如何都是一件好事,然后它允许我们自动导入解析器。

抄送而其他有重大变化的则更多或是一个问题杰尼_前缀将保护它们,直到它们可以同步为止(或者在C/C++的情况下用新的上游替换)。

@elextr公司
复制链接
成员

请参见#2991(评论)

该测试缺失,未执行单元测试。
REST解析器中未使用上述范围分隔符“:::”。这个解析器只包含一级作用域(只有直接父作用域),因此它不使用范围分隔符。
它在初始提交时被删除,但即使它不使用嵌套范围“.”可能出现在标题中,所以使用一些不太可能的内容。
它以前用作txt2标记范围分隔符,因为不可打印,这比可以写在标题中的:::要好。途中,将开关上方的真正“\”\“”范围分隔符移动所以特殊的“\x3”案例列在最后。
@技术人员
复制链接
成员 作者

为了澄清我在最后几次提交中所做的工作,首先,我对中提到各种文本文件格式解析器的原因有点困惑tm_parser_context_sepator()即使它们不报告嵌套范围,上下文分隔符也不会出现在上下文中。它们出现在这里的唯一原因是报告了一些无效的上下文分隔符,因此tm_parser_context_sepator()不报告假阳性范围分隔符(例如想象1.1我的小节哪里.将被检测为范围分隔符)。

这些语言的情况是-txt2tags现在报告完整的上下文""作为其上下文分隔符。Asciidoc、conf、rest不使用上下文分隔符并使用\x3个作为一个不太可能出现的文本字符。

我浏览了这个pull请求中的所有语言,从我的角度来看一切似乎都很好,据我所知,这个pull要求已经准备好合并了。

@elextr公司 elextr公司合并提交808b7a3号进入之内 杰尼:主人 2021年12月2日
@elextr公司
复制链接
成员

此拉请求已准备好合并。

完成!!!

免费注册 在GitHub上加入此对话.已经有帐户了吗?登录以发表评论
标签
还没有
项目
还没有
开发

成功合并此请求可能会解决这些问题。

3名参与者