mirror of
https://github.com/ppy/osu.git
synced 2025-01-13 14:12:56 +08:00
Apply suggestions from code review
Co-authored-by: Bartłomiej Dach <dach.bartlomiej@gmail.com>
This commit is contained in:
parent
e0a3696145
commit
9facfe8964
@ -629,7 +629,7 @@ namespace osu.Game.Screens.Play
|
|||||||
/// The display of the results screen may be delayed by any work being done in <see cref="PrepareScoreForResults"/> and <see cref="PrepareScoreForResultsAsync"/>.
|
/// The display of the results screen may be delayed by any work being done in <see cref="PrepareScoreForResults"/> and <see cref="PrepareScoreForResultsAsync"/>.
|
||||||
/// </summary>
|
/// </summary>
|
||||||
/// <remarks>
|
/// <remarks>
|
||||||
/// Once set, this can *only* be cancelled by rewinding, ie. if ScoreProcessor.HasCompleted becomes <c>false</c>.
|
/// Once set, this can *only* be cancelled by rewinding, ie. if <see cref="JudgementProcessor.HasCompleted"/>ScoreProcessor.HasCompleted</see> becomes <see langword="false"/>.
|
||||||
/// Even if the user requests an exit, it will forcefully proceed to the results screen (see special case in <see cref="OnExiting"/>).
|
/// Even if the user requests an exit, it will forcefully proceed to the results screen (see special case in <see cref="OnExiting"/>).
|
||||||
/// </remarks>
|
/// </remarks>
|
||||||
private ScheduledDelegate resultsDisplayDelegate;
|
private ScheduledDelegate resultsDisplayDelegate;
|
||||||
@ -647,7 +647,7 @@ namespace osu.Game.Screens.Play
|
|||||||
/// <exception cref="InvalidOperationException">Thrown if this method is called more than once without changing state.</exception>
|
/// <exception cref="InvalidOperationException">Thrown if this method is called more than once without changing state.</exception>
|
||||||
private void updateCompletionState(bool skipStoryboardOutro = false)
|
private void updateCompletionState(bool skipStoryboardOutro = false)
|
||||||
{
|
{
|
||||||
// If this player instance is already exiting upwards, don't attempt any kind of forward progress.
|
// If this player instance is in the middle of an exit, don't attempt any kind of state update.
|
||||||
if (!this.IsCurrentScreen())
|
if (!this.IsCurrentScreen())
|
||||||
return;
|
return;
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user