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

ast 모듈에서 any 타입을 쓴 코드를 조금 고침 #78

Merged
merged 2 commits into from
Mar 11, 2020

Conversation

Kroisse
Copy link
Contributor

@Kroisse Kroisse commented Mar 11, 2020

ast/base.ts 모듈은 전반적으로 hack 덩어리라 당분간은 여기서 any를 줄이는 게 어려울 것 같습니다. 여기서 뭔가 해 보려면 ast 모듈에 대한 전반적인 구조 개선이 필요합니다.

builtin 모듈의 Yaksok 클래스를 ast 모듈의 Def와 어떻게든 연관지어야 일이 쉬워질 것 같은데 일단 여기서는 그대로 두었습니다. 이 때문에 analyzer 모듈의 동작이 기존과 조금 달라졌을 수 있습니다.

@disjukr
Copy link
Member

disjukr commented Mar 11, 2020

말씀하신대로 builtin.Yaksok 클래스가 ast.Def를 상속받지 않고있기 때문에, 아마 이 수정을 반영하고서 UnusedDefRemover plugin을 사용하면 빌트인 선언이 날라가버릴 것 같네요.

...아닌가? 그러고보니 빌트인 선언은 어차피 원래 AST 상에 있지도 않으니 상관없을거 같기도

Copy link
Member

@disjukr disjukr left a comment

Choose a reason for hiding this comment

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

일단 ㅇㅋ입니다

@disjukr disjukr merged commit b355c30 into yaksok:master Mar 11, 2020
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.

2 participants