[PR #1066] [MERGED] Simplify precedence and associativity parsing #872

New Issue

adam · 2025-12-30T01:27:26+01:00

adam commented

2025-12-30 01:27:26 +01:00

📋 Pull Request Information

Original PR: https://github.com/apple/pkl/pull/1066
Author: @stackoverflow
Created: 5/9/2025
Status: ✅ Merged
Merged: 5/12/2025
Merged by: @stackoverflow

Base: main ← Head: better-precedence-parsing

📝 Commits (3)

abb8d04 Simplify precedence and associativity parsing
b9deb42 Reduce allocations
7d818b2 Special case for dot

📊 Changes

7 files changed (+112 additions, -324 deletions)

View changed files

📝 pkl-parser/src/main/java/org/pkl/parser/Lexer.java (+50 -64)
➖ pkl-parser/src/main/java/org/pkl/parser/OperatorResolver.java (+0 -140)
📝 pkl-parser/src/main/java/org/pkl/parser/Parser.java (+25 -27)
📝 pkl-parser/src/main/java/org/pkl/parser/syntax/Expr.java (+0 -61)
📝 pkl-parser/src/main/java/org/pkl/parser/syntax/Operator.java (+37 -21)
📝 pkl-parser/src/test/kotlin/org/pkl/parser/LexerTest.kt (+0 -5)
📝 pkl-parser/src/test/kotlin/org/pkl/parser/SexpRenderer.kt (+0 -6)

📄 Description

No description provided

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/apple/pkl/pull/1066 **Author:** [@stackoverflow](https://github.com/stackoverflow) **Created:** 5/9/2025 **Status:** ✅ Merged **Merged:** 5/12/2025 **Merged by:** [@stackoverflow](https://github.com/stackoverflow) **Base:** `main` ← **Head:** `better-precedence-parsing` --- ### 📝 Commits (3) - [`abb8d04`](https://github.com/apple/pkl/commit/abb8d041417f4528a9382812d4cdb142ccc08ad2) Simplify precedence and associativity parsing - [`b9deb42`](https://github.com/apple/pkl/commit/b9deb4222b8b5b84d5e5f73b1fa005af7def0ec6) Reduce allocations - [`7d818b2`](https://github.com/apple/pkl/commit/7d818b24c8fe5f5010c2d19e37277570c57412d5) Special case for dot ### 📊 Changes **7 files changed** (+112 additions, -324 deletions) <details> <summary>View changed files</summary> 📝 `pkl-parser/src/main/java/org/pkl/parser/Lexer.java` (+50 -64) ➖ `pkl-parser/src/main/java/org/pkl/parser/OperatorResolver.java` (+0 -140) 📝 `pkl-parser/src/main/java/org/pkl/parser/Parser.java` (+25 -27) 📝 `pkl-parser/src/main/java/org/pkl/parser/syntax/Expr.java` (+0 -61) 📝 `pkl-parser/src/main/java/org/pkl/parser/syntax/Operator.java` (+37 -21) 📝 `pkl-parser/src/test/kotlin/org/pkl/parser/LexerTest.kt` (+0 -5) 📝 `pkl-parser/src/test/kotlin/org/pkl/parser/SexpRenderer.kt` (+0 -6) </details> ### 📄 Description _No description provided_ --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

adam added the pull-request label 2025-12-30 01:27:26 +01:00

adam closed this issue

2025-12-30 01:27:26 +01:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/pkl#872