actboy168 / lua-debug

Lua Debug Adapter for Visual Studio Code
MIT License
431 stars 93 forks source link

add launcher config #234

Open fesily opened 1 year ago

fesily commented 1 year ago

signature两个配置module和version,module需要用户填写,version自动从luaVersion字段读取

将attach的luaversion默认设置为unknown,允许launcher自动判断版本

~将luadbug和launcher添加为module黑名单~ 将调试器目录下的所有module为黑名单

actboy168 commented 1 year ago

package_json.lua的格式全乱了

actboy168 commented 1 year ago

由launcher加载调试器dll的代码也可以包含进来,这很有用。

fesily commented 1 year ago

package_json.lua的格式全乱了

设置中保存自动format,我先添加一个pull来完成code-format的功能把现有的lua文件都format一遍

actboy168 commented 1 year ago

format是个大工程,尤其是在lua 的format还不是很成熟的情况下。所以你还是先别改吧。

fesily commented 1 year ago

format是个大工程,尤其是在lua 的format还不是很成熟的情况下。所以你还是先别改吧。

嗯,就先启用设置,保证以后format的结果

fesily commented 1 year ago

把我搞懵了,package_json.lua文件用的是windows的换行符号,怪不得一format整个文件变了,你是如何打算的,统一使用哪种?

fesily commented 1 year ago

由launcher加载调试器dll的代码也可以包含进来,这很有用。

单纯加载调试器?还是要把导入表修复的部分也加入?

actboy168 commented 1 year ago

只加载就好

actboy168 commented 1 year ago

把我搞懵了,package_json.lua文件用的是windows的换行符号,怪不得一format整个文件变了,你是如何打算的,统一使用哪种?

用lf吧,我记得我的git配置是会强制转为lf的

fesily commented 1 year ago

把我搞懵了,package_json.lua文件用的是windows的换行符号,怪不得一format整个文件变了,你是如何打算的,统一使用哪种?

用lf吧,我记得我的git配置是会强制转为lf的

主干上直接format成lf吧,pull里就不做这个操作了

fesily commented 1 year ago

这个代码也测试过了.reviwe一下

actboy168 commented 1 year ago

那个黑名单,应该比较模块的全路径,确保确实只是过滤掉调试器的模块。或者launcher是可以拿到自己和luadebug的模块handle的,直接对比handle是否一致

fesily commented 1 year ago

那个黑名单,应该比较模块的全路径,确保确实只是过滤掉调试器的模块。或者launcher是可以拿到自己和luadebug的模块handle的,直接对比handle是否一致

先匹配luadebug目录的根路径就行

fesily commented 1 year ago

最近在忙其他的,review咋样,还有需要哪里改进的吗 @actboy168

actboy168 commented 1 year ago

我最近也挺忙的,所以还没空review

fesily commented 1 year ago

我最近也挺忙的,所以还没空review

好的,等你忙完