Skip to content

arp: fixes to implementation#18

Merged
soypat merged 1 commit intosoypat:mainfrom
egonelbre:fix-arp
Jan 12, 2026
Merged

arp: fixes to implementation#18
soypat merged 1 commit intosoypat:mainfrom
egonelbre:fix-arp

Conversation

@egonelbre
Copy link
Copy Markdown
Contributor

  • typo in error text
  • compactQueries was keeping invalid queries
  • compactQueries accidentally ended up creating shared slices

* typo in error text
* compactQueries was keeping invalid queries
* compactQueries accidentally ended up creating shared slices
@egonelbre
Copy link
Copy Markdown
Contributor Author

PS: AI Disclosure, I was testing AI tool capabilities and ran it against arp package.

@codecov
Copy link
Copy Markdown

codecov bot commented Jan 12, 2026

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@soypat
Copy link
Copy Markdown
Owner

soypat commented Jan 12, 2026

Impressive find. I suspect there may be more near identical bugs since I was completely unaware of the danger of assignment of referential types during compaction :s

@soypat soypat merged commit be2d380 into soypat:main Jan 12, 2026
3 checks passed
@soypat
Copy link
Copy Markdown
Owner

soypat commented Jan 12, 2026

Thank you for your contribution Egon!

@egonelbre egonelbre deleted the fix-arp branch January 12, 2026 21:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants