Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(225)

Issue 4009043: code review 4009043: misc: Import/Drop commands for Vim (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
14 years, 10 months ago by niemeyer
Modified:
14 years, 10 months ago
Reviewers:
CC:
adg, golang-dev
Visibility:
Public.

Description

misc: Import/Drop commands for Vim New ftplugin adds Import and Drop commands for Go buffers in Vim. These commands ensure that the provided package is imported (or not imported) in the current Go buffer, using proper style and ordering, without moving the cursor. E.g. :Import strings :ImportAs . strings :Drop strings Two mappings are also introduced to help with the fmt package: \f => :Import fmt \F => :Drop fmt

Patch Set 1 #

Patch Set 2 : code review 4009043: misc: Import/Drop commands for Vim #

Patch Set 3 : code review 4009043: misc: Import/Drop commands for Vim #

Total comments: 3

Patch Set 4 : code review 4009043: misc: Import/Drop commands for Vim #

Unified diffs Side-by-side diffs Delta from patch set Stats (+217 lines, -0 lines) Patch
A misc/vim/ftplugin/go/import.vim View 1 2 1 chunk +203 lines, -0 lines 0 comments Download
M misc/vim/readme.txt View 1 2 3 1 chunk +14 lines, -0 lines 0 comments Download

Messages

Total messages: 6
niemeyer
Hello golang-dev@googlegroups.com (cc: golang-dev@googlegroups.com), I'd like you to review this change.
14 years, 10 months ago (2011-01-24 13:58:34 UTC) #1
rsc
R=adg
14 years, 10 months ago (2011-01-24 18:51:32 UTC) #2
adg
Really cool plugin, thanks! http://codereview.appspot.com/4009043/diff/6001/misc/vim/readme.txt File misc/vim/readme.txt (right): http://codereview.appspot.com/4009043/diff/6001/misc/vim/readme.txt#newcode29 misc/vim/readme.txt:29: In addition to syntax highlighting, ...
14 years, 10 months ago (2011-01-25 00:48:40 UTC) #3
niemeyer
All done, please take another look.
14 years, 10 months ago (2011-01-25 01:28:27 UTC) #4
adg
*** Submitted as http://code.google.com/p/go/source/detail?r=cec205247d83 *** misc: Import/Drop commands for Vim New ftplugin adds Import and ...
14 years, 10 months ago (2011-01-25 01:36:25 UTC) #5
adg
14 years, 10 months ago (2011-01-25 01:36:45 UTC) #6
LGTM

On 25 January 2011 11:28,  <n13m3y3r@gmail.com> wrote:
> All done, please take another look.
>
> http://codereview.appspot.com/4009043/
>
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b