You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In many cases it's probably easier to deal with when we use the simpler form
boolean isFinite();
The only real use case within lib-recur itself is in LastInstance, which only has a result if the recurrence set is finite.
At present this requires a negation, however, along with dmfs/jems#385 this could be simplified from
new Restrained<>(() -> !recurrenceSet.isInfinite(), new Last<>(recurrenceSet))
to
new If<>(recurrenceSet::isFinite, new Last<>(recurrenceSet))
reducing the cognitive load this needs to be understood.
Initially, to avoid a breaking change, the isFinite method could be added as a default method. In a second step we could swap this to make isInfinite the (deprecated) default method and in a third step we would just drop it entirely.
The text was updated successfully, but these errors were encountered:
In many cases it's probably easier to deal with when we use the simpler form
The only real use case within lib-recur itself is in
LastInstance
, which only has a result if the recurrence set is finite.At present this requires a negation, however, along with dmfs/jems#385 this could be simplified from
to
reducing the cognitive load this needs to be understood.
Initially, to avoid a breaking change, the
isFinite
method could be added as a default method. In a second step we could swap this to makeisInfinite
the (deprecated) default method and in a third step we would just drop it entirely.The text was updated successfully, but these errors were encountered: