I like no-braces for single-line ifs as much as anyone, but with inline comments involved it gets rather dicey.
Reverse()
SliderPath
segmentEnds
IndexOutOfRangeException
.Distinct()
truncateEndingDuplicates()
Extract control point reversing to separate method
GenerateTicks
Judged
Without this change, when the `Judged` value is checked on an `HitObjectLifetimeEntry` it would return `true` if a `DrawableHitObject` has not yet been associated with the entry. Which is completely wrong. Of note, the usage in `DrawableHitObject` will have never fallen through to this incorrect value as they always have a result populated: https://github.com/ppy/osu/blob/f26f001e1d01ca6bb53225da7bf06c0ad21153c5/osu.Game/Rulesets/Objects/Drawables/DrawableHitObject.cs#L721-L726
HitObjectEntryManager
HitObjectLifetimeEntry.NestedEntries
Fix timeline blueprint display not showing correct combo number / colour during placement
HitSampleInfo.With
GetSampleInfo
Also add full xmldoc
HitSampleInfo.Bank
IAnimationTimeReference
DrawableHitObject
Co-authored-by: Dean Herbert <pe@ppy.sh>