Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -229,7 +229,7 @@ public Encoding Encoding
}
}

private Encoding _encoding = ClrFacade.GetDefaultEncoding();
private Encoding _encoding = Encoding.Default;

/// <summary>
/// Gets or sets property that sets append parameter.
Expand Down Expand Up @@ -615,7 +615,7 @@ public Encoding Encoding
}
}

private Encoding _encoding = ClrFacade.GetDefaultEncoding();
private Encoding _encoding = Encoding.Default;

/// <summary>
/// Avoid writing out duplicate warning messages when there are one or more unspecified names.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ public Encoding Encoding
}
}

private Encoding _encoding = ClrFacade.GetDefaultEncoding();
private Encoding _encoding = Encoding.Default;

/// <summary>
/// Gets or sets count of bytes to read from the input stream.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,7 @@ public Encoding Encoding
}
}

private Encoding _encoding = ClrFacade.GetDefaultEncoding();
private Encoding _encoding = Encoding.Default;

/// <summary>
/// Property that sets append parameter.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -90,7 +90,7 @@ public Encoding Encoding
}
}

private Encoding _encoding = ClrFacade.GetDefaultEncoding();
private Encoding _encoding = Encoding.Default;

#endregion Parameters

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1363,7 +1363,7 @@ public Encoding Encoding
}
}

private Encoding _encoding = ClrFacade.GetDefaultEncoding();
private Encoding _encoding = Encoding.Default;

/// <summary>
/// Gets or sets the number of context lines to collect. If set to a
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -81,7 +81,7 @@ public SwitchParameter Append
[ArgumentToEncodingTransformationAttribute]
[ArgumentEncodingCompletionsAttribute]
[ValidateNotNullOrEmpty]
public Encoding Encoding { get; set; } = ClrFacade.GetDefaultEncoding();
public Encoding Encoding { get; set; } = Encoding.Default;

/// <summary>
/// Variable parameter.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -128,7 +128,7 @@ public Encoding Encoding
}
}

private Encoding _encoding = ClrFacade.GetDefaultEncoding();
private Encoding _encoding = Encoding.Default;

#endregion Command Line Parameters

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -515,9 +515,7 @@ private void ReadScriptContents()
{
using (FileStream readerStream = new FileStream(_path, FileMode.Open, FileAccess.Read))
{
Encoding defaultEncoding = ClrFacade.GetDefaultEncoding();

using (StreamReader scriptReader = new StreamReader(readerStream, defaultEncoding))
using (StreamReader scriptReader = new StreamReader(readerStream, Encoding.Default))
{
_scriptContents = scriptReader.ReadToEnd();
_originalEncoding = scriptReader.CurrentEncoding;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -94,10 +94,9 @@ internal static string ReadScript(string path)
{
using (FileStream readerStream = new FileStream(path, FileMode.Open, FileAccess.Read))
{
Encoding defaultEncoding = ClrFacade.GetDefaultEncoding();
Microsoft.Win32.SafeHandles.SafeFileHandle safeFileHandle = readerStream.SafeFileHandle;

using (StreamReader scriptReader = new StreamReader(readerStream, defaultEncoding))
using (StreamReader scriptReader = new StreamReader(readerStream, Encoding.Default))
{
return scriptReader.ReadToEnd();
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -794,7 +794,7 @@ private bool ReadByteEncoded(bool waitChanges, List<object> blocks, bool readBac
// the changes
if (waitChanges)
{
WaitForChanges(_path, _mode, _access, _share, ClrFacade.GetDefaultEncoding());
WaitForChanges(_path, _mode, _access, _share, Encoding.Default);
byteRead = _stream.ReadByte();
}
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6610,7 +6610,7 @@ public IContentReader GetContentReader(string path)

// Defaults for the file read operation
string delimiter = "\n";
Encoding encoding = ClrFacade.GetDefaultEncoding();
Encoding encoding = Encoding.Default;
bool waitForChanges = false;

bool streamTypeSpecified = false;
Expand Down Expand Up @@ -6792,7 +6792,7 @@ public IContentWriter GetContentWriter(string path)
// If this is true, then the content will be read as bytes
bool usingByteEncoding = false;
bool streamTypeSpecified = false;
Encoding encoding = ClrFacade.GetDefaultEncoding();
Encoding encoding = Encoding.Default;
const FileMode filemode = FileMode.OpenOrCreate;
string streamName = null;
bool suppressNewline = false;
Expand Down Expand Up @@ -7668,7 +7668,7 @@ public Encoding Encoding
}
}

private Encoding _encoding = ClrFacade.GetDefaultEncoding();
private Encoding _encoding = Encoding.Default;

/// <summary>
/// Return file contents as a byte stream or create file from a series of bytes.
Expand Down
21 changes: 2 additions & 19 deletions src/System.Management.Automation/utils/ClrFacade.cs
Original file line number Diff line number Diff line change
Expand Up @@ -100,23 +100,6 @@ private static IEnumerable<Assembly> GetPSVisibleAssemblies()

#region Encoding

/// <summary>
/// Facade for getting default encoding.
/// </summary>
internal static Encoding GetDefaultEncoding()
{
if (s_defaultEncoding == null)
{
// load all available encodings
EncodingRegisterProvider();
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't remember if we have tests for that.

s_defaultEncoding = new UTF8Encoding(false);
}

return s_defaultEncoding;
}

private static volatile Encoding s_defaultEncoding;

/// <summary>
/// Facade for getting OEM encoding
/// OEM encodings work on all platforms, or rather codepage 437 is available on both Windows and Non-Windows.
Expand All @@ -142,7 +125,7 @@ internal static Encoding GetOEMEncoding()

private static void EncodingRegisterProvider()
{
if (s_defaultEncoding == null && s_oemEncoding == null)
if (s_oemEncoding == null)
{
Encoding.RegisterProvider(CodePagesEncodingProvider.Instance);
}
Expand Down Expand Up @@ -263,7 +246,7 @@ private static SecurityZone ReadFromZoneIdentifierDataStream(string filePath)
}

// If we successfully get the zone data stream, try to read the ZoneId information
using (StreamReader zoneDataReader = new StreamReader(zoneDataStream, GetDefaultEncoding()))
using (StreamReader zoneDataReader = new StreamReader(zoneDataStream, Encoding.Default))
{
string line = null;
bool zoneTransferMatched = false;
Expand Down
8 changes: 4 additions & 4 deletions src/System.Management.Automation/utils/EncodingUtils.cs
Original file line number Diff line number Diff line change
Expand Up @@ -33,15 +33,15 @@ internal static class EncodingConversion
{ Ascii, System.Text.Encoding.ASCII },
{ BigEndianUnicode, System.Text.Encoding.BigEndianUnicode },
{ BigEndianUtf32, new UTF32Encoding(bigEndian: true, byteOrderMark: true) },
{ Default, ClrFacade.GetDefaultEncoding() },
{ Default, Encoding.Default },
{ OEM, ClrFacade.GetOEMEncoding() },
{ Unicode, System.Text.Encoding.Unicode },
#pragma warning disable SYSLIB0001
{ Utf7, System.Text.Encoding.UTF7 },
#pragma warning restore SYSLIB0001
{ Utf8, ClrFacade.GetDefaultEncoding() },
{ Utf8, Encoding.Default },
{ Utf8Bom, System.Text.Encoding.UTF8 },
{ Utf8NoBom, ClrFacade.GetDefaultEncoding() },
{ Utf8NoBom, Encoding.Default },
{ Utf32, System.Text.Encoding.UTF32 },
{ String, System.Text.Encoding.Unicode },
{ Unknown, System.Text.Encoding.Unicode },
Expand All @@ -57,7 +57,7 @@ internal static Encoding Convert(Cmdlet cmdlet, string encoding)
if (string.IsNullOrEmpty(encoding))
{
// no parameter passed, default to UTF8
return ClrFacade.GetDefaultEncoding();
return Encoding.Default;
}

Encoding foundEncoding;
Expand Down