Skip to content

Conversation

Gaelan
Copy link
Contributor

@Gaelan Gaelan commented Mar 2, 2014

My little contribution to #209.

This is a replacement for my previous PR #248, which:

  1. used Gaelan/master instead of a topic branch
  2. fixed a bunch of trailing whitespace
  3. didn't compile

My little contribution to neovim#209.
@Gaelan
Copy link
Contributor Author

Gaelan commented Mar 2, 2014

I'm pretty sure this is ready to be merged.

@simendsjo
Copy link
Contributor

Nice. I can verify that it builds without warnings and pass the tests on Arch x64.

@Gaelan
Copy link
Contributor Author

Gaelan commented Mar 2, 2014

Also builds on an x86_64 Mac running 10.9.

@aktau
Copy link
Contributor

aktau commented Mar 2, 2014

Much better, looks good!

@tarruda
Copy link
Member

tarruda commented Mar 3, 2014

Merged, ty

@tarruda tarruda closed this Mar 3, 2014
Grimy pushed a commit to Grimy/neovim that referenced this pull request Jan 7, 2015
butwerenotthereyet pushed a commit to butwerenotthereyet/neovim that referenced this pull request Dec 30, 2019
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.

4 participants