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

perf: optimize performance #1789

Merged
merged 19 commits into from
Oct 22, 2024
Merged

perf: optimize performance #1789

merged 19 commits into from
Oct 22, 2024

Conversation

Aarebecca
Copy link
Contributor

@Aarebecca Aarebecca commented Oct 16, 2024

标有 ⚠️ 的表示具有 break change,可能对现有依赖造成影响

  1. 移除 enableCSSParsing ⚠️
  2. 清理部分 CSS 解析模块
  3. 优化对象分配以减少 GC
  4. 提供 Fragment 以支持批量插入元素
  5. 优化内存多个内存泄漏点
  6. 移除 G 3.0 相关适配逻辑 ⚠️
  7. 优化部分语法性能
  8. Transform 支持数组格式传入

packages/g-lite/src/css/parser/transform.ts Dismissed Show dismissed Hide dismissed
@Aarebecca Aarebecca force-pushed the perf/optimize-proposal branch 2 times, most recently from 99a1b4a to 01ba274 Compare October 17, 2024 07:25
Comment on lines +143 to +144
!isNaN(rBushNode.aabb.maxX) &&
!isNaN(rBushNode.aabb.maxX) &&

Check warning

Code scanning / CodeQL

Identical operands Warning

Operands
!isNaN( ... b.maxX)
and
!isNaN( ... b.maxX)
are identical.
@Aarebecca Aarebecca changed the title wip: perf optimize proposal perf: optimize performance Oct 22, 2024
@Aarebecca Aarebecca mentioned this pull request Oct 22, 2024
8 tasks
@Aarebecca Aarebecca changed the base branch from next to release October 22, 2024 03:47
@wang1212
Copy link
Member

  1. 移除 enableCSSParsing
  2. 清理部分 CSS 解析模块
  3. 优化对象分配以减少 GC
  4. 提供 Fragment 以支持批量插入元素
  5. 优化内存多个内存泄漏点
  6. 移除 G 3.0 相关适配逻辑
  7. 优化部分语法性能
  8. Transform 支持数组格式传入

建议标记一下 BREAKING CHANGE

@Aarebecca Aarebecca merged commit df950a9 into release Oct 22, 2024
3 checks passed
@Aarebecca Aarebecca deleted the perf/optimize-proposal branch October 22, 2024 11:44
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