Skip to content

Commit 9b4cee1

Browse files
authored
Merge pull request #783 from unoplatform/dev/jela/perf-update
perf: Use parallel file copy
2 parents fc30f86 + a3ea647 commit 9b4cee1

File tree

1 file changed

+19
-16
lines changed

1 file changed

+19
-16
lines changed

src/Uno.Wasm.Packager/packager.cs

+19-16
Original file line numberDiff line numberDiff line change
@@ -833,24 +833,26 @@ int Run (string[] args) {
833833
Directory.CreateDirectory (builddir);
834834
}
835835

836-
if (!emit_ninja) {
837-
if (!Directory.Exists (out_prefix))
838-
Directory.CreateDirectory (out_prefix);
839-
var bcl_dir = Path.Combine (out_prefix, assembly_root);
840-
if (Directory.Exists (bcl_dir))
841-
Directory.Delete (bcl_dir, true);
842-
Directory.CreateDirectory (bcl_dir);
843-
foreach (var f in file_list) {
844-
836+
if (!emit_ninja)
837+
{
838+
if (!Directory.Exists(out_prefix))
839+
Directory.CreateDirectory(out_prefix);
840+
var bcl_dir = Path.Combine(out_prefix, assembly_root);
841+
if (Directory.Exists(bcl_dir))
842+
Directory.Delete(bcl_dir, true);
843+
Directory.CreateDirectory(bcl_dir);
844+
845+
file_list.AsParallel().ForAll(f =>
846+
{
845847
var fileName = Path.GetFileName(f);
846848

847-
if(IsResourceAssembly(f, out var culture))
849+
if (IsResourceAssembly(f, out var culture))
848850
{
849851
fileName = Path.Combine(culture, fileName);
850852
}
851853

852-
CopyFile(f, Path.Combine (bcl_dir, fileName), copyType);
853-
}
854+
CopyFile(f, Path.Combine(bcl_dir, fileName), copyType);
855+
});
854856
}
855857

856858
if (assembly_root.EndsWith ("/"))
@@ -1042,10 +1044,11 @@ int Run (string[] args) {
10421044
Path.Combine (out_prefix, fname));
10431045
}
10441046

1045-
foreach(var asset in assets) {
1046-
CopyFile (asset,
1047-
Path.Combine (out_prefix, Path.GetFileName (asset)), copyType, "Asset: ");
1048-
}
1047+
assets.AsParallel().ForAll(asset =>
1048+
{
1049+
CopyFile(asset,
1050+
Path.Combine(out_prefix, Path.GetFileName(asset)), copyType, "Asset: ");
1051+
});
10491052
}
10501053

10511054
if (!emit_ninja)

0 commit comments

Comments
 (0)