Skip to content

gccrs: refactor default infer vars to be its own function#3806

Merged
philberty merged 1 commit into
masterfrom
phil/refactor-default
May 26, 2025
Merged

gccrs: refactor default infer vars to be its own function#3806
philberty merged 1 commit into
masterfrom
phil/refactor-default

Conversation

@philberty

Copy link
Copy Markdown
Member

This is just a simple refactor to pull all the logic outside of the closure which makes it more readable.

gcc/rust/ChangeLog:

* typecheck/rust-hir-type-check.h: new function
* typecheck/rust-typecheck-context.cc (TypeCheckContext::compute_inference_variables): call the new helper
(TypeCheckContext::compute_infer_var): refactored code

This is just a simple refactor to pull all the logic outside of the
closure which makes it more readable.

gcc/rust/ChangeLog:

	* typecheck/rust-hir-type-check.h: new function
	* typecheck/rust-typecheck-context.cc (TypeCheckContext::compute_inference_variables):
	call the new helper
	(TypeCheckContext::compute_infer_var): refactored code

Signed-off-by: Philip Herron <herron.philip@googlemail.com>
@philberty philberty force-pushed the phil/refactor-default branch from 7a1c15c to fd5045b Compare May 26, 2025 16:40
@philberty philberty added this pull request to the merge queue May 26, 2025
Merged via the queue into master with commit 457b3d1 May 26, 2025
12 checks passed
@github-project-automation github-project-automation Bot moved this from Todo to Done in libcore 1.49 May 26, 2025
@philberty philberty deleted the phil/refactor-default branch June 14, 2025 17:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant