Skip to content

Comments

word mix-up#2610

Closed
cPlayIt wants to merge 1 commit intocaffeinelabs:masterfrom
cPlayIt:patch-15
Closed

word mix-up#2610
cPlayIt wants to merge 1 commit intocaffeinelabs:masterfrom
cPlayIt:patch-15

Conversation

@cPlayIt
Copy link
Contributor

@cPlayIt cPlayIt commented Jun 19, 2021

I believe destructured is the correct word here.

I believe destructured is the correct word here.
@dfinity-ci
Copy link

This PR does not affect the produced WebAssembly code.

Copy link
Contributor

@crusso crusso left a comment

Choose a reason for hiding this comment

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

I actually prefer deconstructed since destructuring implies destruction of the original value (which is not the case).

@rossberg?

@cPlayIt
Copy link
Contributor Author

cPlayIt commented Jun 21, 2021 via email

@rossberg
Copy link
Contributor

"Destructuring" is the established term in a number of communities, so I'm fine with using it.

@crusso crusso mentioned this pull request Jun 22, 2021
@crusso
Copy link
Contributor

crusso commented Jun 22, 2021

Superseeded by #2620

@crusso crusso closed this Jun 22, 2021
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