antlr / antlr/antlr5

Renaming Modes.DEFAULT_MODE to Modes.DEFAULT in the Kotlin runtime and target

Open
#10 1 comment 0 reactions 1 assignee Claimed by @ftomassetti View on GitHub
Dominant language
Java
Stars
75
Forks
8
PR merge metrics
No merged PRs in 30d

Description

Currently we generate this code for the Kotlin target:

```
object Modes {
const val DEFAULT_MODE = 0
const val FOO = 1 // made uppercase by #8
const val BAR = 2
}
```

This is inconsistent because only the default mode has the `_MODE` suffix. We should therefore rename it, by working on the template for the Kotlin target and updating references in the Kotlin runtime

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.