Merge pull request #383 from Si1ver/better-xml-docs

Improved inline xml documentation

Author: chkuang-g

source/AndroidResolver/src/AndroidAbis.cs

@@ -143,7 +143,7 @@ public AndroidAbis(IEnumerable<string> abisSet) {
     /// <summary>
     /// Create a set of ABIs from a comma separated set of ABI strings.
     /// </summary>
-    /// <param name="abisSet">Set of ABI strings.</param>
+    /// <param name="abiString">Comma separated set of ABI strings.</param>
     public AndroidAbis(string abiString) {
         if (String.IsNullOrEmpty(abiString)) {
             abis = new HashSet<string>(Supported);
@@ -190,7 +190,7 @@ public override bool Equals(System.Object obj) {
     /// Get the supported set of Android ABIs for the current Unity version.
     /// The dictionary maps the official Android ABI name (i.e the directory name looked up by the
     /// operating system) to the UnityEditor.AndroidTargetDevice (Unity 5.x & 2017.x) or
-    // UnityEditor.AndroidArchitecture enumeration value name.
+    /// UnityEditor.AndroidArchitecture enumeration value name.
     /// </summary>
     private static Dictionary<string, string> SupportedAbiToAbiEnumValue {
         get { return PropertyConfiguration.Instance.SupportedAbiToAbiEnumValue; }
@@ -221,10 +221,10 @@ public static IEnumerable<string> AllSupported {
     /// </summary>
     /// <param name="enumValueObject">Enum object to convert.</param>
     private static ulong EnumValueObjectToULong(object enumValueObject) {
-        /// Flags enum values can't be cast directly to an integral type, however it is possible to
-        /// print the value as an integer string so convert to a string and then parse as an int.
-        /// Enums are considered unsigned by the formatter, so if an enum is defined as -1 it will
-        /// be formatted as UInt32.MaxValue, i.e. 4294967295.
+        // Flags enum values can't be cast directly to an integral type, however it is possible to
+        // print the value as an integer string so convert to a string and then parse as an int.
+        // Enums are considered unsigned by the formatter, so if an enum is defined as -1 it will
+        // be formatted as UInt32.MaxValue, i.e. 4294967295.
         return UInt64.Parse(String.Format("{0:D}", enumValueObject));
     }
 
@@ -240,7 +240,7 @@ private static ulong EnumValueStringToULong(Type enumType, string enumValueStrin
     /// <summary>
     /// Get / set the target device ABI (Unity >= 5.0.x)
     /// Unity >= 2017.4 supports armeabi-v7a, arm64-v8a, x86 & fat (i.e armeabi-v7a, arm64, x86)
-    /// Unity >= 5.0.x & <= 2017.3 only support armeabi-v7a, x86 & fat (i.e armeabi-v7a & x86)
+    /// Unity >= 5.0.x & &lt;= 2017.3 only support armeabi-v7a, x86 & fat (i.e armeabi-v7a & x86)
     /// </summary>
     public static AndroidAbis Current {
         set {

source/AndroidResolver/src/AndroidSdkManager.cs

@@ -975,7 +975,7 @@ internal class AndroidSdkManager {
         /// </summary>
         /// <param name="toolName">Name of the tool to search for.</param>
         /// <param name="sdkPath">SDK path to search for the tool.  If this is null or empty, the
-        // system path is searched instead.</param>
+        /// system path is searched instead.</param>
         /// <returns>String with the path to the tool if found, null otherwise.</returns>
         private static string FindAndroidSdkTool(string toolName, string sdkPath = null) {
             if (String.IsNullOrEmpty(sdkPath)) {

source/AndroidResolver/src/CommandLine.cs

@@ -375,7 +375,7 @@ public LineReader(IOHandler handler = null)
             /// a newline isn't present.
             /// </summary>
             /// <param name="handle">Handle of the stream to query.</param>
-            /// <returns>List of data for the requested stream.</return>
+            /// <returns>List of data for the requested stream.</returns>
             public List<StreamData> GetBufferedData(int handle)
             {
                 List<StreamData> handleData;
@@ -397,7 +397,6 @@ public void Flush()
             /// <summary>
             /// Aggregate the specified list of StringBytes into a single structure.
             /// </summary>
-            /// <param name="handle">Stream handle.</param>
             /// <param name="dataStream">Data to aggregate.</param>
             public static StreamData Aggregate(List<StreamData> dataStream)
             {

source/AndroidResolver/src/EmbeddedResource.cs

@@ -132,7 +132,7 @@ public bool Extract(Logger logger) {
         /// directories if they're required.
         /// </summary>
         /// <param name="assembly">Assembly to extract resources from.</param>
-        /// <param name="resourceNameToTargetPath">Each Key is the resource to extract and each
+        /// <param name="resourceNameToTargetPaths">Each Key is the resource to extract and each
         /// Value is the path to extract to. If the resource name (Key) is null or empty, this
         /// method will attempt to extract a resource matching the filename component of the path.
         /// </param>

source/AndroidResolver/src/GradleResolver.cs

@@ -139,7 +139,7 @@ internal static Dictionary<string, string> DependenciesToPackageSpecs(
         /// * Path relative to project folder, ex."Assets/Firebase/m2repository"
         /// </summary>
         /// <param name="repoPath">Repo path to convert.</param>
-        /// <param name="sourceLocation>XML or source file this path is referenced from. If this is
+        /// <param name="sourceLocation">XML or source file this path is referenced from. If this is
         /// null the calling method's source location is used when logging the source of this
         /// repo declaration.</param>
         /// <returns>URI to the repo.</returns>
@@ -1025,16 +1025,20 @@ private string DetermineExplodedAarPath(string aarPath) {
         /// <summary>
         /// Processes the aars.
         /// </summary>
-        /// <remarks>Each aar copied is inspected and determined if it should be
+        /// <remarks>
+        /// <para>
+        /// Each aar copied is inspected and determined if it should be
         /// exploded into a directory or not. Unneeded exploded directories are
         /// removed.
+        /// </para>
         /// <para>
         /// Exploding is needed if the version of Unity is old, or if the artifact
         /// has been explicitly flagged for exploding.  This allows the subsequent
         /// processing of variables in the AndroidManifest.xml file which is not
         /// supported by the current versions of the manifest merging process that
         /// Unity uses.
         /// </para>
+        /// </remarks>
         /// <param name="dir">The directory to process.</param>
         /// <param name="updatedFiles">Set of files that were recently updated and should be
         /// processed.</param>

source/AndroidResolver/src/GradleWrapper.cs

@@ -133,7 +133,7 @@ public bool Extract(Logger logger) {
         /// <param name="buildScript">Path to the Gradle build script to use.</param>
         /// <param name="projectProperties">Project properties to use when running the script.
         /// </param>
-        /// <param name="arguments>Other arguments to pass to Gradle.</param>
+        /// <param name="arguments">Other arguments to pass to Gradle.</param>
         /// <param name="logger">Logger to report errors to.</param>
         /// <param name="executeCommand">Closure which takes the tool path to execute
         /// (gradle wrapper) and a string of arguments returning true if successful,

source/AndroidResolver/src/JavaUtilities.cs

@@ -132,7 +132,7 @@ private static string JavaHomeBinaryPath(string javaTool) {
         /// <summary>
         /// Find a Java tool.
         /// </summary>
-        /// <param name="toolName">Name of the tool to search for.</param>
+        /// <param name="javaTool">Name of the tool to search for.</param>
         /// <returns>Path to the tool if it's found, throws a ToolNotFoundException
         /// otherwise.</returns>
         private static string FindJavaTool(string javaTool)

source/AndroidResolver/src/LocalMavenRepository.cs

@@ -82,7 +82,7 @@ public static List<string> FindAarsInLocalRepos(ICollection<Dependency> dependen
         /// <summary>
         /// Get a path without a filename extension.
         /// </summary>
-        /// <param name="filename">Path to a file.</param>
+        /// <param name="path">Path to a file.</param>
         /// <returns>Path (including directory) without a filename extension.</returns>
         internal static string PathWithoutExtension(string path) {
             return Path.Combine(Path.GetDirectoryName(path),

source/AndroidResolver/src/PlayServicesResolver.cs

@@ -261,7 +261,7 @@ private static string SetToString(HashSet<string> setToConvert) {
             /// <summary>
             /// Convert a dictionary to a sorted comma separated string.
             /// </summary>
-            /// <returns>Comma separated string in the form key=value.<returns>
+            /// <returns>Comma separated string in the form key=value.</returns>
             private static string DictionaryToString(Dictionary<string, string> dict) {
                 var components = new List<string>();
                 foreach (var key in SortSet(dict.Keys)) {
@@ -319,7 +319,7 @@ internal class PropertyPoller<T> {
             /// <param name="delayTimeInSeconds">Time to wait before signalling that the value
             /// has changed.</param>
             /// <param name="checkIntervalInSeconds">Time to check the value of the property for
-            /// changes.<param>
+            /// changes.</param>
             public PropertyPoller(string propertyName,
                                   int delayTimeInSeconds = 3,
                                   int checkIntervalInSeconds = 1) {
@@ -628,7 +628,7 @@ public static bool GradleProjectExportEnabled {
         /// </summary>
         private struct AndroidBuildSystemSettings {
             /// <summary>
-            // Whether the Gradle build is enabled.
+            /// Whether the Gradle build is enabled.
             /// </summary>
             public bool GradleBuildEnabled { get; private set; }
 
@@ -638,7 +638,7 @@ private struct AndroidBuildSystemSettings {
             public bool GradleTemplateEnabled { get; private set; }
 
             /// <summary>
-            // Whether project export is enabled.
+            /// Whether project export is enabled.
             /// </summary>
             public bool ProjectExportEnabled { get; private set; }
 
@@ -656,7 +656,7 @@ public static AndroidBuildSystemSettings Current {
             }
 
             /// <summary>
-            // Compare with another AndroidBuildSystemSettings.
+            /// Compare with another AndroidBuildSystemSettings.
             /// </summary>
             /// <param name="obj">Object to compare with.</param>
             /// <returns>true if the object is the same as this, false otherwise.</returns>
@@ -1217,7 +1217,7 @@ private static void Reresolve() {
         /// * foo.com.my.app.service --> foo.com.my.app.service (unchanged)
         /// </param>
         /// <param name="path">Path of this node in the hierarchy of nodes. For example:
-        /// given node is "<c>" in "<a><b><c>" this should be "a/b/c".  If this
+        /// given node is "&lt;c>" in "&lt;a>&lt;b>&lt;c>" this should be "a/b/c".  If this
         /// value is null the name of the current node is used.</param>
         /// <returns>true if any replacements are applied, false otherwise.</returns>
         private static bool ReplaceVariablesInXmlElementTree(

source/AndroidResolver/src/UnityCompat.cs

@@ -399,7 +399,7 @@ public static int FindNewestInstalledAndroidSDKVersion() {
     /// </summary>
     /// Unfortunately the Unity API does not provide a way to get the current BuildTargetGroup from
     /// the currently active BuildTarget.
-    /// <param name="target">BuildTarget to convert.</param>
+    /// <param name="buildTarget">BuildTarget to convert.</param>
     /// <returns>BuildTargetGroup enum value.</returns>
     private static BuildTargetGroup ConvertBuildTargetToBuildTargetGroup(BuildTarget buildTarget) {
         var buildTargetToGroup = new Dictionary<string, string>() {