Skip to content

Commit

Permalink
Edit: comment.
Browse files Browse the repository at this point in the history
  • Loading branch information
Charles Torre committed Mar 7, 2023
1 parent c8b8d76 commit 67aa5d0
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -386,8 +386,8 @@ private float GetProcessMemoryMbPerfCounter(string procName, int procId, Cancell

// Handle the case where supplied process name exceeds the maximum length (64) supported by PerformanceCounter's InstanceName field (.NET Core).
// This should be very rare given this is a Windows/.NET platform restriction and users should understand the limits of the platform they use. However,
// the documentation (and source code comments) are confusing: One (doc) says 128 chars is max value. The other (source code comment) says 127. In reality,
// it is 64 for .NET Core, based on my tests..
// the documentation is confusing: One (doc) says 128 chars is the max value. The other (source code comment) says 127. In reality,
// it is 64 for .NET Core, based on my tests.
if (procName.Length >= MaxInstanceNameLengthTruncated)
{
// Only log this once to limit disk IO noise and log file size.
Expand Down

0 comments on commit 67aa5d0

Please sign in to comment.