Skip to content

OCTO-11626 - #437

Merged
OlteanuRares merged 1 commit into
mainfrom
OCTO-11626
Oct 6, 2026
Merged

OlteanuRares merged 1 commit into
mainfrom
OCTO-11626

Conversation

@OlteanuRares

Copy link
Copy Markdown
Contributor

CaptionLineLengthError lines now end with a fixed suffix when the long line came from text that kept going on its row:

around 00:12:01.400 - <text> - Length 58 (continues on the row without a row change)

The suffix is the public constant pycaption.scc.reader.ROW_CONTINUATION_SUFFIX. Skylab will parse it (OCTO-11625), so the wording is part of the API. When an indent is reported, the suffix comes after the indent clause (... plus a K-column indent = M (continues ...)).

The flag is set when text lands on a row that already has text and nothing moved it to a new row:

  • New file line: a line of the file writes text before any positioning command.
  • Same-row PAC: a PAC on the row being written moves the cursor along that row.
  • Text before any PAC: text is sent before the first PAC, then a PAC names the row that text already landed on.

It is cleared:

  • after a row change, a repositioning or a Carriage Return;
  • when a PAC sends the row again in full;
  • on rows that fit, when simulated roll-up joins them past 32.

@OlteanuRares
OlteanuRares requested a review from a team as a code owner October 5, 2026 12:26
@github-actions

github-actions Bot commented Oct 5, 2026

Copy link
Copy Markdown

🟢 PR Compliance Review

Risk Level: LOW

  • Compliance Issues: 0 (0 critical)
  • Regressions: 0

SAFE TO MERGE - No critical issues found

Full report available in workflow artifacts

@OlteanuRares
OlteanuRares merged commit 5918c6e into main Oct 6, 2026
4 checks passed
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.

3 participants