Mercurial > vim
annotate runtime/syntax/squirrel.vim @ 32876:522f16e3e058 v9.0.1747
patch 9.0.1747: screenpos() may cause unnecessary redraw
Commit: https://github.com/vim/vim/commit/6235a109c48ff2559eca3b16578c429ffb61eadc
Author: zeertzjq <zeertzjq@outlook.com>
Date: Sat Aug 19 14:12:42 2023 +0200
patch 9.0.1747: screenpos() may cause unnecessary redraw
Problem: screenpos() may cause unnecessary redraw.
Solution: Don't unnecessarily reset VALID_WROW flag.
VALID_WROW flag is only used by two functions: validate_cursor() and
cursor_valid(), and cursor_valid() is only used once in ex_sleep().
When adjust_plines_for_skipcol() was first added in patch 9.0.0640, it
was called in two functions: comp_botline() and curs_rows().
- comp_botline() is called in two places:
- onepage(), which resets VALID_WROW flag immediately afterwards.
- validate_botline_win(), where resetting a VALID_ flag is strange.
- curs_rows() is called in two places:
- curs_columns(), which sets VALID_WROW flag afterwards.
- validate_cline_row(), which is only used by GUI mouse focus.
Therefore resetting VALID_WROW there doesn't seem to do anything useful.
Also, a w_skipcol check (which resets VALID_WROW flag) was added to
check_cursor_moved() in patch 9.0.0734, which seems to make more sense
than resetting that flag in the middle of a computation.
While at it make adjust_plines_for_skipcol() and textpos2screenpos() a
bit less confusing:
- Make adjust_plines_for_skipcol() return "off" instead of "n - off".
- Use 0-based "row" in textpos2screenpos() until W_WINROW is added.
closes: #12832
Signed-off-by: Christian Brabandt <cb@256bit.org>
Co-authored-by: zeertzjq <zeertzjq@outlook.com>
| author | Christian Brabandt <cb@256bit.org> |
|---|---|
| date | Sat, 19 Aug 2023 14:30:02 +0200 |
| parents | c725b8e17f1f |
| children | c62bca3b8d82 |
| rev | line source |
|---|---|
| 26438 | 1 " Vim syntax file |
| 2 " Language: squirrel | |
| 3 " Current Maintainer: Matt Dunford (zenmatic@gmail.com) | |
| 4 " URL: https://github.com/zenmatic/vim-syntax-squirrel | |
| 5 " Last Change: 2021 Nov 28 | |
| 6 | |
| 7 " http://squirrel-lang.org/ | |
| 8 | |
| 9 " quit when a syntax file was already loaded | |
| 10 if exists("b:current_syntax") | |
| 11 finish | |
| 12 endif | |
| 13 | |
| 14 " inform C syntax that the file was included from cpp.vim | |
| 15 let b:filetype_in_cpp_family = 1 | |
| 16 | |
| 17 " Read the C syntax to start with | |
| 18 runtime! syntax/c.vim | |
| 19 unlet b:current_syntax | |
| 20 | |
| 21 " squirrel extensions | |
| 22 syn keyword squirrelStatement delete this in yield resume base clone | |
| 23 syn keyword squirrelAccess local | |
| 24 syn keyword cConstant null | |
| 25 syn keyword squirrelModifier static | |
| 26 syn keyword squirrelType bool instanceof typeof | |
| 27 syn keyword squirrelExceptions throw try catch | |
| 28 syn keyword squirrelStructure class function extends constructor | |
| 29 syn keyword squirrelBoolean true false | |
| 30 syn keyword squirrelRepeat foreach | |
| 31 | |
| 32 syn region squirrelMultiString start='@"' end='"$' end='";$'me=e-1 | |
| 33 | |
| 34 syn match squirrelShComment "^\s*#.*$" | |
| 35 | |
| 36 " Default highlighting | |
| 37 hi def link squirrelAccess squirrelStatement | |
| 38 hi def link squirrelExceptions Exception | |
| 39 hi def link squirrelStatement Statement | |
| 40 hi def link squirrelModifier Type | |
| 41 hi def link squirrelType Type | |
| 42 hi def link squirrelStructure Structure | |
| 43 hi def link squirrelBoolean Boolean | |
| 44 hi def link squirrelMultiString String | |
| 45 hi def link squirrelRepeat cRepeat | |
| 46 hi def link squirrelShComment Comment | |
| 47 | |
| 48 let b:current_syntax = "squirrel" | |
| 49 | |
| 50 " vim: ts=8 |