Skip to content

Conversation

CppCXY
Copy link
Member

@CppCXY CppCXY commented May 17, 2022

  1. 是否要集成拼写检查
  2. 如果要集成拼写检查那么词典放哪里,词典可以以很多形式存在比如用[[]]扩起来的lua文件或者txt或者其他形式
  3. 单词拼写纠正:如果要集成纠正当前缺少一些设施:
  • 代码诊断的时候不能传入data
  • 也不能根据range获得既定文本

@sumneko
Copy link
Collaborator

sumneko commented May 18, 2022

拼写检查已经完全和特定的语言服务没有关系了吧,用户自己安装单独的插件就行了

@CppCXY
Copy link
Member Author

CppCXY commented May 18, 2022

拼写检查依然是一个很好的特性,intellij平台自己的插件大多都带有拼写检查, vs对自己主要支持的语言都带有拼写检查。目前来看vscode已经有的拼写检查插件cSpell没有对lua有专门的支持。
拼写错误的检查和纠正可以有效的修复一些未定义类型,来自宿主的全局变量的拼写错误.

@sumneko
Copy link
Collaborator

sumneko commented May 18, 2022

image
cSpell你去设置里加一下lua就可以了

@CppCXY
Copy link
Member Author

CppCXY commented May 18, 2022

cSpell是没有专门的对lua做处理,这一点我是确认过的,他的确是可以通过设置强行支持lua文件,但是只是当作一般文本来处理。
没有专门支持lua语言的词典,在纠错上也只是符合一般单词,纠错能力上其实很弱,候选单词大多无用。
插件内集成相关算法可能确实有争议,可以讨论一下。或者功能可以先放插件里面不开启。

@sumneko
Copy link
Collaborator

sumneko commented May 18, 2022

我没有用过拼写检查插件,你举个你支持但是cSpell不支持的功能?

@CppCXY
Copy link
Member Author

CppCXY commented May 18, 2022

那论功能,我这是远弱于cSpell,我这个地方只是小而精的支持必要的特性,好处就是配置跟着插件走, 可以按文件或者按行屏蔽,cSpell就不能这样

@CppCXY CppCXY marked this pull request as ready for review May 18, 2022 09:52
@CppCXY CppCXY force-pushed the master branch 2 times, most recently from 7e50d34 to 906051f Compare May 24, 2022 03:39
@sumneko sumneko merged commit 6185d71 into LuaLS:master May 27, 2022
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