Skip to content

Commit

Permalink
sam.c: rename a number of options to match lua api
Browse files Browse the repository at this point in the history
  • Loading branch information
rnpnr committed Aug 25, 2023
1 parent 3a93220 commit a35e7ea
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 9 deletions.
8 changes: 4 additions & 4 deletions man/vis.1
Original file line number Diff line number Diff line change
Expand Up @@ -1393,16 +1393,16 @@ section.
.It Cm syntax Op Cm auto
Syntax highlighting lexer to use, name without file extension.
.
.It Cm show-tabs Op Cm off
.It Cm showtabs Op Cm off
Whether to display replacement symbol instead of tabs.
.
.It Cm show-newlines Op Cm off
.It Cm shownewlines Op Cm off
Whether to display replacement symbol instead of newlines.
.
.It Cm show-spaces Op Cm off
.It Cm showspaces Op Cm off
Whether to display replacement symbol instead of blank cells.
.
.It Cm show-eof Op Cm on
.It Cm showeof Op Cm on
Whether to display replacement symbol for lines after the end of the file.
.
.It Cm savemethod Op Ar auto
Expand Down
10 changes: 5 additions & 5 deletions sam.c
Original file line number Diff line number Diff line change
Expand Up @@ -332,22 +332,22 @@ static const OptionDef options[] = {
VIS_HELP("Number of spaces to display (and insert if `expandtab` is enabled) for a tab")
},
[OPTION_SHOW_SPACES] = {
{ "show-spaces" },
{ "showspaces" },
VIS_OPTION_TYPE_BOOL|VIS_OPTION_NEED_WINDOW,
VIS_HELP("Display replacement symbol instead of a space")
},
[OPTION_SHOW_TABS] = {
{ "show-tabs" },
{ "showtabs" },
VIS_OPTION_TYPE_BOOL|VIS_OPTION_NEED_WINDOW,
VIS_HELP("Display replacement symbol for tabs")
},
[OPTION_SHOW_NEWLINES] = {
{ "show-newlines" },
{ "shownewlines" },
VIS_OPTION_TYPE_BOOL|VIS_OPTION_NEED_WINDOW,
VIS_HELP("Display replacement symbol for newlines")
},
[OPTION_SHOW_EOF] = {
{ "show-eof" },
{ "showeof" },
VIS_OPTION_TYPE_BOOL|VIS_OPTION_NEED_WINDOW,
VIS_HELP("Display replacement symbol for lines after the end of the file")
},
Expand Down Expand Up @@ -382,7 +382,7 @@ static const OptionDef options[] = {
VIS_HELP("How to load existing files 'auto', 'read' or 'mmap'")
},
[OPTION_CHANGE_256COLORS] = {
{ "change-256colors" },
{ "change256colors" },
VIS_OPTION_TYPE_BOOL,
VIS_HELP("Change 256 color palette to support 24bit colors")
},
Expand Down

2 comments on commit a35e7ea

@milhnl
Copy link
Contributor

@milhnl milhnl commented on a35e7ea Aug 26, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't this have some sort of backwards compatibility?

@rnpnr
Copy link
Collaborator Author

@rnpnr rnpnr commented on a35e7ea Aug 26, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, that is fair. I think I will come up with a patch that warns about
the old ones being deprecated and that they will be removed in the next
release (i.e. they will remain supported for vis-0.9).

Please sign in to comment.