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

Bug fixes #634

Merged
merged 2 commits into from
Mar 5, 2017
Merged

Bug fixes #634

merged 2 commits into from
Mar 5, 2017

Conversation

KronicDeth
Copy link
Owner

@KronicDeth KronicDeth commented Mar 5, 2017

Changelog

Bug Fixes

  • Variable scope for QualifiedMultipleAliases, which occurs when qualified call occurs over a line with assignment to a tuple, such as
    Qualifier.
    {:ok, value} = call()
  • Remove call definition clauses (function or macro) completion for bare words as it had a detrimental impact on typing feedback (the editor still took input, but it wasn't rendered until the completion returned OR ESC was hit to cancel the completion, which became excessive once the index of call definition clauses was expanded by the decompilation of the Elixir standard library .beams, so disable it. If bare-words completion is restored. It will either (1) need to not use the Reference#getVariants() API because it generates too many objects that need to be thrown away or (2) need to only complete call definition clauses that are provably in-scope from imports or other macros.

Occurs when qualified call occurs over a line with assignment ot a
tuple, such as `Qualifier.\n{:ok, value} = call()`.
Completion of bare words as call definition clauses (functions or
macros) had a detrimental impact on typing feedback, which became
excessive once the index of call definition clauses was expanded by the
decompilation of the Elixir standard library `.beam`s, so disable it.
If bare-words completion is restored.  It will either (1) need to not
use the `Reference#getVariants()` API because it generates too many
objects that need to be thrown away or (2) need to only complete call
definition clauses that are provably in-scope from imports or other
macros.
@KronicDeth KronicDeth added this to the v5.0.0 milestone Mar 5, 2017
@KronicDeth KronicDeth self-assigned this Mar 5, 2017
@KronicDeth KronicDeth merged commit 7386865 into master Mar 5, 2017
@KronicDeth KronicDeth deleted the bug-fixes branch March 5, 2017 15:01
KronicDeth added a commit that referenced this pull request Mar 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant