Removing items from job queue is now done while a lock is acquired.

pull/7/merge
kay.one 13 years ago
parent 9cb33770a0
commit 79b2fad6e3

@ -1,4 +1,5 @@
using System; //https://github.com/kayone/NzbDrone/blob/master/NzbDrone.Core/Providers/Jobs/JobProvider.cs
using System;
using System.Collections.Generic; using System.Collections.Generic;
using System.ComponentModel; using System.ComponentModel;
using System.Diagnostics; using System.Diagnostics;
@ -201,6 +202,7 @@ namespace NzbDrone.Core.Providers.Jobs
if (Queue.Count != 0) if (Queue.Count != 0)
{ {
job = Queue.First(); job = Queue.First();
Queue.Remove(job);
} }
} }
@ -214,13 +216,6 @@ namespace NzbDrone.Core.Providers.Jobs
{ {
Logger.FatalException("An error has occurred while processing queued job.", e); Logger.FatalException("An error has occurred while processing queued job.", e);
} }
finally
{
if (job != null)
{
Queue.Remove(job);
}
}
} }
} while (Queue.Count != 0); } while (Queue.Count != 0);

Loading…
Cancel
Save