notepad-plus-plus / notepad-plus-plus/notepad-plus-plus

Search result lexer assert problem in debug mode

Open
#9,207 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
C++
Stars
29.4k
Forks
5.4k
PR merge metrics
No merged PRs in 30d

Description

Steps to Reproduce the Issue
  1. Build N++ code from source; the 64-bit Debug config.
  2. Place the two supplied files in the root folder of a drive, after unzipping from the attached zip file below
  3. Make sure they are the only two .txt files in that folder
  4. Start N++ under the debugger
  5. Run the Find All the search shown in the screenshot below
  6. Observe the successful output in the Search results window
  7. Run the same Find All search again
  8. Observe the "Assertion failed!" popup box.
Expected Behavior

No assert crash

Actual Behavior

Assert crash

Notes

This issue has its roots here: https://github.com/notepad-plus-plus/notepad-plus-plus/issues/9199#issuecomment-734600679

Files: files.zip

Screenshot

image

Debug Information

Code built from version 7.9.1 or later intermediate commits.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Build Notepad++ from source in the 64-bit Debug configuration, place the two supplied files in a drive-root folder, and reproduce the issue by running the same Find All search twice under the debugger. Done means the repeated search completes successfully without the assertion failure.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
search
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.