mirror of https://github.com/pulumi/pulumi.git
571fadae3f
Fixes https://github.com/pulumi/pulumi/issues/12738 https://github.com/pulumi/pulumi/pull/11834 turned on the prealloc linter and changed a load of slice uses from just `var x T[]` to `x := make([]T, 0, preallocSize)`. This was good for performance but it turns out there are a number of places in the codebase that treat a `nil` slice as semnatically different to an empty slice. Trying to test that, or even reason that through for every callsite is untractable, so this PR replaces all expressions of the form `make([]T, 0, size)` with a call to `slice.Prealloc[T](size)`. When size is 0 that returns a nil array, rather than an empty array. |
||
|---|---|---|
| .. | ||
| deploytest | ||
| providers | ||
| builtins.go | ||
| deployment.go | ||
| deployment_executor.go | ||
| deployment_executor_test.go | ||
| deployment_test.go | ||
| doc.go | ||
| import.go | ||
| manifest.go | ||
| plan.go | ||
| snapshot.go | ||
| snapshot_test.go | ||
| source.go | ||
| source_error.go | ||
| source_eval.go | ||
| source_eval_test.go | ||
| source_null.go | ||
| source_query.go | ||
| source_query_test.go | ||
| state_builder.go | ||
| state_builder_test.go | ||
| step.go | ||
| step_executor.go | ||
| step_generator.go | ||
| step_generator_test.go | ||
| target.go | ||