Ozaki Kiichi wrote: > > From your description, this option behaves like a boolean. Why > > would you use the numerical values 1 and -1 instead? > > Sorry for lack of a description about 'cursorlinenr=0'. I have fixed it. > > That is, 'cursorlinenr' is set to: > > 1: highlighting the line number > 0: no highlighting > -1: highlighting depends on 'cursorline' or 'relativenumber' > (this is current behavior.) > > In the previous patch, type of 'cursorlinenr' was boolean. > But it must set default value to other than 'on' and 'off', i.e. 'unset' (-1) > for the > compatibility with current behavior, and I was not sure whether 'unset' can be > permitted. > (and changing value 'unset' to 'on' or 'off' is irreversible)
Using the magic numbers is hard to understand. New options are probably better done with string values, such as "on", "off", and empty for not set. Perhaps in this case "number", "line" and "both"? But then naming it 'cursorlineopt' would be better. -- "I can't complain, but sometimes I still do." (Joe Walsh) /// Bram Moolenaar -- b...@moolenaar.net -- http://www.Moolenaar.net \\\ /// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\ \\\ an exciting new programming language -- http://www.Zimbu.org /// \\\ help me help AIDS victims -- http://ICCF-Holland.org /// -- -- You received this message from the "vim_dev" maillist. Do not top-post! Type your reply below the text you are replying to. For more information, visit http://www.vim.org/maillist.php --- You received this message because you are subscribed to the Google Groups "vim_dev" group. To unsubscribe from this group and stop receiving emails from it, send an email to vim_dev+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.