aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge branch 'release/v0.5.0'v0.5.0Tom Ryder2017-11-0425-83/+96
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * release/v0.5.0: (25 commits) Bump version number to 0.5.0 Update documentation to reflect ftplugin changes Add lint mapping for Vimscript Specify scope of mapleader variables Use underscore as local map leader Add check and lint mappings for shell script Add tidy mapping for HTML Break long lines in check/lint/tidy mappings Make all lint/check/tidy maps local and silent Improve comments on check/lint/tidy maps Use long form options for tidy(1) Vim call Use direct :write !cmd instead of shellescape() Use full ':execute' not just ':exe' in VimL Check for availability of Vim shellescape() Revert "Adjust UrlLink() to yank word without t... Adjust UrlLink() to yank word without text objects Refactor UrlLink() function normal! commands Use single quotes for HTML link mapping :execute Refactor HTML tidy(1) mapping Use <Leader>/<LocalLeader> correctly in Vim config ...
| * Bump version number to 0.5.0Tom Ryder2017-11-041-2/+2
| |
| * Merge branch 'feature/review-ftplugin' into developTom Ryder2017-11-046-13/+66
| |\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * feature/review-ftplugin: Update documentation to reflect ftplugin changes Add lint mapping for Vimscript Specify scope of mapleader variables Use underscore as local map leader Add check and lint mappings for shell script Add tidy mapping for HTML Break long lines in check/lint/tidy mappings Make all lint/check/tidy maps local and silent Improve comments on check/lint/tidy maps Use long form options for tidy(1) Vim call Use direct :write !cmd instead of shellescape() Use full ':execute' not just ':exe' in VimL Check for availability of Vim shellescape() Revert "Adjust UrlLink() to yank word without t... Adjust UrlLink() to yank word without text objects Refactor UrlLink() function normal! commands Use single quotes for HTML link mapping :execute Refactor HTML tidy(1) mapping
| | * Update documentation to reflect ftplugin changesTom Ryder2017-11-041-2/+6
| | |
| | * Add lint mapping for VimscriptTom Ryder2017-11-041-0/+5
| | | | | | | | | | | | Runs `vint -s`; the -s includes stylistic suggestions.
| | * Specify scope of mapleader variablesTom Ryder2017-11-041-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `vint -s` says: vim/config/leader.vim:2:5: Make the scope explicit like `g:mapleader` (see Anti-pattern of vimrc (Scope of identifier)) vim/config/leader.vim:3:5: Make the scope explicit like `g:maplocalleader` (see Anti-pattern of vimrc (Scope of identifier)) This does still seem to work with the prefixes, despite not being the way the documentation specifies the variables.
| | * Use underscore as local map leaderTom Ryder2017-11-041-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This should allow me to mentally separate actions specific to a buffer type from actions that apply to buffers in general. It also removes the overlap of <leader>l for 'list' toggling and filetype linting. From ":help maplocalleader": > <LocalLeader> is just like <Leader>, except that it uses > "maplocalleader" instead of "mapleader". <LocalLeader> is to be used > for mappings which are local to a buffer. Example: > > :map <buffer> <LocalLeader>A oanother line<Esc> > > In a global plugin <Leader> should be used and in a filetype plugin > <LocalLeader>. "mapleader" and "maplocalleader" can be equal. > Although, if you make them different, there is a smaller chance of > mappings from global plugins to clash with mappings for filetype > plugins. For example, you could keep "mapleader" at the default > backslash, and set "maplocalleader" to an underscore.
| | * Add check and lint mappings for shell scriptTom Ryder2017-11-041-0/+22
| | | | | | | | | | | | | | | The commands to use in this case are dependent on the particular shell being used.
| | * Add tidy mapping for HTMLTom Ryder2017-11-041-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This mapping mirrors the one for Perl that passes the content of the buffer through a program to tidy it (i.e. not merely check but actively change it). The tidy(1) option chosen here, -quiet, is the bare minimum to make this invocation useful. We would never want the boilerplate it otherwise emits to be in the buffer after a call. Everything else should be applied in a configuration file, which I'll do in a separate feature.
| | * Break long lines in check/lint/tidy mappingsTom Ryder2017-11-042-5/+12
| | |
| | * Make all lint/check/tidy maps local and silentTom Ryder2017-11-042-5/+5
| | | | | | | | | | | | | | | | | | That is, apply <buffer> and <silent> to each of them, to make them only apply to the current buffer and to prevent them from echoing the command they're running.
| | * Improve comments on check/lint/tidy mapsTom Ryder2017-11-032-4/+4
| | |
| | * Use long form options for tidy(1) Vim callTom Ryder2017-11-031-2/+2
| | |
| | * Use direct :write !cmd instead of shellescape()Tom Ryder2017-11-032-17/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | This is a much better method of calling external programs on the buffer's contents, not just because it avoids the mess of :execute evaluation but also because it doesn't require that there actually be a filename for the current buffer. This drastically simplifies the HTML tidy(1) call in particular.
| | * Use full ':execute' not just ':exe' in VimLTom Ryder2017-11-032-3/+3
| | | | | | | | | | | | We should probably avoid this sort of abbreviation in scripts.
| | * Check for availability of Vim shellescape()Tom Ryder2017-11-032-10/+17
| | | | | | | | | | | | | | | It doesn't seem to be in very old Vims; worth testing for to avoid errors if I try to use the function.
| | * Revert "Adjust UrlLink() to yank word without t...Tom Ryder2017-11-031-3/+1
| | | | | | | | | | | | | | | | | | | | | | | | I thought text objects were introduced to Vim a lot later than they actually were; this works fine even in Vim 6, so I'll leave it as it's nicer. This reverts commit ffb5cbc7c681e2fdcb780dbdc51cf3458a937791.
| | * Adjust UrlLink() to yank word without text objectsTom Ryder2017-11-031-1/+3
| | |
| | * Refactor UrlLink() function normal! commandsTom Ryder2017-11-031-2/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | Tidy up the 'normal!' commands and comment them in appropriate groups. Take advantage of the natural command-termination at the end of a 'normal!' string to end insert mode. It would be better to do all of this with pure VimL functions, but I don't know how yet.
| | * Use single quotes for HTML link mapping :executeTom Ryder2017-11-031-2/+2
| | | | | | | | | | | | Saves some backslashing, just like in shell and Perl!
| | * Refactor HTML tidy(1) mappingTom Ryder2017-11-031-1/+4
| |/ | | | | | | | | Move the logic into a script function. Use single quotes for the strings, too, since we don't need interpolation.
| * Merge branch 'feature/local-leader' into developTom Ryder2017-11-039-18/+18
| |\ | | | | | | | | | | | | * feature/local-leader: Use <Leader>/<LocalLeader> correctly in Vim config
| | * Use <Leader>/<LocalLeader> correctly in Vim configTom Ryder2017-11-039-18/+18
| |/ | | | | | | | | | | | | From ":help <LocalLeader>": > In a global plugin <Leader> should be used and in a filetype plugin > <LocalLeader>. "mapleader" and "maplocalleader" can be equal.
| * Merge branch 'hotfix/v0.4.2' into developTom Ryder2017-11-032-10/+15
| |\ | | | | | | | | | | | | | | | * hotfix/v0.4.2: Require eval feature for netrw opts assignment Bump version number to 0.4.2
| * \ Merge branch 'hotfix/v0.4.1' into developTom Ryder2017-11-032-4/+4
| |\ \ | | | | | | | | | | | | | | | | | | | | * hotfix/v0.4.1: Don't show search option state on mode changes Bump version number to 0.4.1
| * \ \ Merge branch 'feature/d2u-u2d-con...' into developTom Ryder2017-11-034-5/+4
| |\ \ \ | | | | | | | | | | | | | | | | | | | | * feature/d2u-u2d-consist: Make d2u(1df)/u2d(1df) like their stream analogues
| | * | | Make d2u(1df)/u2d(1df) like their stream analoguesTom Ryder2017-11-034-5/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove the idempotency guarantee, and simplify the ed(1) scripts. See commits 2905980 and cd8e9cc: >commit 29059804f7708413843687c1764bc845d374a82d >Author: Tom Ryder <tom@sanctum.geek.nz> >Date: Fri Nov 3 13:58:23 2017 > > Remove idempotency assert for sd2u(1df)/su2d(1df) > > Commit cd8e9cc applies a cleaner implementation of these tools but > loses the idempotency: > > * Repeated su2d applications will result in double \r, so \r\r\n > * Repeated s2ru applications will result in an extra newline at the > end of the file, because the whole file will be interpreted as > one line > > However, I am OK with this, as I think of the operation as simpler > and more predictable, and I wouldn't apply it as a means to "force" > a file of unknown or various line-ending types to one type. > >commit cd8e9cc27f7dd9d360b64f4a34b8c2d048f42e45 >Author: Tom Ryder <tom@sanctum.geek.nz> >Date: Fri Nov 3 13:46:30 2017 > > Apply simpler method for sd2u(1df) and su2d(1df) > > This method is shorter, easier to read, and more idiomatic.
| * | | | Merge branch 'feature/sd2u-su2d-d...' into developTom Ryder2017-11-032-2/+2
| |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | * feature/sd2u-su2d-docfix: Remove idempotency assert for sd2u(1df)/su2d(1df)
| | * | | | Remove idempotency assert for sd2u(1df)/su2d(1df)Tom Ryder2017-11-032-2/+2
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Commit cd8e9cc applies a cleaner implementation of these tools but loses the idempotency: * Repeated su2d applications will result in double \r, so \r\r\n * Repeated s2ru applications will result in an extra newline at the end of the file, because the whole file will be interpreted as one line However, I am OK with this, as I think of the operation as simpler and more predictable, and I wouldn't apply it as a means to "force" a file of unknown or various line-ending types to one type.
| * | | | Merge branch 'feature/sd2u-su2d-r...' into developTom Ryder2017-11-032-6/+6
| |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | * feature/sd2u-su2d-refact: Apply simpler method for sd2u(1df) and su2d(1df)
| | * | | | Apply simpler method for sd2u(1df) and su2d(1df)Tom Ryder2017-11-032-6/+6
| |/ / / / | | | | | | | | | | | | | | | This method is shorter, easier to read, and more idiomatic.
| * | | | Merge branch 'feature/rm-stray-vim' into developTom Ryder2017-11-032-40/+3
| |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | * feature/rm-stray-vim: Move misplaced command.vim file
| | * | | | Move misplaced command.vim fileTom Ryder2017-11-032-40/+3
| |/ / / / | | | | | | | | | | | | | | | | | | | | Looks like this was added in a1ee04d for v0.4.0 and was intended to replace the file in its correct path at vim/config/comment.vim.
| * | | | Merge branch 'feature/review-ideas' into developTom Ryder2017-11-021-2/+0
| |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | * feature/review-ideas: Remove an idea that has now been implemented
| | * | | | Remove an idea that has now been implementedTom Ryder2017-11-021-2/+0
| |/ / / / | | | | | | | | | | | | | | | | | | | | This was implemented and released in v0.4.0 as vim/plugin/bigfile.vim and vim/doc/bigfile.txt.
| * | | | Merge branch 'release/v0.4.0' into developTom Ryder2017-11-022-2/+3
| |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * release/v0.4.0: Add vim/plugin to lint-vim arguments Bump version number to 0.4.0
* | \ \ \ \ Merge branch 'hotfix/v0.4.2'v0.4.2Tom Ryder2017-11-032-10/+15
|\ \ \ \ \ \ | | |_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | * hotfix/v0.4.2: Require eval feature for netrw opts assignment Bump version number to 0.4.2
| * | | | | Require eval feature for netrw opts assignmentTom Ryder2017-11-031-8/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Tiny builds of Vim that exclude the eval feature throw errors at the :let commands in this file: $ vim.tiny Error detected while processing /home/tom/.vim/config/netrw.vim: line 2: E319: Sorry, the command is not available in this version: let g:netrw_banner = 0 line 5: E319: Sorry, the command is not available in this version: let g:netrw_silent = 1 line 8: E319: Sorry, the command is not available in this version: let g:netrw_liststyle = 3 line 11: E319: Sorry, the command is not available in this version: let g:netrw_list_hide = '^\.$,^tags$' Press ENTER or type command to continue This code was not being run on such builds before commit 538b71c, because it was in an "after" directory for the netrw plugin and would have been skipped. Wrapping a check for has('eval') around this whole file fixes the problem. netrw.vim won't run without this feature anyway.
| * | | | | Bump version number to 0.4.2Tom Ryder2017-11-031-2/+2
|/ / / / /
* | | | | Merge branch 'hotfix/v0.4.1'v0.4.1Tom Ryder2017-11-032-4/+4
|\ \ \ \ \ | |_|_|/ / |/| | | / | | |_|/ | |/| | | | | | * hotfix/v0.4.1: Don't show search option state on mode changes Bump version number to 0.4.1
| * | | Don't show search option state on mode changesTom Ryder2017-11-031-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Commit 92f2c78 added these suffixes to the option settings to show the value of the option after it had been set, but this isn't actually appropriate for the autocmd here; it means the value of 'hlsearch' is echoed every time insert mode is entered or left.
| * | | Bump version number to 0.4.1Tom Ryder2017-11-031-2/+2
|/ / /
* | | Merge branch 'release/v0.4.0'v0.4.0Tom Ryder2017-11-0210-34/+128
|\ \ \ | | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * release/v0.4.0: Add vim/plugin to lint-vim arguments Bump version number to 0.4.0 Have bigfileturn local syntax off (configurably) Make bigfile 'synmaxcol' setting configurable Refactor plugin function for dependency injection Rename variable and autocmd to use plugin prefix Make bigfile size variable an option with default Expand comment header for bigfile.vim Move Vim big file options config into plugin Move 'tildeop' setting to new case.vim file Move 'shellpipe' setting to more logical location Make 'shellslash' setting conditional on feature
| * | Add vim/plugin to lint-vim argumentsTom Ryder2017-11-021-0/+1
| | | | | | | | | | | | | | | | | | | | | This ensures that new plugin code gets tested before being released. We don't add the new vim/doc directory, as it's plain text rather than VimL.
| * | Bump version number to 0.4.0Tom Ryder2017-11-021-2/+2
| |/
| * Merge branch 'feature/vim-plugin' into developTom Ryder2017-11-024-29/+83
| |\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * feature/vim-plugin: Have bigfileturn local syntax off (configurably) Make bigfile 'synmaxcol' setting configurable Refactor plugin function for dependency injection Rename variable and autocmd to use plugin prefix Make bigfile size variable an option with default Expand comment header for bigfile.vim Move Vim big file options config into plugin
| | * Have bigfileturn local syntax off (configurably)Tom Ryder2017-11-021-0/+10
| | |
| | * Make bigfile 'synmaxcol' setting configurableTom Ryder2017-11-021-2/+7
| | | | | | | | | | | | | | | Defaults to 256 columns and only sets it if the option's value isn't already lower than that.
| | * Refactor plugin function for dependency injectionTom Ryder2017-11-021-13/+21
| | | | | | | | | | | | | | | | | | | | | Pass the filename to check and the size limit into the function directly from the autocmd hook. Improve commenting and spacing as we go.
| | * Rename variable and autocmd to use plugin prefixTom Ryder2017-11-021-4/+4
| | | | | | | | | | | | | | | | | | Just removing an underscore from the variable name so that g:big_file_size becomes g:bigfile_size, and remove the "dotfiles" prefix from the autocmd.