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

Add an alias to fix the breakage of #1375 #1377

Merged
merged 1 commit into from
Jul 16, 2016

Conversation

mrBliss
Copy link
Contributor

@mrBliss mrBliss commented Jul 16, 2016

In #1375, haskell-guess-module-name was renamed to
haskell-guess-module-name-from-file-name which caused breakage for
users of the original function name. Fix the breakage by defining an
alias that makes the original name point to the new function name.

@ivan-m
Copy link
Contributor

ivan-m commented Jul 16, 2016

I think I would prefer a version that wraps up the call to buffer-file-name; I'm not sure if that's possible with defalias though.

@mrBliss
Copy link
Contributor Author

mrBliss commented Jul 16, 2016

I think I would prefer a version that wraps up the call to buffer-file-name; I'm not sure if that's possible with defalias though.

You're right, I forgot about that. I'll update the PR.

In haskell#1375, `haskell-guess-module-name` was renamed to
`haskell-guess-module-name-from-file-name` which caused breakage for
users of the original function name. Fix the breakage by defining a stub
function `haskell-guess-module-name` which calls
`haskell-guess-module-name-from-file-name` with the buffer's file name.
@gracjan gracjan merged commit 057cd0a into haskell:master Jul 16, 2016
@gracjan
Copy link
Contributor

gracjan commented Jul 16, 2016

Thanks.

@ivan-m
Copy link
Contributor

ivan-m commented Jul 27, 2016

This has different semantics if the module name can't be guessed: previously it would return nil, now it returns "".

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

3 participants