Skip to content

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
    • Help
    • Support
    • Submit feedback
    • Contribute to GitLab
  • Sign in
asm
asm
  • Project
    • Project
    • Details
    • Activity
    • Releases
    • Cycle Analytics
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Charts
  • Issues 1
    • Issues 1
    • List
    • Boards
    • Labels
    • Milestones
  • Merge Requests 0
    • Merge Requests 0
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
    • Charts
  • Packages
    • Packages
    • Container Registry
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Members
    • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Charts
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • asm
  • asmasm
  • Issues
  • #317817

Closed
Open
Opened Mar 20, 2018 by Semyon Levin@remal
  • Report abuse
  • New issue
Report abuse New issue

Allow Kotlin back-tick identifiers by CheckMethodAdapter.checkInternalName() and CheckMethodAdapter.checkMethodIdentifier()

Hello,

In Kotlin language you can use identifiers with any(?) characters except '[', ']', '(', ')', '<', '>', '/', '.', ';', ':' by escaping such identifiers with back-tick: '`':

class `Example class - name` {
    val `example field (name)` = "123"
    fun `example method - name`() {
        val `example local variable` = "gg"
    }
}

Please make Check*Adapter classes don't throw exception for such identifiers.

You can check if class is Kotlin class by checking if it annotated by kotlin.Metadata annotation. It will be very good if CheckClassAdapter checks this automatically. If you think that it's too complicated, you can add additional constructor parameter.

Thanks!

PS: I can be mistaken with a list of prohibited Kotlin identifier characters...

Assignee
Assign to
None
Milestone
None
Assign milestone
Time tracking
None
Due date
None
2
Labels
#bug util
Assign labels
  • View project labels
Reference: asm/asm#317817