summary refs log tree commit diff
Commit message (Collapse)AuthorAge
...
| * Revert "Disable powerline for now"June McEnroe2012-01-22
| | | | | | | | This reverts commit fa48f5965c57272e616721d92475cd2f1e35f667.
| * Revert "Add VCS repo directories to wildignore"June McEnroe2012-01-22
| | | | | | | | | | | | This reverts commit 41cb1db33ba7b64d8af63bf55bb82f7e8a617518. This fixes not being able to commit with vim-fugitive
| * Disable powerline for nowJune McEnroe2012-01-22
| |
| * Update some pluginsJune McEnroe2012-01-22
| |
| * Add VCS repo directories to wildignoreJune McEnroe2012-01-22
| |
| * Map ,e and ,b to CtrlP file and buffer respectivelyJune McEnroe2012-01-22
| |
| * Add vim-spaceJune McEnroe2012-01-22
| |
| * Add AutoCloseJune McEnroe2012-01-22
| |
| * Add binding for GundoJune McEnroe2012-01-22
| |
| * Add GundoJune McEnroe2012-01-22
| |
| * Add Jellybeans colorschemeJune McEnroe2012-01-22
| |
| * Add syntasticJune McEnroe2012-01-22
| |
| * Add PowerlineJune McEnroe2012-01-21
| |
| * Add quicktaskJune McEnroe2012-01-21
| |
| * Moved comments out of mapsJune McEnroe2012-01-15
| |
| * Disable scrollbarsJune McEnroe2012-01-14
| |
| * Added a silly READMEJune McEnroe2012-01-14
| |
| * Map ,gp to :Git pushJune McEnroe2012-01-14
| |
| * Added ooc.vimJune McEnroe2012-01-14
| |
| * Added gist-vimJune McEnroe2012-01-14
| |
| * Added maps for some Fugitive commandsJune McEnroe2012-01-14
| |
| * Initial commitJune McEnroe2012-01-14
|
* Add GitHub username to gitconfigJune McEnroe2013-11-10
|
* Rehash on reloadJune McEnroe2013-11-02
|
* Remove git prompt from titleJune McEnroe2013-11-02
|
* Update zsh syntax highlighting and configJune McEnroe2013-11-02
|
* Fix host in prompt, part twoJune McEnroe2013-10-31
|
* Fix host in promptJune McEnroe2013-10-31
|
* Wow look reorganized zsh configJune McEnroe2013-10-31
|
* Show hostname in prompt over SSHJune McEnroe2013-10-30
|
* Only load chruby if it existsJune McEnroe2013-10-30
|
* Add auto-title stuff to zshJune McEnroe2013-10-27
|
* Add gvim alias to silence stderrJune McEnroe2013-10-26
|
* Add home.programble.me functionJune McEnroe2013-10-26
|
* Remove unused configsJune McEnroe2013-10-14
|
* Add therefore, because, cards compose sequencesJune McEnroe2013-10-14
|
* Global git ignore .swo filesJune McEnroe2013-10-14
|
* Add Heroku to zshJune McEnroe2013-10-14
|
* Add gst git stash aliasJune McEnroe2013-09-23
|
* Remove pacman stuffJune McEnroe2013-09-05
| | | | Goodbye Arch.
* Only alias hub if it existsJune McEnroe2013-09-05
|
* chruby to ruby-2.0.0June McEnroe2013-09-05
|
* Add nvmJune McEnroe2013-08-14
|
* Add title commandJune McEnroe2013-07-09
|
* Use default client keysJune McEnroe2013-06-09
|
* Run games in separate X serverJune McEnroe2013-06-09
|
* Add game wrapper to kill/restart unclutterJune McEnroe2013-06-08
|
* Add git configJune McEnroe2013-06-08
|
* Ignore effuse backup filesJune McEnroe2013-06-07
|
* Autochruby and default to ruby-2.0.0-p195June McEnroe2013-06-07
|
generated by cgit-pink 1.4.1 (git 2.36.1) at 2026-10-10 04:57:59 +0000
his then causes *quotedp to be set to true, meaning evalvar()'s quoted > variable is turned on. quoted is then passed to recordregion() as the > nulonly parameter. > > ifsp->nulonly has a bigger effect than merely selecting whether to use > $IFS or whether to only split on null bytes: in ifsbreakup(), nulonly > also causes string termination to be suppressed. That's correct: that > special treatment is required to preserve empty fields in "$@" > expansion. But it should *only* be used when $@ is quoted: ifsbreakup() > takes nulonly from the last IFS region, even if it's empty, so having an > additional zero-length region with nulonly enabled causes confusion. > > Passing quoted by value to varvalue() and not attempting to modify it > should therefore, and in my quick testing does, also work to fix the > original $@ bug. You're right. The proper fix to this is to ensure that nulonly is not set in varvalue for $*. It should only be set for $@ when it's inside double quotes. In fact there is another bug while we're playing with $@/$*. When IFS is set to a non-whitespace character such as :, $* outside quotes won't remove empty fields as it should. This patch fixes both problems. Reported-by: Martijn Dekker <martijn@inlv.org> Suggested-by: Harald van Dijk <harald@gigawatt.nl> Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au> 2018-04-02parser: Allow newlines within parameter substitutionHerbert Xu On Fri, Mar 16, 2018 at 11:27:22AM +0800, Herbert Xu wrote: > On Thu, Mar 15, 2018 at 10:49:15PM +0100, Harald van Dijk wrote: > > > > Okay, it can be trivially modified to something that does work in other > > shells (even if it were actually executed), but gets rejected at parse time > > by dash: > > > > if false; then > > : ${$+ > > } > > fi > > That's just a bug in dash's parser with ${} in general, because > it bombs out without the if clause too: > > : ${$+ > } This patch fixes the parsing of newlines with parameter substitution. Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au> 2018-04-02expand: Fix bugs with words connected to the right of $@Herbert Xu On Sun, Mar 04, 2018 at 12:44:59PM +0100, Harald van Dijk wrote: > > command: set -- a ""; space=" "; printf "<%s>" "$@"$space > bash: <a><> > dash 0.5.8: <a>< > > dash 0.5.9.1: <a>< > > dash patched: <a><> This is actually composed of two bugs. First of all our tracking of quotemark is wrong so anything after "$@" becomes quoted. Once we fix that then the problem is that the first space character after "$@" is not recognised as an IFS. This patch fixes both. Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au> 2018-03-25Revert "[BUILTIN] Remove unnecessary restoration of format string in printf"Herbert Xu This reverts commit 7bb413255368e94395237d789f522891093c5774. The commit breaks printf with more than argument. Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au> 2018-03-22parser: Fix backquote support in here-document EOF markHerbert Xu