Skip to content

Preserve Glacier RFL chunks through Alpine RED saves#392

Open
GooberRF wants to merge 3 commits into
masterfrom
glacierchunks
Open

Preserve Glacier RFL chunks through Alpine RED saves#392
GooberRF wants to merge 3 commits into
masterfrom
glacierchunks

Conversation

@GooberRF

Copy link
Copy Markdown
Owner
  • Retain Glacier-specific chunks when RFLs loaded and re-saved in level editor

@GooberRF
GooberRF requested a review from Copilot July 15, 2026 19:05
@GooberRF
GooberRF marked this pull request as ready for review July 15, 2026 19:06

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates AlpineEditor’s RFL load/save path to preserve Glacier-specific (unknown) RFL chunks by capturing them verbatim during load and re-emitting them during save, so Glacier-authored metadata isn’t lost when round-tripping through Alpine RED.

Changes:

  • Add identification helpers and an in-memory container (retained_chunks) for Glacier-specific RFL chunks.
  • Hook RFL chunk loading to capture 0x6ED***** chunk payloads verbatim and hook saving to write them back out.
  • Document the behavior change in the changelog.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
editor_patch/level.h Adds Glacier chunk ID detection helpers and storage (RetainedRflChunk, retained_chunks) on AlpineLevelProperties.
editor_patch/level.cpp Implements retained-chunk deserialize/serialize and wires it into the existing load/save injections.
docs/CHANGELOG.md Notes Glacier chunk retention in the upcoming release changelog.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread editor_patch/level.cpp
Comment thread editor_patch/level.cpp

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

Comment thread editor_patch/level.cpp
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