Skip to content

add ltk_troybin: INIBIN v1/v2 (troybin) parser and writer#118

Closed
FrogCsLoL wants to merge 1 commit intoLeagueToolkit:mainfrom
FrogCsLoL:add-ltk-troybin
Closed

add ltk_troybin: INIBIN v1/v2 (troybin) parser and writer#118
FrogCsLoL wants to merge 1 commit intoLeagueToolkit:mainfrom
FrogCsLoL:add-ltk-troybin

Conversation

@FrogCsLoL
Copy link
Contributor

Full round-trip support for the .troybin particle config format:

  • Binary read for v1 (old) and v2 (new) formats
  • Binary write for v2 with all 14 block types
  • INI text read/write for human-editable representation
  • Hash dictionary with all known property names
  • StorageType preserved per entry for lossless round-trip

Format based on Leischii's TroybinConverter and Rey's TroybinEditor.

@Crauzer Crauzer requested review from Crauzer and alanpq March 25, 2026 12:11
@Crauzer Crauzer added the area:api Public API design label Mar 25, 2026
Complete read/write support for League of Legends troybin particle files.

- Binary reader for INIBIN v1 (old format) and v2 (typed blocks)
- Binary writer for v2 format with all 14 storage types
- INI text reader/writer for human-readable editing
- Hash dictionary for resolving property names (based on Leischii's
  TroybinConverter and Rey's TroybinEditor, both MIT licensed)
- Lossless binary round-trip via StorageType preservation

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@alanpq
Copy link
Contributor

alanpq commented Mar 25, 2026

Thanks for the PR and the effort put into it. At this stage it doesn’t meet the bar for merging, and addressing the gaps would likely require more review cycles than implementing the crate separately, so I’m going to close this.

@alanpq alanpq closed this Mar 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:api Public API design

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants