Skip to content

Fix bug when inserting text in the middle#305

Merged
dimkanovikov merged 1 commit intostory-apps:masterfrom
iljukhaput:fix/crash-when-inserting
Jul 3, 2025
Merged

Fix bug when inserting text in the middle#305
dimkanovikov merged 1 commit intostory-apps:masterfrom
iljukhaput:fix/crash-when-inserting

Conversation

@iljukhaput
Copy link
Contributor

@iljukhaput iljukhaput commented Jun 27, 2025

У меня (на MSVC) под отладкой постоянно крашится приложение, когда делаю вставку текста в середину документа или когда в поэпизоднике создаю бит и т.п.
Инвалидации итератора тут быть не должно, на сколько я понимаю, но она как будто происходит.
Может быть поправим это?

Fixes #219

@iljukhaput iljukhaput force-pushed the fix/crash-when-inserting branch from 69018d6 to 5ab6689 Compare June 27, 2025 10:19
@dimkanovikov
Copy link
Member

А крашится на какой конкретно строчке?

сейчас мне не нравится, что добавились ещё два цикла, которые могли бы внутри выполняться

@iljukhaput iljukhaput force-pushed the fix/crash-when-inserting branch from 5ab6689 to 3439769 Compare July 1, 2025 09:42
@iljukhaput
Copy link
Contributor Author

Так же закинул сюда коммит с исправлением ещё одного бага, который, например, происходит при нажатии delete перед невидимым битом или при отмене удаления текста с невидимыми блоками

@dimkanovikov
Copy link
Member

Так же закинул сюда коммит с исправлением ещё одного бага, который, например, происходит при нажатии delete перед невидимым битом или при отмене удаления текста с невидимыми блоками

А можешь пожалуйста привести кейс, где это проявляется/как воспроизвести? Не могу понять пока как там могут оказаться итераторы указывающие за конец вектора...

The application crashed in debug mode when inserting text into the
middle of the document, when creating a beat in the treatment, etc.
@iljukhaput iljukhaput force-pushed the fix/crash-when-inserting branch from 3439769 to 272f350 Compare July 3, 2025 04:31
@iljukhaput
Copy link
Contributor Author

А можешь пожалуйста привести кейс, где это проявляется/как воспроизвести? Не могу понять пока как там могут оказаться итераторы указывающие за конец вектора...

Сейчас понял, что проблема уже не воспроизводится после исправления багов режима изоляции.
Убрал этот коммит

@dimkanovikov dimkanovikov merged commit a4952ef into story-apps:master Jul 3, 2025
5 checks passed
@iljukhaput iljukhaput deleted the fix/crash-when-inserting branch September 7, 2025 10:57
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.

Debug assertion failed when pasting text

2 participants