Skip to content

Commit f57f6a0

Browse files
committed
Use map for lookups in compareDirs
1 parent 7bb0612 commit f57f6a0

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

web/src/routes/diff/LoadDiffDialog.svelte

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -132,12 +132,13 @@
132132
return !dirBlacklistRegexes.some((pattern) => pattern.test(entry.path));
133133
};
134134
const entriesA: ProtoFileDetails[] = flatten(dirA).filter(blacklist);
135+
const entriesAMap = new Map(entriesA.map((entry) => [entry.path, entry]));
135136
const entriesB: ProtoFileDetails[] = flatten(dirB).filter(blacklist);
137+
const entriesBMap = new Map(entriesB.map((entry) => [entry.path, entry]));
136138
137139
const fileDetails: FileDetails[] = [];
138-
139140
for (const entry of entriesA) {
140-
const entryB = entriesB.find((e) => e.path === entry.path);
141+
const entryB = entriesBMap.get(entry.path);
141142
if (entryB) {
142143
// File exists in both directories
143144
const [aBinary, bBinary] = await Promise.all([isBinaryFile(entry.file), isBinaryFile(entryB.file)]);
@@ -198,7 +199,7 @@
198199
199200
// Check for added files
200201
for (const entry of entriesB) {
201-
const entryA = entriesA.find((e) => e.path === entry.path);
202+
const entryA = entriesAMap.get(entry.path);
202203
if (!entryA) {
203204
if (isImageFile(entry.file.name)) {
204205
fileDetails.push({

0 commit comments

Comments
 (0)