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

replace with antlr4-runtime #4

Closed
wants to merge 15 commits into from
Closed

replace with antlr4-runtime #4

wants to merge 15 commits into from

Conversation

namenu
Copy link
Member

@namenu namenu commented Feb 12, 2024

  1. clj-antlr 를 제거하였음.
  2. schema.g4 와 Graphql.g4 문법 정의로부터 각각의 Lexer 와 Parser 를 생성하였음. (gen/src)
  3. Lexer 와 Parser 를 컴파일하는 빌드 스크립트를 추가하고, 바이트코드를 생성하였음. (gen/classes)
  4. c.w.l.parser 에서 .g4 를 직접 인터프리터에 넣는 로직을 제거하였음
  5. 제거된 clj-antlr 에 대응하는 shim 로직을 c.w.l.parser.antlr 에 작성하였음
  6. 3번을 -X:deps prep 으로 대체하였음

@bakyeono
Copy link

bakyeono commented Feb 13, 2024

훌륭하십니다. 혹시 바이트코드는 JVM 버전 호환성 문제는 없나요?

@bakyeono
Copy link

훌륭하십니다. 혹시 바이트코드는 JVM 버전 호환성 문제는 없나요?

아, 직접 빌드를 하는 거군요

@namenu
Copy link
Member Author

namenu commented Feb 13, 2024

훌륭하십니다. 혹시 바이트코드는 JVM 버전 호환성 문제는 없나요?

아, 직접 빌드를 하는 거군요

네 직접 빌드 할 때 바이트코드 타겟을 라시니아와 동일한 Java 11 로 맞추려고 합니다.

@namenu namenu closed this Feb 14, 2024
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