Skip to content

fix(compiler-cli): generate less type checking code in for loops#53515

Closed
crisbeto wants to merge 1 commit intoangular:mainfrom
crisbeto:for-loop-optional-vars
Closed

fix(compiler-cli): generate less type checking code in for loops#53515
crisbeto wants to merge 1 commit intoangular:mainfrom
crisbeto:for-loop-optional-vars

Conversation

@crisbeto
Copy link
Copy Markdown
Member

The ops for the implicit variables in @for loops (e.g. $index) are marked as being mandatory which means that they're generated even if they aren't used. These changes make them optional so they're only added when necessary.

The ops for the implicit variables in `@for` loops (e.g. `$index`) are marked as being mandatory which means that they're generated even if they aren't used. These changes make them optional so they're only added when necessary.
@crisbeto crisbeto added action: review The PR is still awaiting reviews from at least one requested reviewer target: patch This PR is targeted for the next patch release area: compiler Issues related to `ngc`, Angular's template compiler labels Dec 12, 2023
@crisbeto crisbeto requested a review from atscott December 12, 2023 08:15
@ngbot ngbot bot added this to the Backlog milestone Dec 12, 2023
@JoostK JoostK added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Dec 12, 2023
@alxhub
Copy link
Copy Markdown
Member

alxhub commented Dec 12, 2023

This PR was merged into the repository by commit 9e54569.

alxhub pushed a commit that referenced this pull request Dec 12, 2023
)

The ops for the implicit variables in `@for` loops (e.g. `$index`) are marked as being mandatory which means that they're generated even if they aren't used. These changes make them optional so they're only added when necessary.

PR Close #53515
@alxhub alxhub closed this in 9e54569 Dec 12, 2023
@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 Jan 12, 2024
ChellappanRajan pushed a commit to ChellappanRajan/angular that referenced this pull request Jan 23, 2024
…ular#53515)

The ops for the implicit variables in `@for` loops (e.g. `$index`) are marked as being mandatory which means that they're generated even if they aren't used. These changes make them optional so they're only added when necessary.

PR Close angular#53515
rlmestre pushed a commit to rlmestre/angular that referenced this pull request Jan 26, 2024
…ular#53515)

The ops for the implicit variables in `@for` loops (e.g. `$index`) are marked as being mandatory which means that they're generated even if they aren't used. These changes make them optional so they're only added when necessary.

PR Close angular#53515
amilamen pushed a commit to amilamen/angular that referenced this pull request Jan 26, 2024
…ular#53515)

The ops for the implicit variables in `@for` loops (e.g. `$index`) are marked as being mandatory which means that they're generated even if they aren't used. These changes make them optional so they're only added when necessary.

PR Close angular#53515
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker area: compiler Issues related to `ngc`, Angular's template compiler target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants