Remove the startup pass that folds files WordPress listed twice #83

Closed
opened 2026-09-29 06:06:57 -07:00 by rays · 1 comment
Owner

merge_repeated_enclosures (src/db.rs:1697), its call at startup (src/main.rs:425) and its test, about 115 lines. It cleaned up files downloaded twice before the parser knew WordPress's numbered player URLs; the parser now drops the repeats while reading (feed.rs:727), and production has run the pass, so it only costs a query on every start. Found in a repo-wide audit for code that can go.

merge_repeated_enclosures (src/db.rs:1697), its call at startup (src/main.rs:425) and its test, about 115 lines. It cleaned up files downloaded twice before the parser knew WordPress's numbered player URLs; the parser now drops the repeats while reading (feed.rs:727), and production has run the pass, so it only costs a query on every start. Found in a repo-wide audit for code that can go.
rays added the enhancement label 2026-09-29 06:06:57 -07:00
Author
Owner

Fixed in 8c5eddd: merge_repeated_enclosures, its start-up call and its test are gone; the parser still drops the repeats as it reads. Not deployed yet.

Fixed in 8c5eddd: merge_repeated_enclosures, its start-up call and its test are gone; the parser still drops the repeats as it reads. Not deployed yet.
rays closed this issue 2026-09-29 06:12:12 -07:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: rays/ipx#83