Change version to 1.5.12

ChangeLog.md

@@ -1,3 +1,21 @@
+### 1.5.12 (2017-10-19)
+
+#### Analyzers
+
+* Add analyzer ReturnTaskInsteadOfNull (RCS1210).
+* Add analyzer RemoveUnnecessaryElseClause (RCS1211).
+* Remove analyzer SimplifyLambdaExpressionParameterList (RCS1022).
+
+#### Refactorings
+
+* Replace refactoring ChangeMemberTypeAccordingToReturnExpression (RR0019) with code fix.
+* Replace refactoring ChangeMemberTypeAccordingToYieldReturnExpression (RR0020) with code fix.
+* Replace refactoring AddDefaultValueToReturnStatement (RR0008) with code fix.
+
+#### Code Fixes
+
+ * Add code fix for CS0126, CS0139, CS0713 and CS1750.
+
 ### 1.5.10 (2017-10-04)
 
 #### Code Fixes

docs/analyzers/RCS1039.md

@@ -1,4 +1,4 @@
-# RCS1039: Remove empty attribute argument list
+# RCS1039: Remove argument list from attribute
 
 Property | Value
 --- | --- 
@@ -14,14 +14,14 @@ Supports Fade-Out Analyzer | no
 ### SuppressMessageAttribute
 
 ```csharp
-[assembly: SuppressMessage("Redundancy", "RCS1039:Remove empty attribute argument list.", Justification = "<Pending>")]
+[assembly: SuppressMessage("Redundancy", "RCS1039:Remove argument list from attribute.", Justification = "<Pending>")]
 ```
 
 ### \#pragma
 
 ```csharp
-#pragma warning disable RCS1039 // Remove empty attribute argument list.
-#pragma warning restore RCS1039 // Remove empty attribute argument list.
+#pragma warning disable RCS1039 // Remove argument list from attribute.
+#pragma warning restore RCS1039 // Remove argument list from attribute.
 ```
 
 ### Ruleset

docs/analyzers/RCS1050.md

@@ -1,10 +1,10 @@
-# RCS1050: Add constructor argument list
+# RCS1050: Add argument list to object creation expression
 
 Property | Value
 --- | --- 
 Id | RCS1050
 Category | Style
-Default Severity | Warning
+Default Severity | Info
 Enabled by Default | no
 Supports Fade-Out | no
 Supports Fade-Out Analyzer | no
@@ -14,14 +14,14 @@ Supports Fade-Out Analyzer | no
 ### SuppressMessageAttribute
 
 ```csharp
-[assembly: SuppressMessage("Style", "RCS1050:Add constructor argument list.", Justification = "<Pending>")]
+[assembly: SuppressMessage("Style", "RCS1050:Add argument list to object creation expression.", Justification = "<Pending>")]
 ```
 
 ### \#pragma
 
 ```csharp
-#pragma warning disable RCS1050 // Add constructor argument list.
-#pragma warning restore RCS1050 // Add constructor argument list.
+#pragma warning disable RCS1050 // Add argument list to object creation expression.
+#pragma warning restore RCS1050 // Add argument list to object creation expression.
 ```
 
 ### Ruleset

docs/analyzers/RCS1067.md

@@ -1,4 +1,4 @@
-# RCS1067: Remove empty argument list
+# RCS1067: Remove argument list from object creation expression
 
 Property | Value
 --- | --- 
@@ -14,14 +14,14 @@ Supports Fade-Out Analyzer | no
 ### SuppressMessageAttribute
 
 ```csharp
-[assembly: SuppressMessage("Style", "RCS1067:Remove empty argument list.", Justification = "<Pending>")]
+[assembly: SuppressMessage("Style", "RCS1067:Remove argument list from object creation expression.", Justification = "<Pending>")]
 ```
 
 ### \#pragma
 
 ```csharp
-#pragma warning disable RCS1067 // Remove empty argument list.
-#pragma warning restore RCS1067 // Remove empty argument list.
+#pragma warning disable RCS1067 // Remove argument list from object creation expression.
+#pragma warning restore RCS1067 // Remove argument list from object creation expression.
 ```
 
 ### Ruleset

docs/analyzers/RCS1210.md

@@ -3,8 +3,8 @@
 Property | Value
 --- | --- 
 Id | RCS1210
-Category | Reliability
-Default Severity | Info
+Category | Usage
+Default Severity | Warning
 Enabled by Default | yes
 Supports Fade-Out | no
 Supports Fade-Out Analyzer | no
@@ -14,7 +14,7 @@ Supports Fade-Out Analyzer | no
 ### SuppressMessageAttribute
 
 ```csharp
-[assembly: SuppressMessage("Reliability", "RCS1210:Return Task.FromResult instead of returning null.", Justification = "<Pending>")]
+[assembly: SuppressMessage("Usage", "RCS1210:Return Task.FromResult instead of returning null.", Justification = "<Pending>")]
 ```
 
 ### \#pragma

source/Analyzers/AnalyzersByCategory.md

@@ -87,7 +87,7 @@ Readability|[Use explicit type instead of 'var' \(when the type is obvious\)](..
 Readability|[Use regular string literal instead of verbatim string literal](../../docs/analyzers/RCS1192.md)|RCS1192|x
 Redundancy|[Avoid interpolated string with no interpolation](../../docs/analyzers/RCS1062.md)|RCS1062|x
 Redundancy|[Avoid semicolon at the end of declaration](../../docs/analyzers/RCS1055.md)|RCS1055|x
-Redundancy|[Remove empty attribute argument list](../../docs/analyzers/RCS1039.md)|RCS1039|x
+Redundancy|[Remove argument list from attribute](../../docs/analyzers/RCS1039.md)|RCS1039|x
 Redundancy|[Remove empty destructor](../../docs/analyzers/RCS1106.md)|RCS1106|x
 Redundancy|[Remove empty else clause](../../docs/analyzers/RCS1040.md)|RCS1040|
 Redundancy|[Remove empty finally clause](../../docs/analyzers/RCS1066.md)|RCS1066|x
@@ -120,6 +120,7 @@ Redundancy|[Remove redundant 'ToCharArray' call](../../docs/analyzers/RCS1107.md
 Redundancy|[Remove redundant 'ToString' call](../../docs/analyzers/RCS1097.md)|RCS1097|x
 Redundancy|[Remove trailing white\-space](../../docs/analyzers/RCS1037.md)|RCS1037|x
 Redundancy|[Remove unnecessary case label](../../docs/analyzers/RCS1069.md)|RCS1069|x
+Redundancy|[Remove unnecessary else clause](../../docs/analyzers/RCS1211.md)|RCS1211|x
 Redundancy|[Unused parameter](../../docs/analyzers/RCS1163.md)|RCS1163|x
 Redundancy|[Unused this parameter](../../docs/analyzers/RCS1175.md)|RCS1175|x
 Redundancy|[Unused type parameter](../../docs/analyzers/RCS1164.md)|RCS1164|x
@@ -140,7 +141,6 @@ Simplification|[Simplify boolean comparison](../../docs/analyzers/RCS1049.md)|RC
 Simplification|[Simplify boolean expression](../../docs/analyzers/RCS1199.md)|RCS1199|x
 Simplification|[Simplify coalesce expression](../../docs/analyzers/RCS1143.md)|RCS1143|x
 Simplification|[Simplify conditional expression](../../docs/analyzers/RCS1104.md)|RCS1104|x
-Simplification|[Simplify lambda expression parameter list](../../docs/analyzers/RCS1022.md)|RCS1022|
 Simplification|[Simplify lambda expression](../../docs/analyzers/RCS1021.md)|RCS1021|x
 Simplification|[Simplify lazily initialized property](../../docs/analyzers/RCS1171.md)|RCS1171|x
 Simplification|[Simplify LINQ method chain](../../docs/analyzers/RCS1077.md)|RCS1077|x
@@ -161,10 +161,10 @@ Simplification|[Use return instead of assignment](../../docs/analyzers/RCS1179.m
 Simplification|[Use 'var' instead of explicit type \(in foreach\)](../../docs/analyzers/RCS1177.md)|RCS1177|
 Simplification|[Use 'var' instead of explicit type \(when the type is not obvious\)](../../docs/analyzers/RCS1176.md)|RCS1176|
 Simplification|[Use 'var' instead of explicit type \(when the type is obvious\)](../../docs/analyzers/RCS1010.md)|RCS1010|x
+Style|[Add argument list to object creation expression](../../docs/analyzers/RCS1050.md)|RCS1050|
 Style|[Add braces to if\-else](../../docs/analyzers/RCS1003.md)|RCS1003|x
 Style|[Add braces to switch section with multiple statements](../../docs/analyzers/RCS1111.md)|RCS1111|
 Style|[Add braces](../../docs/analyzers/RCS1001.md)|RCS1001|x
-Style|[Add constructor argument list](../../docs/analyzers/RCS1050.md)|RCS1050|
 Style|[Avoid embedded statement in if\-else](../../docs/analyzers/RCS1126.md)|RCS1126|
 Style|[Avoid embedded statement](../../docs/analyzers/RCS1007.md)|RCS1007|
 Style|[Avoid multiline expression body](../../docs/analyzers/RCS1017.md)|RCS1017|
@@ -174,12 +174,13 @@ Style|[Avoid usage of while statement to create an inifinite loop](../../docs/an
 Style|[Call extension method as instance method](../../docs/analyzers/RCS1196.md)|RCS1196|x
 Style|[Parenthesize condition in conditional expression](../../docs/analyzers/RCS1051.md)|RCS1051|
 Style|[Reduce if nesting](../../docs/analyzers/RCS1208.md)|RCS1208|
+Style|[Remove argument list from object creation expression](../../docs/analyzers/RCS1067.md)|RCS1067|
 Style|[Remove braces from if\-else](../../docs/analyzers/RCS1004.md)|RCS1004|
 Style|[Remove braces](../../docs/analyzers/RCS1002.md)|RCS1002|
-Style|[Remove empty argument list](../../docs/analyzers/RCS1067.md)|RCS1067|
 Usage|[Avoid NullReferenceException](../../docs/analyzers/RCS1202.md)|RCS1202|x
 Usage|[Call Debug\.Fail instead of Debug\.Assert](../../docs/analyzers/RCS1178.md)|RCS1178|x
 Usage|[Call 'Enumerable\.ThenBy' instead of 'Enumerable\.OrderBy'](../../docs/analyzers/RCS1200.md)|RCS1200|x
+Usage|[Return Task\.FromResult instead of returning null](../../docs/analyzers/RCS1210.md)|RCS1210|x
 Usage|[Unconstrained type parameter checked for null](../../docs/analyzers/RCS1165.md)|RCS1165|x
 Usage|[Use conditional access instead of conditional expression](../../docs/analyzers/RCS1206.md)|RCS1206|x
 Usage|[Use conditional access](../../docs/analyzers/RCS1146.md)|RCS1146|x

source/Analyzers/Properties/AssemblyInfo.cs

@@ -28,6 +28,6 @@
 //
 // You can specify all the values or you can default the Build and Revision Numbers 
 // by using the '*' as shown below:
-[assembly: AssemblyVersion("1.5.11")]
+[assembly: AssemblyVersion("1.5.12")]
 //[assembly: AssemblyFileVersion("1.0.0.0")]
 [assembly: NeutralResourcesLanguage("en-US")]

source/Analyzers/README.md

@@ -22,7 +22,6 @@ RCS1018|[Add default access modifier](../../docs/analyzers/RCS1018.md)|Readabili
 RCS1019|[Reorder modifiers](../../docs/analyzers/RCS1019.md)|Readability|
 RCS1020|[Simplify Nullable\<T\> to T?](../../docs/analyzers/RCS1020.md)|Simplification|x
 RCS1021|[Simplify lambda expression](../../docs/analyzers/RCS1021.md)|Simplification|x
-RCS1022|[Simplify lambda expression parameter list](../../docs/analyzers/RCS1022.md)|Simplification|
 RCS1023|[Format empty block](../../docs/analyzers/RCS1023.md)|Formatting|
 RCS1024|[Format accessor list](../../docs/analyzers/RCS1024.md)|Formatting|
 RCS1025|[Format each enum member on a separate line](../../docs/analyzers/RCS1025.md)|Formatting|
@@ -38,7 +37,7 @@ RCS1035|[Remove redundant comma in initializer](../../docs/analyzers/RCS1035.md)
 RCS1036|[Remove redundant empty line](../../docs/analyzers/RCS1036.md)|Redundancy|x
 RCS1037|[Remove trailing white\-space](../../docs/analyzers/RCS1037.md)|Redundancy|x
 RCS1038|[Remove empty statement](../../docs/analyzers/RCS1038.md)|Redundancy|x
-RCS1039|[Remove empty attribute argument list](../../docs/analyzers/RCS1039.md)|Redundancy|x
+RCS1039|[Remove argument list from attribute](../../docs/analyzers/RCS1039.md)|Redundancy|x
 RCS1040|[Remove empty else clause](../../docs/analyzers/RCS1040.md)|Redundancy|
 RCS1041|[Remove empty initializer](../../docs/analyzers/RCS1041.md)|Redundancy|x
 RCS1042|[Remove enum default underlying type](../../docs/analyzers/RCS1042.md)|Redundancy|x
@@ -49,7 +48,7 @@ RCS1046|[Asynchronous method name should end with 'Async'](../../docs/analyzers/
 RCS1047|[Non\-asynchronous method name should not end with 'Async'](../../docs/analyzers/RCS1047.md)|Naming|x
 RCS1048|[Use lambda expression instead of anonymous method](../../docs/analyzers/RCS1048.md)|Usage|x
 RCS1049|[Simplify boolean comparison](../../docs/analyzers/RCS1049.md)|Simplification|x
-RCS1050|[Add constructor argument list](../../docs/analyzers/RCS1050.md)|Style|
+RCS1050|[Add argument list to object creation expression](../../docs/analyzers/RCS1050.md)|Style|
 RCS1051|[Parenthesize condition in conditional expression](../../docs/analyzers/RCS1051.md)|Style|
 RCS1052|[Declare each attribute separately](../../docs/analyzers/RCS1052.md)|Readability|
 RCS1055|[Avoid semicolon at the end of declaration](../../docs/analyzers/RCS1055.md)|Redundancy|x
@@ -64,7 +63,7 @@ RCS1063|[Avoid usage of do statement to create an infinite loop](../../docs/anal
 RCS1064|[Avoid usage of for statement to create an infinite loop](../../docs/analyzers/RCS1064.md)|Style|
 RCS1065|[Avoid usage of while statement to create an inifinite loop](../../docs/analyzers/RCS1065.md)|Style|
 RCS1066|[Remove empty finally clause](../../docs/analyzers/RCS1066.md)|Redundancy|x
-RCS1067|[Remove empty argument list](../../docs/analyzers/RCS1067.md)|Style|
+RCS1067|[Remove argument list from object creation expression](../../docs/analyzers/RCS1067.md)|Style|
 RCS1068|[Simplify logical not expression](../../docs/analyzers/RCS1068.md)|Simplification|x
 RCS1069|[Remove unnecessary case label](../../docs/analyzers/RCS1069.md)|Redundancy|x
 RCS1070|[Remove redundant default switch section](../../docs/analyzers/RCS1070.md)|Redundancy|x
@@ -193,3 +192,5 @@ RCS1206|[Use conditional access instead of conditional expression](../../docs/an
 RCS1207|[Use method group instead of anonymous function](../../docs/analyzers/RCS1207.md)|Simplification|x
 RCS1208|[Reduce if nesting](../../docs/analyzers/RCS1208.md)|Style|
 RCS1209|[Reorder type parameter constraints](../../docs/analyzers/RCS1209.md)|Readability|x
+RCS1210|[Return Task\.FromResult instead of returning null](../../docs/analyzers/RCS1210.md)|Usage|x
+RCS1211|[Remove unnecessary else clause](../../docs/analyzers/RCS1211.md)|Redundancy|x

source/CodeFixes/CodeFixesByDiagnosticId.md

@@ -5,7 +5,7 @@ Diagnostic | Code Fixes
 [CS0019](http://docs.microsoft.com/en-us/dotnet/csharp/language-reference/compiler-messages/cs0019)|RCF0023
 [CS0021](http://docs.microsoft.com/en-us/dotnet/csharp/misc/cs0021)|RCF0077
 [CS0023](http://docs.microsoft.com/en-us/dotnet/csharp/misc/cs0023)|RCF0039
-[CS0029](http://docs.microsoft.com/en-us/dotnet/csharp/language-reference/compiler-messages/cs0029)|RCF0021, RCF0022
+[CS0029](http://docs.microsoft.com/en-us/dotnet/csharp/language-reference/compiler-messages/cs0029)|RCF0021, RCF0022, RCF0080
 [CS0037](http://docs.microsoft.com/en-us/dotnet/csharp/language-reference/compiler-messages/cs0037)|RCF0058
 [CS0069](http://docs.microsoft.com/en-us/dotnet/csharp/misc/cs0069)|RCF0032
 [CS0077](http://docs.microsoft.com/en-us/dotnet/csharp/misc/cs0077)|RCF0074
@@ -20,12 +20,13 @@ Diagnostic | Code Fixes
 [CS0115](https://docs.microsoft.com/en-us/dotnet/csharp/language-reference/compiler-messages/cs0115)|RCF0055
 [CS0120](http://docs.microsoft.com/en-us/dotnet/csharp/language-reference/compiler-messages/cs0120)|RCF0042
 [CS0123](http://docs.microsoft.com/en-us/dotnet/csharp/misc/cs0123)|RCF0041
-[CS0127](http://docs.microsoft.com/en-us/dotnet/csharp/misc/cs0127)|RCF0052, RCF0057
+[CS0126](http://docs.microsoft.com/en-us/dotnet/csharp/misc/cs0126)|RCF0081
+[CS0127](http://docs.microsoft.com/en-us/dotnet/csharp/misc/cs0127)|RCF0052, RCF0057, RCF0080
 [CS0128](http://docs.microsoft.com/en-us/dotnet/csharp/misc/cs0128)|RCF0059
 [CS0132](http://docs.microsoft.com/en-us/dotnet/csharp/misc/cs0132)|RCF0053
 [CS0133](http://docs.microsoft.com/en-us/dotnet/csharp/misc/cs0133)|RCF0045
 [CS0136](http://docs.microsoft.com/en-us/dotnet/csharp/misc/cs0136)|RCF0059
-[CS0139](http://docs.microsoft.com/en-us/dotnet/csharp/misc/cs0139)|RCF0072
+[CS0139](http://docs.microsoft.com/en-us/dotnet/csharp/misc/cs0139)|RCF0072, RCF0079
 [CS0161](http://docs.microsoft.com/en-us/dotnet/csharp/misc/cs0161)|RCF0020
 [CS0162](http://docs.microsoft.com/en-us/dotnet/csharp/misc/cs0162)|RCF0017
 [CS0163](http://docs.microsoft.com/en-us/dotnet/csharp/language-reference/compiler-messages/cs0163)|RCF0002
@@ -35,15 +36,15 @@ Diagnostic | Code Fixes
 [CS0173](http://docs.microsoft.com/en-us/dotnet/csharp/language-reference/compiler-messages/cs0173)|RCF0063
 [CS0177](http://docs.microsoft.com/en-us/dotnet/csharp/misc/cs0177)|RCF0067
 [CS0192](http://docs.microsoft.com/en-us/dotnet/csharp/misc/cs0192)|RCF0027, RCF0070
-[CS0201](http://docs.microsoft.com/en-us/dotnet/csharp/language-reference/compiler-messages/cs0201)|RCF0047, RCF0076
+[CS0201](http://docs.microsoft.com/en-us/dotnet/csharp/language-reference/compiler-messages/cs0201)|RCF0047, RCF0076, RCF0080
 [CS0214](http://docs.microsoft.com/en-us/dotnet/csharp/misc/cs0214)|RCF0050, RCF0051
 [CS0219](http://docs.microsoft.com/en-us/dotnet/csharp/misc/cs0219)|RCF0001
 [CS0221](http://docs.microsoft.com/en-us/dotnet/csharp/misc/cs0221)|RCF0031
 [CS0225](http://docs.microsoft.com/en-us/dotnet/csharp/misc/cs0225)|RCF0030
 [CS0260](http://docs.microsoft.com/en-us/dotnet/csharp/language-reference/compiler-messages/cs0260)|RCF0025
 CS0261|RCF0029
 [CS0262](http://docs.microsoft.com/en-us/dotnet/csharp/misc/cs0262)|RCF0036
-[CS0266](http://docs.microsoft.com/en-us/dotnet/csharp/language-reference/compiler-messages/cs0266)|RCF0003, RCF0023, RCF0073
+[CS0266](http://docs.microsoft.com/en-us/dotnet/csharp/language-reference/compiler-messages/cs0266)|RCF0003, RCF0023, RCF0073, RCF0080
 [CS0267](http://docs.microsoft.com/en-us/dotnet/csharp/misc/cs0267)|RCF0054
 [CS0275](http://docs.microsoft.com/en-us/dotnet/csharp/misc/cs0275)|RCF0055
 [CS0305](http://docs.microsoft.com/en-us/dotnet/csharp/misc/cs0305)|RCF0044
@@ -76,6 +77,7 @@ CS0261|RCF0029
 [CS0693](http://docs.microsoft.com/en-us/dotnet/csharp/misc/cs0693)|RCF0061
 [CS0708](http://docs.microsoft.com/en-us/dotnet/csharp/misc/cs0708)|RCF0033, RCF0062
 [CS0710](http://docs.microsoft.com/en-us/dotnet/csharp/misc/cs0710)|RCF0033, RCF0062
+[CS0713](http://docs.microsoft.com/en-us/dotnet/csharp/misc/cs0713)|RCF0068
 [CS0718](http://docs.microsoft.com/en-us/dotnet/csharp/misc/cs0718)|RCF0068
 [CS0750](http://docs.microsoft.com/en-us/dotnet/csharp/misc/cs0750)|RCF0055
 [CS0751](http://docs.microsoft.com/en-us/dotnet/csharp/misc/cs0751)|RCF0025
@@ -108,8 +110,9 @@ CS0261|RCF0029
 [CS1715](http://docs.microsoft.com/en-us/dotnet/csharp/misc/cs1715)|RCF0019
 [CS1717](http://docs.microsoft.com/en-us/dotnet/csharp/misc/cs1717)|RCF0028
 [CS1722](http://docs.microsoft.com/en-us/dotnet/csharp/misc/cs1722)|RCF0005
+CS1750|RCF0058
 [CS1955](http://docs.microsoft.com/en-us/dotnet/csharp/misc/cs1955)|RCF0037
 CS1988|RCF0027, RCF0070
 CS1994|RCF0055
-CS1997|RCF0052, RCF0057
+CS1997|RCF0052, RCF0057, RCF0080
 CS8070|RCF0002

source/CodeFixes/Properties/AssemblyInfo.cs

@@ -28,6 +28,6 @@
 //
 // You can specify all the values or you can default the Build and Revision Numbers 
 // by using the '*' as shown below:
-[assembly: AssemblyVersion("1.5.11")]
+[assembly: AssemblyVersion("1.5.12")]
 //[assembly: AssemblyFileVersion("1.0.0.0")]
 [assembly: NeutralResourcesLanguage("en-US")]