GNU nano - Patches: patch #9653, docs: mention that "Ins"...
You are not allowed to post comments on this tracker with your current authentication level.
patch #9653: docs: mention that "Ins" and "Del" are rebindable, and fix nano.texi error
Submitter: | David Lawrence Ramsey <dolorous> | ||
Submitted: | Sat 09 Jun 2018 05:11:42 PM UTC | ||
Priority: | 3 - Low | Status: | Done |
Privacy: | Public | Assigned to: | dolorous |
Open/Closed: | Closed | Release: | None |
Jump to the original submission
Sun 23 Sep 2018 06:33:05 AM UTC, comment #8: |
Benno Schulenberg <bens> |
Wed 11 Jul 2018 12:49:57 PM UTC, comment #7: Thanks. Applied to master, with the addition of @ and ], commit 4c023292. |
Benno Schulenberg <bens> |
Tue 10 Jul 2018 03:37:23 PM UTC, comment #6: Okay. Updated version against git 0782654 attached.
|
David Lawrence Ramsey <dolorous> |
Tue 10 Jul 2018 01:03:16 PM UTC, comment #5: Hmm... Still not good enough. The description for ^ is too broad, and the one for M- too narrow. For example, ^# is not valid, but M-# is; ^% is not valid, but M-% is. In fact, all ASCII characters except ] and a space itself are valid after M-, but after ^ only a few symbols and some digits are valid.
|
Benno Schulenberg <bens> |
Mon 09 Jul 2018 05:43:18 PM UTC, comment #4: Okay. Taking all this into account, I've attached an improved patch against git f456794.
|
David Lawrence Ramsey <dolorous> |
Mon 09 Jul 2018 08:18:24 AM UTC, comment #3: (Okay, no hurry.)
|
Benno Schulenberg <bens> |
Sun 08 Jul 2018 09:04:46 PM UTC, comment #2: With that in mind, "ASCII character" and "Latin letter" do seem to be better descriptions, yes. The only problem was that it should be "an ASCII character" instead of "a ASCII character". As for the other cases, I'm away from my development machine at the moment, but I'll look into them once I'm back in front of it (which should be tomorrow or so). |
David Lawrence Ramsey <dolorous> |
Sat 07 Jul 2018 06:28:26 PM UTC, comment #1: Thanks for the patch. But, as posted on bug #54071, the current descriptions don't match ^\ (plus some others) either. That would need to be fixed too. Plus: what is a printable character? Any Chinese character is a printable character too, but is not acceptable there. So I think the "ASCII character" is the better description in this case. And instead of "alpha character" it would be more precise to say "Latin letter". What do you think? |
Benno Schulenberg <bens> |
Sat 09 Jun 2018 05:11:42 PM UTC, original submission:
Attached, against git 9a7ba5d.
|
David Lawrence Ramsey <dolorous> |
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 8 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2018-09-23 | bens | Open/Closed | Open | Closed | |
2018-07-11 | bens | Priority | 5 - Normal | 3 - Low | |
Status | In Progress | Done | |||
2018-07-10 | dolorous | Attached File | - | Added 0001-docs-mention-that-Ins-and-Del-are-valid-rebindable-k.patch, #44536 | |
2018-07-10 | bens | Status | None | In Progress | |
Assigned to | None | dolorous | |||
2018-07-09 | dolorous | Attached File | - | Added 0001-docs-mention-that-Ins-and-Del-are-valid-rebindable-k.patch, #44528 | |
2018-06-09 | dolorous | Attached File | - | Added 0001-docs-mention-that-Ins-and-Del-are-valid-rebindable-k.patch, #44339 |
Powered by Savane 3.13-f8d8.
Corresponding source code
Was released in 3.0.