Skip to content

#1646 - AlignedString cannot insert at the end of the data - #1647

Merged
reckart merged 1 commit into
mainfrom
bugfix/1646-AlignedString-cannot-insert-at-the-end-of-the-data
Aug 9, 2026
Merged

#1646 - AlignedString cannot insert at the end of the data#1647
reckart merged 1 commit into
mainfrom
bugfix/1646-AlignedString-cannot-insert-at-the-end-of-the-data

Conversation

@reckart

@reckart reckart commented Aug 9, 2026

Copy link
Copy Markdown
Member

What's in the PR

  • Fix AlignedString.insert() to support appending at the end of data via special case for pos == length()
  • Add regression tests for append operations: testInsertAtEnd, testInsertAtEndOnBase, testReplaceAtEnd, testInsertAtEndRepeatedly, testInsertAtEndAfterDeletingTheEnd, testInsertBeyondEndFails, testResolveAfterInsertAtEnd

How to test manually

  • See issue description

Automatic testing

  • PR includes unit tests

Documentation

  • PR updates documentation

- Fix AlignedString.insert() to support appending at the end of data via special case for pos == length()
- Add regression tests for append operations: testInsertAtEnd, testInsertAtEndOnBase, testReplaceAtEnd, testInsertAtEndRepeatedly, testInsertAtEndAfterDeletingTheEnd, testInsertBeyondEndFails, testResolveAfterInsertAtEnd
@reckart reckart added this to the 3.0.0 milestone Aug 9, 2026
@reckart reckart self-assigned this Aug 9, 2026
@reckart reckart added this to Kanban Aug 9, 2026
@reckart reckart added 🐛Bug Something isn't working Module-api.transform labels Aug 9, 2026
@github-project-automation github-project-automation Bot moved this to In progress in Kanban Aug 9, 2026
@reckart
reckart merged commit 462c8bc into main Aug 9, 2026
5 checks passed
@reckart
reckart deleted the bugfix/1646-AlignedString-cannot-insert-at-the-end-of-the-data branch August 9, 2026 12:01
@github-project-automation github-project-automation Bot moved this from In progress to Done in Kanban Aug 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛Bug Something isn't working Module-api.transform

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant