Skip to content
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

完善语法分析 #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Baytars
Copy link

@Baytars Baytars commented Nov 24, 2019

Eclipse的API似乎不能获取弹出语法提示前你输入了什么,所以只能获取行首至光标文本,对比行首至光标文本末尾与语法建议开头,找出需要补全的部分。More comprehensible version 下方注释是从这个思路出发设计的原始代码,经过优化成为 More comprehensible version 上方代码。
原谅我习惯使用英文变量😂如果理解我的思路,可以把变量改为中文。

@nobodxbodon
Copy link
Member

为明确修改目的,可否添加对应测试用例?刚提交了一些基本测试
另外,代码格式器我用的是 GoogleStyle,可否一致?

如果不太麻烦的话,还是请改成中文变量吧(也欢迎改进原有命名),可以尽早对一些术语用词达成一致,也便于我阅读。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants