Skip to content

Fix multi bind - #419

Merged
bryanjos merged 3 commits into
masterfrom
fix-multi-bind
Sep 17, 2017
Merged

Fix multi bind#419
bryanjos merged 3 commits into
masterfrom
fix-multi-bind

Conversation

@bryanjos

Copy link
Copy Markdown
Collaborator

fixes #413

|> Map.put(:expression, right)
end

defp compile_match(%{patterns: [left], expression: right}, state) do

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Function is too complex (ABC size is 38, max is 30).

{ js_ast, state }
end

defp compile_match(%{patterns: lefts, expression: right}, state) do

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Function is too complex (ABC size is 43, max is 30).

[array_pattern, ref_declaration]
end


Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There should be no more than 1 consecutive blank lines.

@bryanjos
bryanjos merged commit af24949 into master Sep 17, 2017
@bryanjos
bryanjos deleted the fix-multi-bind branch September 17, 2017 21:46
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.

Using multple match operators in same expression does not work

1 participant