[0027] 修复 gf fmt 对纯分号注释和开头空行的格式化问题#789
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
gf fmt对以分号开头的注释内容（纯分号分隔线和;;;注释）格式化时错误插入空格的问题gf fmt格式化时文件开头空行被意外移除的问题Changes
tools/fmt/liii/goldfmt-format.scm:format-comment-content: 注释内容以;开头时不插入空格join-top-level: 正确处理开头的空 piece，保留文件开头空行format-top-level-nodes: 移除首个 newline 节点的特殊处理tools/fmt/liii/goldfmt-scan.scm: 处理文件开头空行和末尾换行符tools/fmt/liii/goldfmt.scm: 使用format-nodes统一格式化输出tools/fmt/tests/liii/goldfmt-format/format-string-test.scm: 补充 5 个单元测试devel/0027.md: 添加开发文档Test plan
bin/gf test tools/fmt/tests/liii/goldfmt-format/format-string-test.scmbin/gf test tools/fmt/tests/liii/goldfmt-scan/bin/gf test tools/fmt/tests/liii/goldfmt-tokenize/bin/gf test tools/fmt/tests/liii/goldfmt-rule/bin/gf test tools/fmt/tests/liii/goldfmt-record/🤖 Generated with Claude Code