Skip to content

Commit ecaa673

Browse files
k-ibarakiclaude
andcommitted
docs: clarify merged cell cache comment
AIレビューのフィードバックに対応し、コメントを実装の動作と一致させた。 「merged_rangesを渡す」という誤った説明を、「effective_rangeを渡して キャッシュを構築し、merged_rangesを取得する」という正確な説明に修正。 Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
1 parent bea5940 commit ecaa673

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

src/sharepoint_excel.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -356,8 +356,8 @@ def _parse_sheet(
356356
effective_range_for_merge = effective_range
357357

358358
# マージセル情報をキャッシュ(パフォーマンス最適化)
359-
# 結合セルの場合はmerged_rangesを渡す
360-
# 計算済みのeffective_rangeを渡すことで重複計算を回避
359+
# 計算済みのeffective_range(effective_range_for_merge)を渡してキャッシュを構築し、
360+
# 戻り値としてmerged_ranges(結合セル範囲の一覧)を取得することで重複計算を回避
361361
merged_cell_map, merged_anchor_value_map, merged_ranges = (
362362
self._build_merged_cell_cache(sheet, effective_range_for_merge)
363363
)

0 commit comments

Comments
 (0)