Skip to content

Commit

Permalink
Fix 'jump to definition' for singleton operations (#646) (#681)
Browse files Browse the repository at this point in the history
Resolves #646 

Tested locally, here's video proof: :)

![Screen Recording 2024-11-08 at 14 54
13-2](https://github.com/user-attachments/assets/e5ab9a1e-1976-4f72-ba63-672d5e2a74d6)
  • Loading branch information
jiribenes authored Nov 8, 2024
1 parent 000a768 commit c914e37
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions effekt/shared/src/main/scala/effekt/RecursiveDescent.scala
Original file line number Diff line number Diff line change
Expand Up @@ -471,8 +471,7 @@ class RecursiveDescent(positions: Positions, tokens: Seq[Token], source: Source)
nonterminal:
`effect` ~> operation() match {
case op @ Operation(id, tps, vps, bps, ret) =>
// TODO is the `true` flag used at all anymore???
InterfaceDef(IdDef(id.name), tps, List(Operation(id, Nil, vps, bps, ret) withPositionOf op))
InterfaceDef(IdDef(id.name) withPositionOf op, tps, List(Operation(id, Nil, vps, bps, ret) withPositionOf op))
}

def operation(): Operation =
Expand Down

0 comments on commit c914e37

Please sign in to comment.