SWEEP: Fixed various build warnings

Author: NightOwl888

tests/ICU4N.TestFramework/Dev/Test/AbstractTestLog.cs

@@ -10,15 +10,15 @@ namespace ICU4N.Dev.Test
     public abstract class AbstractTestLog : TestLog
     {
         /**
-         * Returns true if ICU_Version < major.minor.
+         * Returns true if ICU_Version &lt; major.minor.
          */
         static public bool IsICUVersionBefore(int major, int minor)
         {
             return IsICUVersionBefore(major, minor, 0);
         }
 
         /**
-         * Returns true if ICU_Version < major.minor.milli.
+         * Returns true if ICU_Version &lt; major.minor.milli.
          */
         static public bool IsICUVersionBefore(int major, int minor, int milli)
         {

tests/ICU4N.TestFramework/Dev/Test/TestBoilerplate.cs

@@ -113,7 +113,7 @@ private void Errln(string title, int i, int j)
         protected abstract bool AddTestObject(IList<T> c);
         /**
          * Override if the tested objects are mutable.
-         * <br>Since Java doesn't tell us, we need a function to tell if so.
+         * <br/>Since Java doesn't tell us, we need a function to tell if so.
          * The default is true, so must be overridden if not.
          */
         protected virtual bool IsMutable(T a)

tests/ICU4N.TestFramework/Dev/Test/TestFmwk.cs

@@ -19,7 +19,7 @@ namespace ICU4N.Dev.Test
     /// TestFmwk is a base class for tests that can be run conveniently from the
     /// command line as well as in Visual Studio.
     /// <para/>
-    /// Sub-classes implement a set of methods named Test <something>. Each of these
+    /// Sub-classes implement a set of methods named Test &lt;something&gt;. Each of these
     /// methods performs some test. Test methods should indicate errors by calling
     /// either err or Errln. This will increment the errorCount field and may
     /// optionally print a message to the log. Debugging information may also be
@@ -30,9 +30,9 @@ public abstract class TestFmwk : AbstractTestLog
     {
         private static readonly object syncLock = new object();
 
-        /**
-     * The default time zone for all of our tests. Used in @Before
-     */
+        /////**
+        //// * The default time zone for all of our tests. Used in @Before
+        //// */
         // ICU4N NOTE: In .NET, there is no way to set the time zone for the entire app, so
         // we need to use another approach than setting it for the test fixture.
 
@@ -143,9 +143,9 @@ protected Random CreateRandom()
 
         /**
          * Integer Random number generator, produces positive int values.
-         * Similar to C++ std::minstd_rand, with the same algorithm & constants.
+         * Similar to C++ std::minstd_rand, with the same algorithm &amp; constants.
          * Provided for compatibility with ICU4C.
-         * Get & set of the seed allows for reproducible monkey tests.
+         * Get &amp; set of the seed allows for reproducible monkey tests.
          */
         public class ICU_Rand // ICU4N: made public because of accessiblity issues
         {
@@ -441,7 +441,7 @@ private class TestParams
             internal int seed;
             internal int loggingLevel;
 
-            internal string policyFileName;
+            //internal string policyFileName;
             //internal SecurityManager testSecurityManager;
             //internal SecurityManager originalSecurityManager;
 

tests/ICU4N.TestFramework/Dev/Test/TestUtil.cs

@@ -106,8 +106,8 @@ public static bool IsUnprintable(int c)
             return !(c >= 0x20 && c <= 0x7E);
         }
         /**
-         * Escape unprintable characters using <backslash>uxxxx notation
-         * for U+0000 to U+FFFF and <backslash>Uxxxxxxxx for U+10000 and
+         * Escape unprintable characters using /uxxxx notation
+         * for U+0000 to U+FFFF and /Uxxxxxxxx for U+10000 and
          * above.  If the character is printable ASCII, then do nothing
          * and return FALSE.  Otherwise, append the escaped notation and
          * return TRUE.

tests/ICU4N.TestFramework/Dev/Test/UTF16Util.cs

@@ -399,7 +399,7 @@ public static int CharAt(char[] source, int start, int limit,
 
 
         /**
-         * Forms a supplementary code point from the argument character<br>
+         * Forms a supplementary code point from the argument character<br/>
          * Note this is for internal use hence no checks for the validity of the
          * surrogate characters are done
          * @param lead lead surrogate character

tests/ICU4N.TestFramework/Dev/Util/CollectionUtilities.cs

@@ -641,7 +641,7 @@
 //            : b == null ? false : a.equals(b);
 //}
 
-///**
+////**
 // * Compare, allowing nulls and putting them first
 // * @param a
 // * @param b
@@ -654,7 +654,7 @@
 //                    : b == null ? 1 : a.compareTo(b);
 //}
 
-///**
+////**
 // * Compare iterators
 // * @param iterator1
 // * @param iterator2
@@ -681,7 +681,7 @@
 //    }
 //}
 
-///**
+////**
 // * Compare, with shortest first, and otherwise lexicographically
 // * @param a
 // * @param b
@@ -699,7 +699,7 @@
 //    return compare(iterator1, iterator2);
 //}
 
-///**
+////**
 // * Compare, with shortest first, and otherwise lexicographically
 // * @param a
 // * @param b
@@ -734,7 +734,7 @@
 //    }
 //};
 
-///**
+////**
 // * Compare, allowing nulls and putting them first
 // * @param a
 // * @param b

tests/ICU4N.TestFramework/Dev/Util/UnicodeMap.cs

@@ -1228,15 +1228,15 @@ public void Reset()
                 iterator.Reset();
             }
 
-            ///* (non-Javadoc)
+            ////* (non-Javadoc)
             // * @see java.util.Iterator#hasNext()
             // */
             //public bool HasNext()
             //{
             //    return iterator.hasNext();
             //}
 
-            ///* (non-Javadoc)
+            ////* (non-Javadoc)
             // * @see java.util.Iterator#next()
             // */
             //public Entry<String, T> next()
@@ -1245,7 +1245,7 @@ public void Reset()
             //    return new ImmutableEntry(key, get(key));
             //}
 
-            ///* (non-Javadoc)
+            ////* (non-Javadoc)
             // * @see java.util.Iterator#remove()
             // */
             //public void remove()

tests/ICU4N.TestFramework/Dev/Util/UnicodeMapIterator.cs

@@ -63,16 +63,16 @@ public UnicodeMapIterator()
         /**
          * Returns the next element in the set, either a single code point
          * or a string.  If there are no more elements in the set, return
-         * false.  If <tt>codepoint == IS_STRING</tt>, the value is a
-         * string in the <tt>string</tt> field.  Otherwise the value is a
-         * single code point in the <tt>codepoint</tt> field.
+         * false.  If <c>codepoint == IS_STRING</c>, the value is a
+         * string in the <c>string</c> field.  Otherwise the value is a
+         * single code point in the <c>codepoint</c> field.
          * 
-         * <p>The order of iteration is all code points in sorted order,
-         * followed by all strings sorted order.  <tt>codepointEnd</tt> is
-         * undefined after calling this method.  <tt>string</tt> is
-         * undefined unless <tt>codepoint == IS_STRING</tt>.  Do not mix
-         * calls to <tt>next()</tt> and <tt>nextRange()</tt> without
-         * calling <tt>reset()</tt> between them.  The results of doing so
+         * <para/>The order of iteration is all code points in sorted order,
+         * followed by all strings sorted order.  <c>codepointEnd</c> is
+         * undefined after calling this method.  <c>string</c> is
+         * undefined unless <c>codepoint == IS_STRING</c>.  Do not mix
+         * calls to <c>next()</c> and <c>nextRange()</c> without
+         * calling <c>reset()</c> between them.  The results of doing so
          * are undefined.
          *
          * @return true if there was another element in the set and this
@@ -114,17 +114,17 @@ public bool Next()
         /**
          * Returns the next element in the set, either a code point range
          * or a string.  If there are no more elements in the set, return
-         * false.  If <tt>codepoint == IS_STRING</tt>, the value is a
+         * false.  If <c>codepoint == IS_STRING</c>, the value is a
          * string in the <tt>string</tt> field.  Otherwise the value is a
-         * range of one or more code points from <tt>codepoint</tt> to
-         * <tt>codepointeEnd</tt> inclusive.
+         * range of one or more code points from <c>codepoint</c> to
+         * <c>codepointeEnd</c> inclusive.
          * 
-         * <p>The order of iteration is all code points ranges in sorted
+         * <para/>The order of iteration is all code points ranges in sorted
          * order, followed by all strings sorted order.  Ranges are
-         * disjoint and non-contiguous.  <tt>string</tt> is undefined
-         * unless <tt>codepoint == IS_STRING</tt>.  Do not mix calls to
-         * <tt>next()</tt> and <tt>nextRange()</tt> without calling
-         * <tt>reset()</tt> between them.  The results of doing so are
+         * disjoint and non-contiguous.  <c>string</c> is undefined
+         * unless <c>codepoint == IS_STRING</c>.  Do not mix calls to
+         * <c>next()</c> and <c>nextRange()</c> without calling
+         * <c>reset()</c> between them.  The results of doing so are
          * undefined.
          *
          * @return true if there was another element in the set and this

tests/ICU4N.Tests.Collation/Dev/Test/Collate/CollationAPITest.cs

@@ -12,10 +12,10 @@
 using System.Text;
 using StringBuffer = System.Text.StringBuilder;
 
-/// <summary>
-/// Port From:   ICU4C v2.1 : collate/CollationAPITest
-/// Source File: $ICU4CRoot/source/test/intltest/apicoll.cpp
-/// </summary>
+//
+// Port From:   ICU4C v2.1 : collate/CollationAPITest
+// Source File: $ICU4CRoot/source/test/intltest/apicoll.cpp
+//
 namespace ICU4N.Dev.Test.Collate
 {
     public class CollationAPITest : TestFmwk

tests/ICU4N.Tests.Collation/Dev/Test/Collate/CollationChineseTest.cs

@@ -3,10 +3,10 @@
 using System;
 using System.Globalization;
 
-/// <summary>
-/// Port From:   ICU4C v2.1 : Collate/CollationTurkishTest
-/// Source File: $ICU4CRoot/source/test/intltest/trcoll.cpp
-/// </summary>
+//
+// Port From:   ICU4C v2.1 : Collate/CollationTurkishTest
+// Source File: $ICU4CRoot/source/test/intltest/trcoll.cpp
+//
 namespace ICU4N.Dev.Test.Collate
 {
     public class CollationChineseTest : TestFmwk