GNU nano - Bugs: bug #54651, removal of formatter makes...
You are not allowed to post comments on this tracker with your current authentication level.
bug #54651: removal of formatter makes per-syntax formatting impossible
Submitter: | Patrick Bogen <pdbogen> | ||
Submitted: | Wed 12 Sep 2018 04:52:14 PM UTC | ||
Severity: | 3 - Normal | Status: | Fixed |
Assigned to: | None | Open/Closed: | Closed |
Jump to the original submission
Mon 14 Oct 2019 09:07:34 AM UTC, comment #9: |
Benno Schulenberg <bens> |
Mon 31 Dec 2018 01:22:02 PM UTC, comment #8: The return of some kind of per-syntax filtering command has been requested in bug #54889. Closing this one. |
Benno Schulenberg <bens> |
Sun 16 Sep 2018 08:36:38 AM UTC, comment #7: Attached patch adds an sbind command to syntaxes, for per-syntax key binds. They only work in the main menu (is there a reason they should work elsewhere?), so there is no menu argument, otherwise it is identical to bind. You can have files of different syntaxes open in buffers and your sbinds automatically update when you switch between them.
|
Brand Huntsman <brand> |
Thu 13 Sep 2018 09:18:10 PM UTC, comment #6: I hope I'm not coming across as argumentative. I've been using nano for a long, long time; and as I lately work in languages (like Go and Terraform) that have strong (and in effect mandatory) formatters, I found the `formatter` functionality extremely useful.
|
Patrick Bogen <pdbogen> |
Thu 13 Sep 2018 06:11:11 PM UTC, comment #5: Apparently you do not understand that the linter command is very special and specific: nano interprets the output of that command as a list of file locations to jump to, and gives the user a special menu to jump between those locations. The formatter command, on the other hand, was basically a per-syntax speller command: it filtered the text through an external command.
|
Benno Schulenberg <bens> |
Wed 12 Sep 2018 06:59:00 PM UTC, comment #4: What I'm thinking about with the former is a genericization of "linter".
|
Patrick Bogen <pdbogen> |
Wed 12 Sep 2018 06:49:28 PM UTC, comment #3: Per-syntax bindable keys are too complicated to implement. And I'm unsure what you with program categories. But, since the formatter command is gone and won't come back, a "program formatter ..." thing will not be possible either. And the linter is already settable per syntax, so there is no need for "program linter ...". |
Benno Schulenberg <bens> |
Wed 12 Sep 2018 05:58:36 PM UTC, comment #2: I have a few ideas for how to address this that aren't "revert the patch to remove formatter" (which I don't really want, anyway).
|
Patrick Bogen <pdbogen> |
Wed 12 Sep 2018 05:31:27 PM UTC, comment #1: True, the formatter command is gone and this it is no longer possible to have a per-syntax specific filtering program. Sorry. But the command will not come back.
|
Benno Schulenberg <bens> |
Wed 12 Sep 2018 04:52:14 PM UTC, original submission:
While it's possible I'm using the feature incorrectly; it appears that removing the `formatter` directive means that I can no longer have a specific formatter paired with a file type, i.e., subordinate to a `syntax` statement.
|
Patrick Bogen <pdbogen> |
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 5 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2019-10-14 | bens | Status | Wont Fix | Fixed | |
2018-12-31 | bens | Open/Closed | Open | Closed | |
2018-09-16 | brand | Attached File | - | Added 0001-add-sbind-command-for-per-syntax-key-binds.patch, #45020 | |
2018-09-13 | bens | Summary | removal of formatter removes per-language formatters | removal of formatter makes per-syntax formatting impossible | |
2018-09-12 | bens | Status | None | Wont Fix |
Powered by Savane 3.13-d3ae.
Corresponding source code
An equivalent feature has been added back in master, commit 34170611.