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

Remove a:module arguments [fix for 5.5] #93

Merged
merged 1 commit into from Jan 25, 2016

Conversation

andy-morris
Copy link
Contributor

Remove the 'a:module' arguments from the calls to ghcmod#build_command in
the sig, split, info, and type functions. No version check because they've
actually been optional since v5.0, but now they have been removed completely.

Also replace them with ... in the function's argument list, so that calls
with or without them are accepted, In case there are any other plugins based
on ghcmod-vim. For the same reason, ghcmod#detect_module is still there.

@andy-morris
Copy link
Contributor Author

btw what are your thoughts on changing the arguments to ... vs removing them entirely?

@andy-morris andy-morris changed the title Remove a:module arguments Remove a:module arguments [fix for 5.5 Jan 20, 2016
@andy-morris andy-morris changed the title Remove a:module arguments [fix for 5.5 Remove a:module arguments [fix for 5.5] Jan 20, 2016
Remove the 'a:module' arguments from the calls to `ghcmod#build_command` in
the sig, split, info, and type functions. No version check because they've
actually been optional since v5.0, but now they have been removed completely.

Also replace them with `...` in the function's argument list, so that calls
with or without them are accepted, In case there are any other plugins based
on ghcmod-vim. For the same reason, `ghcmod#detect_module` is still there.
@andy-morris
Copy link
Contributor Author

[rebased for current HEAD.]

eagletmt added a commit that referenced this pull request Jan 25, 2016
Remove a:module arguments [fix for 5.5]
@eagletmt eagletmt merged commit 815616e into eagletmt:master Jan 25, 2016
@eagletmt
Copy link
Owner

🆒
Thank you so much!

@andy-morris andy-morris deleted the module-args branch February 2, 2016 03:36
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.

None yet

2 participants