Skip to content

Conversation

@stevenlele
Copy link
Contributor

解密重装系统前的数据失败时,增加报错信息。


我本来是想重构一下,让 Master Key 解密失败时也能导出无需解密的数据,不过重构到一半看到你在 24c90f5 (feat/library) 也有重构计划,所以就不弄了,我改到一半的结果在 e623041 (分支),修改点如下:

  • 目前 Browser 的 Chromium/Firefox 多态没什么意义,本来 pickXxx 就是分开的,各个数据项也是分开解析的,所以重构去掉了
  • 把临时文件复制移动到各个数据 parsing 里面,这样就不会留下临时数据;部分没必要复制的就不再复制了
  • 把 Yandex 逻辑合并到 Chromium 里面,减少重复代码
  • 重构不是很彻底,比如可以弄一个 BrowserType 的枚举写进 Browser 里面(看到你弄了),这样更稳健

代码就留着供参考,如果你觉得哪些做法不合适可以指正讨论一下,或者直接无视就好。

@moonD4rk
Copy link
Owner

moonD4rk commented Aug 4, 2023

多谢提交代码,这几天比较忙,下周一开始重构计划,相关进度可以在 projects 这里看到。

@stevenlele
Copy link
Contributor Author

这个 Project 好像没公开,看不到。

@moonD4rk
Copy link
Owner

moonD4rk commented Aug 4, 2023

@stevenlele 为你添加 Project 的权限,Project 会在设计补齐后公开。

@moonD4rk moonD4rk merged commit fa99e85 into moonD4rk:dev Jan 4, 2024
@stevenlele stevenlele deleted the different-os branch January 5, 2024 05:02
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