Identifiers should start with capital letters #2

Closed

ColinTimBarndt wants to merge 1 commit from master into master

pull from: master

merge into: lezer:master

lezer:main

Conversation 1 Commits 1 Files changed 1 +1 -1

ColinTimBarndt commented 2021-06-06 13:58:07 +02:00 (Migrated from github.com)

Copy link

See https://github.com/lezer-parser/java/issues/1 and https://docs.oracle.com/javase/specs/jls/se15/html/jls-3.html#jls-Identifier

See https://github.com/lezer-parser/java/issues/1 and https://docs.oracle.com/javase/specs/jls/se15/html/jls-3.html#jls-Identifier

marijnh commented 2021-06-06 18:13:59 +02:00 (Migrated from github.com)

Copy link

See the capitalIdentifier defined below. These are used for a heuristic to prefer parsing capitalized identifiers as class names, and this change would break that. Attached patch solves this in another way.

See the `capitalIdentifier` defined below. These are used for a heuristic to prefer parsing capitalized identifiers as class names, and this change would break that. Attached patch solves this in another way.

Pull request closed

This pull request cannot be reopened because the branch was deleted.

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

lezer/java!2

No description provided.