GNU nano - Bugs: bug #64945, with --atblanks, miswrapping can...
You are not allowed to post comments on this tracker with your current authentication level.
bug #64945: with --atblanks, miswrapping can occur with very long words
Submitter: | Andreas Schamanek <aschamanek> | ||
Submitted: | Wed 29 Nov 2023 03:27:01 PM UTC | ||
Severity: | 3 - Normal | Status: | Fixed |
Assigned to: | bens | Open/Closed: | Open |
Jump to the original submission
Fri 08 Dec 2023 11:17:54 AM UTC, comment #6: |
Benno Schulenberg <bens> |
Wed 06 Dec 2023 06:37:40 PM UTC, comment #5: I applied the patch (file #55416) and I found no errors, neither with our 2 minimal examples, nor in real-life testing with existing documents.
|
Andreas Schamanek <aschamanek> |
Wed 06 Dec 2023 03:55:08 PM UTC, comment #4: More economical patch is attached. |
Benno Schulenberg <bens> |
Wed 06 Dec 2023 03:21:49 PM UTC, comment #3: The problem is that, when the 3 has been typed, the space after "makesadozen" is exactly at the 'goal_column' (the column just beyond the right edge of the screen) and thus has already been seen by the get_softwrap_breakpoint() routine. And thus, when this routine is called again for the next chunk, it continues searching for the next breaking point after this space -- which is wrong, of course, when softwrapping at blanks.
|
Benno Schulenberg <bens> |
Wed 06 Dec 2023 09:56:11 AM UTC, comment #2: Transforming the recipe to something that one can copy-paste
Start this nano in an 80-column terminal (for example a default xterm), then type: 1 2 3.
|
Benno Schulenberg <bens> |
Wed 29 Nov 2023 04:36:28 PM UTC, comment #1: Thanks for reporting, and especially for the clear reproduction recipe. I can reproduce the issue.
|
Benno Schulenberg <bens> |
Wed 29 Nov 2023 03:27:01 PM UTC, original submission:
To (hopefully) reproduce use an 80 columns wide terminal. Start nano with `nano --atblanks` and generate a text with the following words all on 1 line:
|
Andreas Schamanek <aschamanek> |
Depends on the following items: None found
Items that depend on this one: None found
There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.
Follow 7 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2023-12-08 | bens | Status | Ready For Test | Fixed | |
Summary | atblanks error wrapping lines with very long words | with --atblanks, miswrapping can occur with very long words | |||
2023-12-06 | bens | Attached File | - | Added 0001-softwrap-remember-the-actual-breaking-point-when-wra.patch, #55416 | |
Status | In Progress | Ready For Test | |||
2023-12-06 | bens | Attached File | - | Added move-back-the-text-pointer-to-the-actual-breaking-point.patch, #55411 | |
Status | None | In Progress | |||
2023-11-29 | bens | Assigned to | None | bens |
Powered by Savane 3.13-02a9.
Corresponding source code
Thanks for testing. The patch has been pushed to master, in commit a76a6bf6.