Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

New rule S6561: Do not use DateTime.Now for benchmarking or timing operations #7083

Closed
2 tasks
mary-georgiou-sonarsource opened this issue Apr 14, 2023 · 0 comments · Fixed by #7494
Closed
2 tasks

New rule S6561: Do not use DateTime.Now for benchmarking or timing operations #7083

mary-georgiou-sonarsource opened this issue Apr 14, 2023 · 0 comments · Fixed by #7494
Assignees
@cristian-ambrosini-sonarsource
Labels
Area: C# C# rules related issues. Area: VB.NET VB.NET rules related issues. Type: New Rule Implementation for a rule that HAS been specified.
Milestone
9.5

Comments

@mary-georgiou-sonarsource
Copy link
Contributor

Implement new rule S6561: Do not use DateTime.Now for benchmarking or timing operations.

This rule should raise when Now or UtcNow properties are called multiple times in the same method and they are used later in arithmetic operations.

  • Implement Rule S6561 for C#
  • Implement Rule S6561 for VB.NET
@mary-georgiou-sonarsource mary-georgiou-sonarsource added Area: VB.NET VB.NET rules related issues. Area: C# C# rules related issues. Type: New Rule Implementation for a rule that HAS been specified. labels Apr 14, 2023
@csaba-sagi-sonarsource csaba-sagi-sonarsource changed the title New rule S6561: Do not use DateTime.Now for benchmarking or timing operations New rule S6561: Do not use DateTime.Now for benchmarking or timing operations Apr 14, 2023
@costin-zaharia-sonarsource costin-zaharia-sonarsource added this to the 9.4 milestone Jun 8, 2023
@costin-zaharia-sonarsource costin-zaharia-sonarsource removed this from the 9.4 milestone Jun 12, 2023
Merged
@csaba-sagi-sonarsource csaba-sagi-sonarsource removed their assignment Jun 26, 2023
@cristian-ambrosini-sonarsource cristian-ambrosini-sonarsource added this to the 9.5 milestone Jul 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cristian-ambrosini-sonarsource cristian-ambrosini-sonarsource

Labels
Area: C# C# rules related issues. Area: VB.NET VB.NET rules related issues. Type: New Rule Implementation for a rule that HAS been specified.
Projects
None yet
Milestone
9.5
5 participants
@costin-zaharia-sonarsource @csaba-sagi-sonarsource @mary-georgiou-sonarsource @cristian-ambrosini-sonarsource @zsolt-kolbay-sonarsource