h1or4dme8/osu-lazer
1
0
Fork 0
mirror of https://github.com/ppy/osu.git synced 2025-03-15 22:19:31 +08:00
Code Issues Releases Wiki Activity

Move previousContext assign within lock to make things look safer

Browse Source 
Not an actual requirement, but no harm.
This commit is contained in:
Dean Herbert 2021-07-05 10:49:55 +09:00
parent 3291f15dcc
commit f510ef9153
1 changed files with 6 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
osu.Game/Database/RealmContextFactory.cs
View File

@ -2,6 +2,7 @@
// See the LICENCE file in the repository root for full licence text. // See the LICENCE file in the repository root for full licence text.
using System; using System;
using System.Diagnostics;
using System.Threading; using System.Threading;
using osu.Framework.Allocation; using osu.Framework.Allocation;
using osu.Framework.Development; using osu.Framework.Development;
@ -166,11 +167,15 @@ namespace osu.Game.Database
private void flushContexts() private void flushContexts()
{ {
Logger.Log(@"Flushing realm contexts...", LoggingTarget.Database); Logger.Log(@"Flushing realm contexts...", LoggingTarget.Database);
Debug.Assert(blockingLock.CurrentCount == 0);
var previousContext = context; Realm previousContext;
lock (updateContextLock) lock (updateContextLock)
{
previousContext = context;
context = null; context = null;
}
// wait for all threaded usages to finish // wait for all threaded usages to finish
while (active_usages.Value > 0) while (active_usages.Value > 0)