Question about setting the SignalTime of ElapsedEventArgs

[LeaFrock]

When I read the source codes of System.Timers.Timer, I notice this line:

ElapsedEventArgs elapsedEventArgs = new ElapsedEventArgs(DateTime.UtcNow.ToFileTime());

And in the constructor of ElapsedEventArgs:

SignalTime = DateTime.FromFileTime(fileTime);

Is that necessary or should be optimized? Any comments welcomed.

[stephentoub]

This is a holdover from how it was constructed in .NET Framework. It could be changed to just pass DateTime.UtcNow and have the constructor accept and store a DateTime, rather than roundtripping through filetime.

[LeaFrock]

Well, need I make a pull request? Or you will handle that?

[stephentoub]

Please feel free to do so. Thanks.