Skip to content

Commit

Permalink
Fix SA1515,1616: provide comments on return
Browse files Browse the repository at this point in the history
  • Loading branch information
lewischeng-ms committed Jul 14, 2015
1 parent 9e0567d commit 5ad941f
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 1 deletion.
6 changes: 6 additions & 0 deletions src/Microsoft.Restier.Core/Query/QueryRequest.cs
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,9 @@ public QueryRequest(IQueryable query, bool? includeTotalCount = null)
/// Indicates if the total number of items should be retrieved
/// when the result has been filtered using paging operators.
/// </param>
/// <returns>
/// The created instance of the <see cref="QueryRequest"/> class.
/// </returns>
public static QueryRequest Create<TElement, TResult>(
IQueryable<TElement> query,
Expression<Func<IQueryable<TElement>, TResult>> singularExpression,
Expand All @@ -78,6 +81,9 @@ public static QueryRequest Create<TElement, TResult>(
/// Indicates if the total number of items should be retrieved
/// when the result has been filtered using paging operators.
/// </param>
/// <returns>
/// The created instance of the <see cref="QueryRequest"/> class.
/// </returns>
public static QueryRequest Create(
IQueryable query,
LambdaExpression singularExpression,
Expand Down
7 changes: 6 additions & 1 deletion src/Microsoft.Restier.Core/Submit/ChangeSetEntry.cs
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,9 @@ internal ChangeSetEntry(ChangeSetEntryType type)
/// <summary>
/// Indicates if this change set entry is in a changed state.
/// </summary>
/// <returns></returns>
/// <returns>
/// If this change set entry is in a changed state.
/// </returns>
public bool HasChanged()
{
return this.ChangeSetEntityState == DynamicChangeSetEntityState.Changed ||
Expand Down Expand Up @@ -425,6 +427,9 @@ public ActionInvocationEntry(
/// <summary>
/// Gets an array of the arguments to pass to the action.
/// </summary>
/// <returns>
/// An array of the arguments to pass to the action.
/// </returns>
public object[] GetArgumentArray()
{
if (this.Arguments == null)
Expand Down
3 changes: 3 additions & 0 deletions src/Microsoft.Restier.Core/Submit/ValidationResult.cs
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,9 @@ public class ValidationResult
/// <summary>
/// Returns the string that represents this validation result.
/// </summary>
/// <returns>
/// The string that represents this validation result.
/// </returns>
public override string ToString()
{
return this.Message;
Expand Down

0 comments on commit 5ad941f

Please sign in to comment.