X Tutup
Skip to content

refactor(compiler): speed up proto view merging#3177

Closed
tbosch wants to merge 1 commit intoangular:masterfrom
tbosch:mergeperf
Closed

refactor(compiler): speed up proto view merging#3177
tbosch wants to merge 1 commit intoangular:masterfrom
tbosch:mergeperf

Conversation

@tbosch
Copy link
Copy Markdown
Contributor

@tbosch tbosch commented Jul 21, 2015

  • Don't create intermediate merge results
  • Only merge embedded ProtoViews that contain <ng-content> tags

Closes #3150

@tbosch tbosch added the action: review The PR is still awaiting reviews from at least one requested reviewer label Jul 21, 2015
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

comment: embedded fragment as we need to merge ng-if that contains <ng-content>

- Don't create intermediate merge results
- Only merge embedded ProtoViews that contain `<ng-content>` tags

Closes angular#3150
@tbosch tbosch closed this in 078475a Jul 21, 2015
@tbosch tbosch deleted the mergeperf branch July 21, 2015 21:19
@angular-automatic-lock-bot
Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 6, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: review The PR is still awaiting reviews from at least one requested reviewer cla: yes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Change merging of ProtoViews to be O(N) instead of O(N*DEPTH)

3 participants

X Tutup