d0f52751ce
The indent trigger was using .abs() which fired on both increased indent (non-indented → indented) AND decreased indent (indented → non-indented). This caused drop-cap style paragraphs (indented first line, flush-left continuation) to incorrectly split into two blocks. Per plan Phase 4.4 heuristic #2, indent change should only trigger when the current line is MORE indented (to the right, larger x0) than the block average - i.e., a new paragraph starting after non-indented text. It should NOT trigger for decreased indent (first line indented, rest flush-left). Fix: Remove .abs() and only check if line_x0 - block_avg_x0 > threshold. Tests: - test_indented_first_line_new_block: PASS (non-indented → indented splits) - test_indented_first_line_of_paragraph_not_split: PASS (drop cap stays together) - All 179 line module tests: PASS
31 lines
1.1 KiB
Python
31 lines
1.1 KiB
Python
#!/usr/bin/env python3
|
|
"""Check fixture content streams."""
|
|
import pikepdf
|
|
import sys
|
|
|
|
if len(sys.argv) > 1:
|
|
path = sys.argv[1]
|
|
pdf = pikepdf.open(path)
|
|
page = pdf.pages[0]
|
|
content = page.Contents.get_stream_bytes() if hasattr(page.Contents, 'get_stream_bytes') else page.Contents.getbytes()
|
|
print(f"{path}: {content}")
|
|
else:
|
|
# Check both content edit fixtures
|
|
paths = [
|
|
'tests/fingerprint/fixtures/content_edit_one_glyph/v1.pdf',
|
|
'tests/fingerprint/fixtures/content_edit_one_glyph/v2.pdf',
|
|
'tests/fingerprint/fixtures/content_edit_one_paragraph/v1.pdf',
|
|
'tests/fingerprint/fixtures/content_edit_one_paragraph/v2.pdf',
|
|
]
|
|
|
|
for path in paths:
|
|
try:
|
|
pdf = pikepdf.open(path)
|
|
page = pdf.pages[0]
|
|
if hasattr(page.Contents, 'get_stream_bytes'):
|
|
content = page.Contents.get_stream_bytes()
|
|
else:
|
|
content = page.Contents.getbytes()
|
|
print(f"{path}: {content}")
|
|
except Exception as e:
|
|
print(f"{path}: ERROR - {e}")
|