Skip to content

Use mapAccumulate #698

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 2, 2025
Merged

Use mapAccumulate #698

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
26 changes: 13 additions & 13 deletions core/src/main/scala/Node.scala
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -211,13 +211,13 @@ final case class Node[A](
case None => None
case Some(c) => copy(child = c.some).some
case _ =>
variations.foldLeft((false, List.empty[Variation[A]])) {
case ((true, acc), n) => (true, acc :+ n)
case ((false, acc), n) =>
variations.mapAccumulate(false):
case (true, n) => (true, n)
case (false, n) =>
n.modifyChildAt(path, f) match
case Some(nn) => (true, acc :+ nn)
case None => (false, acc :+ n)
} match
case Some(nn) => (true, nn)
case None => (false, n)
match
case (true, ns) => copy(variations = ns).some
case (false, _) => none

Expand All @@ -230,15 +230,15 @@ final case class Node[A](
case None => None
case Some(c) => copy(child = c.some).some
case head :: _ =>
variations.foldLeft((false.some, List.empty[Variation[A]])):
case ((Some(true), acc), n) => (true.some, n :: acc)
case ((Some(_), acc), n) if n.hasId(head) =>
variations.mapAccumulate(false.some):
case (Some(true), n) => (true.some, n)
case (Some(_), n) if n.hasId(head) =>
n.modifyAt(path, f) match
case None => (None, n :: acc)
case Some(nn) => (true.some, nn :: acc)
case ((x, acc), n) => (x, n :: acc)
case None => (None, n)
case Some(nn) => (true.some, nn)
case (x, n) => (x, n)
match
case (Some(true), ns) => copy(variations = ns.reverse).some
case (Some(true), ns) => copy(variations = ns).some
case _ => none

// delete the node at the end of the path
Expand Down