lua-debug icon indicating copy to clipboard operation
lua-debug copied to clipboard

add launcher config

Open fesily opened this issue 2 years ago • 16 comments

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

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

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

fesily avatar Mar 17 '23 13:03 fesily

package_json.lua的格式全乱了

actboy168 avatar Mar 17 '23 16:03 actboy168

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

actboy168 avatar Mar 17 '23 17:03 actboy168

package_json.lua的格式全乱了

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

fesily avatar Mar 18 '23 03:03 fesily

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

actboy168 avatar Mar 18 '23 03:03 actboy168

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

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

fesily avatar Mar 18 '23 04:03 fesily

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

fesily avatar Mar 18 '23 06:03 fesily

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

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

fesily avatar Mar 18 '23 07:03 fesily

只加载就好

actboy168 avatar Mar 18 '23 07:03 actboy168

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

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

actboy168 avatar Mar 18 '23 07:03 actboy168

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

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

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

fesily avatar Mar 18 '23 08:03 fesily

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

fesily avatar Mar 25 '23 07:03 fesily

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

actboy168 avatar Mar 25 '23 11:03 actboy168

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

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

fesily avatar Mar 28 '23 01:03 fesily

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

fesily avatar Apr 21 '23 05:04 fesily

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

actboy168 avatar Apr 22 '23 01:04 actboy168

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

好的,等你忙完

fesily avatar Apr 22 '23 01:04 fesily