Skip to content

Commit

Permalink
Merge pull request #517 from zenhack/delete-irrelevant-fixme
Browse files Browse the repository at this point in the history
Delete irrelevant FIXME comment.
  • Loading branch information
zenhack authored May 16, 2023
2 parents b6c113c + 876798e commit 6eacdac
Showing 1 changed file with 0 additions and 7 deletions.
7 changes: 0 additions & 7 deletions answer.go
Original file line number Diff line number Diff line change
Expand Up @@ -33,13 +33,6 @@ type Promise struct {
// Next state is resolved.
// - Resolved. Fulfill or Reject has finished.

// When acquiring multiple Promise.state mutexes, they must be acquired
// in traversal order (i.e. p, then p.next, then p.next.next).
//
// FIXME: is the above traversal scheme sound? It depends on avoiding
// cycles in the graph to prevent potential deadlocks, and it is not
// clear to me(zenhack) that this is a safe assumption (I suspect it's
// not).
state mutex.Mutex[promiseState]
}

Expand Down

0 comments on commit 6eacdac

Please sign in to comment.