Skip to content

Remove empty xml comment lines #7401

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

Merged
iSazonov merged 21 commits into from
Aug 1, 2018
Merged

Remove empty xml comment lines #7401

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
3eacbcf
Remove 1
iSazonov Jul 30, 2018
add6649
Remove 2
iSazonov Jul 30, 2018
831cc1f
Remove 3
iSazonov Jul 30, 2018
454abc6
Remove 4
iSazonov Jul 30, 2018
b4ed616
Remove 5
iSazonov Jul 30, 2018
3542ae6
Remove 6
iSazonov Jul 30, 2018
6932b94
Remove 7
iSazonov Jul 30, 2018
fd9901e
Remove 8
iSazonov Jul 30, 2018
6844f1c
Remove 11
iSazonov Jul 30, 2018
c9345c5
Remove 10
iSazonov Jul 30, 2018
be3b168
Remove 10
iSazonov Jul 30, 2018
8bd47c5
Remove 12
iSazonov Jul 30, 2018
fe8692a
Remove 13
iSazonov Jul 30, 2018
a055e59
Remove 14
iSazonov Jul 30, 2018
fd8f7ec
Remove 15
iSazonov Jul 30, 2018
3068d61
Merge branch 'master' into remove-empty-xml-comment-lines1
iSazonov Jul 30, 2018
2cb5f23
Remove tail 1
iSazonov Jul 31, 2018
5af1144
Remove tail 2
iSazonov Jul 31, 2018
93f37bc
Remove tail 3
iSazonov Jul 31, 2018
7dce9af
Remove tail 4
iSazonov Jul 31, 2018
88e23f5
Address feedback
iSazonov Jul 31, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
6 changes: 0 additions & 6 deletions src/Microsoft.PowerShell.Commands.Management/commands/management/AddContentCommand.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,15 +22,12 @@ public class AddContentCommand : WriteContentCommandBase
/// Seeks to the end of the writer stream in each of the writers in the
/// content holders.
/// </summary>
///
/// <param name="contentHolders">
/// The content holders that contain the writers to be moved.
/// </param>
///
/// <exception cref="ProviderInvocationException">
/// If calling Seek on the content writer throws an exception.
/// </exception>
///
internal override void SeekContentPosition(List<ContentHolder> contentHolders)
{
foreach (ContentHolder holder in contentHolders)
Expand Down Expand Up @@ -68,15 +65,12 @@ internal override void SeekContentPosition(List<ContentHolder> contentHolders)
/// <summary>
/// Makes the call to ShouldProcess with appropriate action and target strings.
/// </summary>
///
/// <param name="path">
/// The path to the item on which the content will be added.
/// </param>
///
/// <returns>
/// True if the action should continue or false otherwise.
/// </returns>
///
internal override bool CallShouldProcess(string path)
{
string action = NavigationResources.AddContentAction;
Expand Down
3 changes: 0 additions & 3 deletions src/Microsoft.PowerShell.Commands.Management/commands/management/ClearContentCommand.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -81,16 +81,13 @@ protected override bool ProviderSupportsShouldProcess
/// that require dynamic parameters should override this method and return the
/// dynamic parameter object.
/// </summary>
///
/// <param name="context">
/// The context under which the command is running.
/// </param>
///
/// <returns>
/// An object representing the dynamic parameters for the cmdlet or null if there
/// are none.
/// </returns>
///
internal override object GetDynamicParameters(CmdletProviderContext context)
{
if (Path != null && Path.Length > 0)
Expand Down
4 changes: 0 additions & 4 deletions src/Microsoft.PowerShell.Commands.Management/commands/management/ClearPropertyCommand.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,6 @@ public string[] LiteralPath
/// <summary>
/// The properties to clear from the item
/// </summary>
///
[Parameter(Position = 1, Mandatory = true, ValueFromPipelineByPropertyName = true)]
public string Name
{
Expand All @@ -77,16 +76,13 @@ public string Name
/// that require dynamic parameters should override this method and return the
/// dynamic parameter object.
/// </summary>
///
/// <param name="context">
/// The context under which the command is running.
/// </param>
///
/// <returns>
/// An object representing the dynamic parameters for the cmdlet or null if there
/// are none.
/// </returns>
///
internal override object GetDynamicParameters(CmdletProviderContext context)
{
Collection<string> propertyCollection = new Collection<string>();
Expand Down
8 changes: 0 additions & 8 deletions src/Microsoft.PowerShell.Commands.Management/commands/management/Computer.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,6 @@ public RestartComputerTimeoutException() : base() { }
/// <summary>
/// Constructs a RestartComputerTimeoutException
/// </summary>
///
/// <param name="message">
/// The message used in the exception.
/// </param>
Expand All @@ -86,11 +85,9 @@ public RestartComputerTimeoutException(string message) : base(message) { }
/// <summary>
/// Constructs a RestartComputerTimeoutException
/// </summary>
///
/// <param name="message">
/// The message used in the exception.
/// </param>
///
/// <param name="innerException">
/// An exception that led to this exception.
/// </param>
Expand All @@ -100,11 +97,9 @@ public RestartComputerTimeoutException(string message, Exception innerException)
/// <summary>
/// Serialization constructor for class RestartComputerTimeoutException
/// </summary>
///
/// <param name="info">
/// serialization information
/// </param>
///
/// <param name="context">
/// streaming context
/// </param>
Expand All @@ -123,11 +118,9 @@ private RestartComputerTimeoutException(SerializationInfo info, StreamingContext
/// <summary>
/// Serializes the RestartComputerTimeoutException.
/// </summary>
///
/// <param name="info">
/// serialization information
/// </param>
///
/// <param name="context">
/// streaming context
/// </param>
Expand Down Expand Up @@ -1826,7 +1819,6 @@ internal static string GetRandomPassword(int passwordLength)

/// <summary>
/// Gets the Scope
///
/// </summary>
/// <param name="computer"></param>
/// <param name="namespaceParameter"></param>
Expand Down
31 changes: 0 additions & 31 deletions src/Microsoft.PowerShell.Commands.Management/commands/management/ContentCommandBase.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,6 @@ public override string[] Exclude
/// <summary>
/// Gets or sets the force property
/// </summary>
///
/// <remarks>
/// Gives the provider guidance on how vigorous it should be about performing
/// the operation. If true, the provider should do everything possible to perform
Expand All @@ -84,7 +83,6 @@ public override string[] Exclude
/// the destination is read-only, if force is true, the provider should copy over
/// the existing read-only file. If force is false, the provider should write an error.
/// </remarks>
///
[Parameter]
public override SwitchParameter Force
{
Expand All @@ -104,29 +102,23 @@ public override SwitchParameter Force
/// An array of content holder objects that contain the path information
/// and content readers/writers for the item represented by the path information.
/// </summary>
///
internal List<ContentHolder> contentStreams = new List<ContentHolder>();

/// <summary>
/// Wraps the content into a PSObject and adds context information as notes
/// </summary>
///
/// <param name="content">
/// The content being written out.
/// </param>
///
/// <param name="readCount">
/// The number of blocks that have been read so far.
/// </param>
///
/// <param name="pathInfo">
/// The context the content was retrieved from.
/// </param>
///
/// <param name="context">
/// The context the command is being run under.
/// </param>
///
internal void WriteContentObject(object content, long readCount, PathInfo pathInfo, CmdletProviderContext context)
{
Dbg.Diagnostics.Assert(
Expand Down Expand Up @@ -250,11 +242,9 @@ internal class ContentPathsCache
/// <summary>
/// Constructs a content cache item.
/// </summary>
///
/// <param name="pathInfo">
/// The path information for which the cache will be bound.
/// </param>
///
public ContentPathsCache(PathInfo pathInfo)
{
PathInfo = pathInfo;
Expand All @@ -263,51 +253,42 @@ public ContentPathsCache(PathInfo pathInfo)
/// <summary>
/// The path information for the cached item.
/// </summary>
///
public PathInfo PathInfo { get; }

/// <summary>
/// The cached PSPath of the item.
/// </summary>
///
public String PSPath { get; set; }

/// <summary>
/// The cached parent path of the item.
/// </summary>
///
public String ParentPath { get; set; }

/// <summary>
/// The cached drive for the item.
/// </summary>
///
public PSDriveInfo Drive { get; set; }

/// <summary>
/// The cached provider of the item.
/// </summary>
///
public ProviderInfo Provider { get; set; }

/// <summary>
/// The cached child name of the item.
/// </summary>
///
public String ChildName { get; set; }

/// <summary>
/// Attaches the cached notes to the specified PSObject.
/// </summary>
///
/// <param name="content">
/// The PSObject to attached the cached notes to.
/// </param>
///
/// <returns>
/// The PSObject that was passed in with the cached notes added.
/// </returns>
///
public PSObject AttachNotes(PSObject content)
{
// Construct a provider qualified path as the Path note
Expand Down Expand Up @@ -351,7 +332,6 @@ public PSObject AttachNotes(PSObject content)
/// A struct to hold the path information and the content readers/writers
/// for an item.
/// </summary>
///
internal struct ContentHolder
{
internal ContentHolder(
Expand Down Expand Up @@ -468,16 +448,13 @@ internal void CloseContent(List<ContentHolder> contentHolders, bool disposing)
/// Overridden by derived classes to support ShouldProcess with
/// the appropriate information.
/// </summary>
///
/// <param name="path">
/// The path to the item from which the content writer will be
/// retrieved.
/// </param>
///
/// <returns>
/// True if the action should continue or false otherwise.
/// </returns>
///
internal virtual bool CallShouldProcess(string path)
{
return true;
Expand All @@ -486,11 +463,9 @@ internal virtual bool CallShouldProcess(string path)
/// <summary>
/// Gets the IContentReaders for the current path(s)
/// </summary>
///
/// <returns>
/// An array of IContentReaders for the current path(s)
/// </returns>
///
internal List<ContentHolder> GetContentReaders(
string[] readerPaths,
CmdletProviderContext currentCommandContext)
Expand Down Expand Up @@ -572,28 +547,22 @@ internal List<ContentHolder> GetContentReaders(
/// <summary>
/// Resolves the specified paths to PathInfo objects
/// </summary>
///
/// <param name="pathsToResolve">
/// The paths to be resolved. Each path may contain glob characters.
/// </param>
///
/// <param name="allowNonexistingPaths">
/// If true, resolves the path even if it doesn't exist.
/// </param>
///
/// <param name="allowEmptyResult">
/// If true, allows a wildcard that returns no results.
/// </param>
///
/// <param name="currentCommandContext">
/// The context under which the command is running.
/// </param>
///
/// <returns>
/// An array of PathInfo objects that are the resolved paths for the
/// <paramref name="pathsToResolve"/> parameter.
/// </returns>
///
internal Collection<PathInfo> ResolvePaths(
string[] pathsToResolve,
bool allowNonexistingPaths,
Expand Down
2 changes: 0 additions & 2 deletions src/Microsoft.PowerShell.Commands.Management/commands/management/ControlPanelItemCommand.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -604,7 +604,6 @@ public string[] Category
#endregion "Parameters"

/// <summary>
///
/// </summary>
protected override void ProcessRecord()
{
Expand Down Expand Up @@ -721,7 +720,6 @@ public ControlPanelItem[] InputObject
#endregion "Parameters"

/// <summary>
///
/// </summary>
protected override void ProcessRecord()
{
Expand Down
5 changes: 0 additions & 5 deletions src/Microsoft.PowerShell.Commands.Management/commands/management/CopyPropertyCommand.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -45,15 +45,13 @@ public string[] LiteralPath
/// <summary>
/// The name of the property to create on the item
/// </summary>
///
[Parameter(Position = 2, Mandatory = true, ValueFromPipelineByPropertyName = true)]
[Alias("PSProperty")]
public string Name { get; set; }

/// <summary>
/// The path to the destination item to copy the property to.
/// </summary>
///
[Parameter(Mandatory = true, Position = 1, ValueFromPipelineByPropertyName = true)]
public string Destination { get; set; }

Expand All @@ -62,16 +60,13 @@ public string[] LiteralPath
/// that require dynamic parameters should override this method and return the
/// dynamic parameter object.
/// </summary>
///
/// <param name="context">
/// The context under which the command is running.
/// </param>
///
/// <returns>
/// An object representing the dynamic parameters for the cmdlet or null if there
/// are none.
/// </returns>
///
internal override object GetDynamicParameters(CmdletProviderContext context)
{
if (Path != null && Path.Length > 0)
Expand Down
1 change: 0 additions & 1 deletion src/Microsoft.PowerShell.Commands.Management/commands/management/Eventlog.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1141,7 +1141,6 @@ public class NewEventLogCommand : PSCmdlet
/// <summary>
/// The following is the definition of the input parameter "LogName".
/// Specifies the name of the log
///
/// </summary>
[Parameter(Mandatory = true,
Position = 0)]
Expand Down
Loading