Skip to content

Commit 7c0262c

Browse files
review: use string.Replace instead of for loop
1 parent 85d84aa commit 7c0262c

File tree

2 files changed

+3
-17
lines changed

2 files changed

+3
-17
lines changed

sources/ClangSharp.PInvokeGenerator/CSharp/CSharpOutputBuilder.cs

Lines changed: 1 addition & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -130,21 +130,7 @@ public void WriteLabel(string name)
130130
WriteLine(':');
131131
}
132132

133-
public void WriteNumberLiteral(ReadOnlySpan<char> value)
134-
{
135-
for (var i = 0; i < value.Length; i++)
136-
{
137-
var c = value[i];
138-
if (c == '\'')
139-
{
140-
Write('_');
141-
}
142-
else
143-
{
144-
Write(c);
145-
}
146-
}
147-
}
133+
public void WriteNumberLiteral(string value) => Write(value.Replace('\'', '_'));
148134

149135
public void WriteLine<T>(T value)
150136
{

sources/ClangSharp.PInvokeGenerator/PInvokeGenerator.VisitStmt.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1201,7 +1201,7 @@ private void VisitFloatingLiteral(FloatingLiteral floatingLiteral)
12011201
var outputBuilder = StartCSharpCode();
12021202
if (floatingLiteral.ValueString.EndsWith(".f", StringComparison.Ordinal))
12031203
{
1204-
outputBuilder.WriteNumberLiteral(floatingLiteral.ValueString.AsSpan()[..^1]);
1204+
outputBuilder.WriteNumberLiteral(floatingLiteral.ValueString[..^1]);
12051205
outputBuilder.Write("0f");
12061206
}
12071207
else
@@ -1994,7 +1994,7 @@ void HandleUnmanagedConstant(CSharpOutputBuilder outputBuilder, InitListExpr ini
19941994

19951995
private void VisitIntegerLiteral(IntegerLiteral integerLiteral)
19961996
{
1997-
var valueString = integerLiteral.ValueString.AsSpan();
1997+
var valueString = integerLiteral.ValueString;
19981998
var valueSuffix = "";
19991999

20002000
if (valueString.EndsWith("ui8", StringComparison.OrdinalIgnoreCase))

0 commit comments

Comments
 (0)