Gui.cs: MainLoop.RunTimers(): System.ArgumentException: 'An entry with the same key already exists

[brian-pickens-web]

Hello,

I'm having trouble in the MainLoop execution which is throwing an exception in the RunTimers() method. It is failing when trying to add the Timeout back to be run again in the next iteration. The issue seems to be that the calculated ticks can be a duplicate with what is already in the SortedList which then throws the exception. I'm thinking that when RunTimers() adds the Timeouts back for the next iteration, it causes DateTime.UtcNow to possibly be the same for multiple timeouts. So then any timeouts with the same TimeSpan defined would then cause a collision.

[brian-pickens-web]

One possible solution might be to implement a custom IComparer that allows for duplicate keys.

[BDisp]

It is failing when trying to add the Timeout back to be run again in the next iteration.

Can you show how you are doing this please.

[brian-pickens-web]

Ill see if I can come up with a simple recreation.

[brian-pickens-web]

This code triggers the exception before the MainLoop can even run.

```c#
public static void Main()
{
Application.Init();

var window = new Window() { Height = Dim.Fill(), Width = Dim.Fill() };
Application.Top.Add(window);

var progress1 = new ProgressBar() { Height = 1, Width = Dim.Fill(), Y = 1 };
var progress2 = new ProgressBar() { Height = 1, Width = Dim.Fill(), Y = 2 };

window.Add(progress1);
window.Add(progress2);

Application.MainLoop.AddTimeout(TimeSpan.FromMilliseconds(100), loop =>
{
    progress1.Pulse();
    return true;
});

Application.MainLoop.AddTimeout(TimeSpan.FromMilliseconds(100), loop =>
{
    progress2.Pulse();
    return true;
});

Application.Run();

}
```

[brian-pickens-web]

This code snippet will make it through and execute Application.Run(), but when the MainLoop attempts to setup the next iteration of the Timeouts, thats when it will throw the exception.

```c#
public static void Main()
{
Application.Init();

var window = new Window() { Height = Dim.Fill(), Width = Dim.Fill() };
Application.Top.Add(window);

var progress1 = new ProgressBar() { Height = 1, Width = Dim.Fill(), Y = 1 };
var progress2 = new ProgressBar() { Height = 1, Width = Dim.Fill(), Y = 2 };
var progress3 = new ProgressBar() { Height = 1, Width = Dim.Fill(), Y = 3 };

window.Add(progress1);
window.Add(progress2);
window.Add(progress3);

Application.MainLoop.AddTimeout(TimeSpan.FromMilliseconds(100), loop =>
{
    progress1.Pulse();
    return true;
});

Thread.Sleep(1000);
Application.MainLoop.AddTimeout(TimeSpan.FromMilliseconds(100), loop =>
{
    progress2.Pulse();
    return true;
});

Thread.Sleep(1000);
Application.MainLoop.AddTimeout(TimeSpan.FromMilliseconds(100), loop =>
{
    progress3.Pulse();
    return true;
});

Application.Run();

}
```

[BDisp]

None of them are giving any error. Do you are using the nuget package or the Gui repository?

[brian-pickens-web]

I am running a clone of master.

[BDisp]

I haven't any exception.

[BDisp]

This might be something you are doing that there isn't in these snippets that is causing the exception.

[brian-pickens-web]

I’ll pull from master again and rerun my test but I’ll post the whole Program class along with my .csproj when I have a chance.

[tig]

Does this still repro? I'm closing for now. Please comment if it's still repro and we can reopen.