Scala Programming Language
  1. Scala Programming Language
  2. SI-5739

preserve names of pattern variables in bytecode

    Details

      Issue Links

        Activity

        Hide
        Eugene Burmako added a comment -

        Why was the priority lowered from critical? This issue pretty much forces one to use old patmat when debugging patmat-heavy code.

        Show
        Eugene Burmako added a comment - Why was the priority lowered from critical? This issue pretty much forces one to use old patmat when debugging patmat-heavy code.
        Hide
        Adriaan Moors added a comment -

        because I have 131 bugs assigned to me, give or take,
        and I need to prioritize which ones I'll fix first
        major is still pretty high on my list

        Show
        Adriaan Moors added a comment - because I have 131 bugs assigned to me, give or take, and I need to prioritize which ones I'll fix first major is still pretty high on my list
        Hide
        Adriaan Moors added a comment -

        to further improve debuggability, we should also emit local variables for case class fields (this would then fix SI-5158)

        Show
        Adriaan Moors added a comment - to further improve debuggability, we should also emit local variables for case class fields (this would then fix SI-5158 )
        Show
        Adriaan Moors added a comment - https://github.com/scala/scala/pull/937
        Hide
        Adriaan Moors added a comment -

        reopening because it causes performance regressions

        Show
        Adriaan Moors added a comment - reopening because it causes performance regressions
        Show
        Adriaan Moors added a comment - https://github.com/scala/scala/pull/1060
        Hide
        Adriaan Moors added a comment -

        reopening for a further tweak: shouldn't emit a local variable for debuggability if the original name was a wildcard – it seems unlikely to matter and impacts bytecode size unnecessarily

        Show
        Adriaan Moors added a comment - reopening for a further tweak: shouldn't emit a local variable for debuggability if the original name was a wildcard – it seems unlikely to matter and impacts bytecode size unnecessarily
        Hide
        Adriaan Moors added a comment - - edited
        Show
        Adriaan Moors added a comment - - edited refined in https://github.com/scala/scala/pull/2033

          People

          • Assignee:
            Adriaan Moors
            Reporter:
            Eugene Burmako
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development