Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Some small :GoImpl improvements #1386

Merged
merged 1 commit into from
Sep 6, 2017
Merged

Some small :GoImpl improvements #1386

merged 1 commit into from
Sep 6, 2017

Conversation

arp242
Copy link
Contributor

@arp242 arp242 commented Jul 25, 2017

  • When using :GoImpl on type foo struct{} it would work, but with:

      type foo struct{
      }
    

    or with a struct with fields, it would create the generated methods inside
    the struct. Fixes GoImpl creates method stubs inside struct definiton #1077.

  • Allow using :GoImpl on the type and struct parts too. Makes it a wee
    bit easier to use ;-)

  • Output would include extra newline, and error would include trailing newline
    from shell command:

      vim-go: invalid receiver: "} *}"<00>
    
  • Small documentation fix.

- When using `:GoImpl` on `type foo struct{}` it would work, but with:

        type foo struct{
        }

  or with a struct with fields, it would create the generated methods *inside*
  the struct. Fixes #1077.

- Allow using `:GoImpl` on the `type` and `struct` parts too. Makes it a wee
  bit easier to use ;-)

- Output would include extra newline, and error would include trailing newline
  from shell command:

        vim-go: invalid receiver: "} *}"<00>

- Small documentation fix.
@fatih fatih merged commit b0ae2c5 into fatih:master Sep 6, 2017
@fatih
Copy link
Owner

fatih commented Sep 6, 2017

Thanks @Carpetsmoker as always 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GoImpl creates method stubs inside struct definiton
2 participants