Task #1134: HWPX 본문·표 셀 문단 id 전역 유니크 처리#1222
Merged
Merged
Conversation
…id> in table cells - cell_para_ids_continue_from_context_counter: ctx 카운터 오프셋 N에서 셀 문단 id가 N부터 연속 발급되는지 검증 (본문·표 전역 공유 설계) - two_sequential_tables_have_no_para_id_collision: 같은 ctx로 표 두 개 직렬화 시 카운터가 초기화되지 않음을 확인 (회귀 방지) - multi_para_cells_all_get_unique_ids: 셀당 복수 문단(3개)이 있을 때도 전역 고유성이 보장되는지 확인
edwardkim
added a commit
that referenced
this pull request
Jun 1, 2026
@Mireutale #1206 분리 재제출. 1921 passed, fillBrush 제외 확인. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Owner
|
머지했습니다(devel 첫 기여이심에도 #1206 리뷰 의견을 정확히 반영해 주셨습니다 — fillBrush 직렬화를 분리하고 문단 id 충돌 수정만 깔끔하게 재제출해 주신 덕분에 변경 범위가 명확해져 검토가 한결 수월했습니다. 안내를 진지하게 받아들이고 다시 PR 을 보내주신 점에 특히 감사드립니다.
검증:
앞으로의 기여도 기대하겠습니다. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
변경 요약
HWPX 직렬화 시 본문 문단과 표 셀 내부 문단의
<hp:p id>가 같은 번호 공간을 공유하지 않아 전역 중복이 발생할 수 있는 문제(#1134)를 수정했습니다.SerializeContext에 문서 전역 문단 id 카운터(next_para_id())를 추가basic-table-01.hwpx라운드트립에서 본문+셀 id 전역 유니크 검증분리 안내 (PR #1206 후속)
닫힌 PR #1206 검토 의견에 따라, fillBrush 직렬화 변경(
serializer/hwpx/header.rs)을 제외하고 문단 id 충돌 수정만 분리하여 재제출합니다. fillBrush는 HWPX 쓰기 모듈 정비 일정에 맞춰 별도로 다루는 것으로 이해했습니다.변경 파일은 4개입니다:
serializer/hwpx/context.rs,serializer/hwpx/section.rs,serializer/hwpx/table.rs,tests/hwpx_roundtrip_integration.rs.관련 이슈
closes #1134
테스트
cargo fmt --all -- --check통과cargo test통과 (HWPX serializer의 정보 누락 관련 제안 #1134 단위 11종 + 통합 1종 포함)cargo clippy -- -D warnings통과스크린샷
직렬화 XML의
<hp:p id>발급 로직 변경이라 시각적 변경이 없어 스크린샷 비교는 생략합니다.