Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

component matching optimization #295

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from
Open

component matching optimization #295

wants to merge 1 commit into from

Conversation

hiai
Copy link

@hiai hiai commented Feb 19, 2019

组件查找优化,更正了上次的逻辑错误,看下还有没问题

@superqiaopu
Copy link
Collaborator

空行和空格的修改忽略掉,干扰视线。

@superqiaopu
Copy link
Collaborator

你这个优化,思路是好的,用空间换时间,查询性能提高了,但是内存占用比较大。

可以再折中一下,查询时多拼一次key(hostPackageName + "/" + pluginClassName),初始化map时可以省掉一半的缓存。

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