Skip to content

Commit 653b79a

Browse files
chore: Use Convert.ToHexString (#2322)
* Use Convert.ToHexString Use `Convert.ToHexString()` when targeting .NET 8 instead of using a `StringBuilder`. * Update src/Microsoft.OpenApi/Models/OpenApiDocument.cs --------- Co-authored-by: Vincent Biret <[email protected]>
1 parent e4f5045 commit 653b79a

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

src/Microsoft.OpenApi/Models/OpenApiDocument.cs

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -544,13 +544,17 @@ private static string ConvertByteArrayToString(byte[] hash)
544544
{
545545
// Build the final string by converting each byte
546546
// into hex and appending it to a StringBuilder
547+
#if NET5_0_OR_GREATER
548+
return Convert.ToHexString(hash);
549+
#else
547550
var sb = new StringBuilder();
548551
for (var i = 0; i < hash.Length; i++)
549552
{
550553
sb.Append(hash[i].ToString("X2"));
551554
}
552555

553556
return sb.ToString();
557+
#endif
554558
}
555559

556560
/// <summary>

0 commit comments

Comments
 (0)