Add support for arrays in SetBlockList #102
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
此次修改的时候，只保留了 blockListCompiled 和 ipBlockListCompiled ，然后把config.json，文件或者链接中的内容全部解析到一个位置 |
|
直接 content = StrTrim(ProcessRemark(content)) 吗？ |
|
已经修复 用的时候还遇到一件事，URL多了之后卡了一会，发现URL超时了。现在给URL加了个并发化 |
|
请求审查 |
|
谢谢, 已经合并. 尽管大致上没有问题了, 但有一点想法: 为 URL 引入并发的复杂性是否值得. 考虑到 URL Interval 一般都大于 300 秒, 一段时间的超时应当是可以容忍的, 特别是在首次获取完成有本地列表后, 只需要为这一动作加入异步和防止重复获取的简单锁即可. 但这不对合并构成影响. |
Simple-Tracker
changed the title
|
你不说，我都忘了给map换成线程安全的那一种了 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
使得 SetBlockFromFile SetBlockFromURL SetIPBlockFromFile SetIPBlockFromURL 支持数组