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

fix(es/transform/compat) fix early return caused by comment above async func in object #2766

Conversation

UMASHIBA1
Copy link
Contributor

Fix #2701
I fixed early return caused by comment above async method in object.
if something wrong, please point it out🙇‍♂️

@CLAassistant
Copy link

CLAassistant commented Nov 16, 2021

CLA assistant check
All committers have signed the CLA.

@UMASHIBA1
Copy link
Contributor Author

UMASHIBA1 commented Nov 16, 2021

If I'm wrong I'm sorry. Did this PR solved same problem...?🤣
#2762

@kwonoj
Copy link
Member

kwonoj commented Nov 16, 2021

:oh-no: yes, it looks like aiming same issues while I tried to reuse exising span. Sorry I didn't aware if there was overlapping work.

@kdy1 kdy1 added this to the v1.2.109 milestone Nov 17, 2021
Copy link
Member

@kdy1 kdy1 left a comment

Choose a reason for hiding this comment

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

The new code can panic if someone trolls, but rust code is required to such trolling, so it's fine IMO.

@kdy1 kdy1 merged commit ddfc7e6 into swc-project:main Nov 17, 2021
@kwonoj
Copy link
Member

kwonoj commented Nov 17, 2021

@kdy1 do you think this is still required after #2762? My impression was preserving original span would be ok.

@kwonoj
Copy link
Member

kwonoj commented Nov 17, 2021

Looks like #2758 was dupe of #2701 and I didn't realize it.

@kdy1
Copy link
Member

kdy1 commented Nov 17, 2021

@kwonoj Oh, you are right. I'll test if it works with this PR reverted.

kdy1 added a commit that referenced this pull request Nov 17, 2021
@kdy1
Copy link
Member

kdy1 commented Nov 17, 2021

@kwonoj I'll revert this PR as it's not required.

kdy1 added a commit to kdy1/swc that referenced this pull request Nov 17, 2021
@UMASHIBA1
Copy link
Contributor Author

:oh-no: yes, it looks like aiming same issues while I tried to reuse exising span. Sorry I didn't aware if there was overlapping work.

OK👍 Thank you for your great work. I'm glad the issue was solved early.

kdy1 added a commit that referenced this pull request Nov 17, 2021
kdy1 added a commit to kdy1/swc that referenced this pull request Nov 20, 2021
@swc-project swc-project locked as resolved and limited conversation to collaborators Nov 3, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Comments above an object method declaration cause an early return to be added to the function body
4 participants