fix comiplation error for .NET 3.5

Author: neuecc

Assets/Plugins/UniRx/Scripts/Async/Internal/ArrayPool.cs

@@ -1,4 +1,5 @@
-#pragma warning disable CS1591 // Missing XML comment for publicly visible type or member
+#if CSHARP_7_OR_LATER
+#pragma warning disable CS1591 // Missing XML comment for publicly visible type or member
 
 using System;
 using System.Threading;
@@ -147,4 +148,5 @@ static int GetQueueIndex(int size)
             }
         }
     }
-}
+}
+#endif

Assets/Plugins/UniRx/Scripts/Async/Internal/PlayerLoopRunner.cs

@@ -1,4 +1,4 @@
-#if UNITY_2018_1_OR_NEWER
+#if CSHARP_7_OR_LATER
 
 using System;
 using UnityEngine;

Assets/Plugins/UniRx/Scripts/Async/PlayerLoopHelper.cs

@@ -1,4 +1,4 @@
-#if UNITY_2018_1_OR_NEWER
+#if CSHARP_7_OR_LATER
 #pragma warning disable CS1591 // Missing XML comment for publicly visible type or member
 
 using System;

Assets/Plugins/UniRx/Scripts/Async/UnityAsyncExtensions.cs

@@ -1,4 +1,4 @@
-#if (NET_4_6 || NET_STANDARD_2_0) && CSHARP_7_OR_LATER
+#if CSHARP_7_OR_LATER
 #pragma warning disable CS1591 // Missing XML comment for publicly visible type or member
 
 using System;

Assets/Plugins/UniRx/Scripts/UnityEngineBridge/UnityUIComponentExtensions.cs

@@ -104,6 +104,8 @@ public static IObservable<string> OnValueChangedAsObservable(this InputField inp
         }
 #endif
 
+#if UNITY_5_3_OR_NEWER
+
         /// <summary>Observe onValueChanged with current `value` on subscribe.</summary>
         public static IObservable<int> OnValueChangedAsObservable(this Dropdown dropdown)
         {
@@ -113,6 +115,8 @@ public static IObservable<int> OnValueChangedAsObservable(this Dropdown dropdown
                 return d.onValueChanged.AsObservable().Subscribe(observer);
             });
         }
+
+#endif
     }
 }
 

Assets/Scripts/UnityTests/AsyncTestSample.cs

@@ -1,4 +1,5 @@
-#pragma warning disable CS1591
+#if CSHARP_7_OR_LATER
+#pragma warning disable CS1591
 
 using UnityEngine.TestTools;
 using System.Collections;
@@ -11,6 +12,8 @@
 
 public class AsyncTestSample
 {
+#if ENABLE_WWW
+
     [UnityTest]
     public IEnumerator AsyncTest() => UniTask.ToCoroutine(async () =>
     {
@@ -30,4 +33,8 @@ public IEnumerator AsyncTest() => UniTask.ToCoroutine(async () =>
         var req = await UnityWebRequest.Get("http://google.co.jp/").SendWebRequest();
         req.downloadHandler.text.Contains("<title>Google</title>").IsTrue();
     });
+
+#endif
 }
+
+#endif

Assets/Scripts/UnityTests/ChainingAssertion.Unity.cs

@@ -1,9 +1,6 @@
-#pragma warning disable CS1591
-
-using System;
+using System;
 using System.Collections;
 using System.Collections.Generic;
-using System.Diagnostics.Contracts;
 using System.Linq;
 using System.Reflection;
 
@@ -12,7 +9,6 @@
 // no namespace.
 
 [System.Diagnostics.DebuggerStepThroughAttribute]
-[ContractVerification(false)]
 public static partial class ChainingAssertion
 {
     /// <summary>Assert.AreEqual, if T is IEnumerable then CollectionAssert.AreEqual</summary>
@@ -219,7 +215,7 @@ public static void IsStructuralEqual(this object actual, object expected, string
         if (!r.IsEquals)
         {
             var msg = string.Format("is not structural equal, failed at {0}, actual = {1} expected = {2}",
-                string.Join(".", r.Names), r.Left, r.Right);
+                string.Join(".", r.Names.ToArray()), r.Left, r.Right);
             throw new AssertionException(msg, message);
         }
     }
@@ -318,9 +314,9 @@ static EqualInfo StructuralEqual(object left, object right, IEnumerable<string>
         // is object
         var fields = left.GetType().GetFields(BindingFlags.Instance | BindingFlags.Public);
         var properties = left.GetType().GetProperties(BindingFlags.Instance | BindingFlags.Public).Where(x => x.GetGetMethod(false) != null);
-        var members = fields.Cast<MemberInfo>().Concat(properties);
+        var members = fields.Cast<MemberInfo>().Concat(properties.Cast<MemberInfo>());
 
-        foreach (var mi in fields.Cast<MemberInfo>().Concat(properties))
+        foreach (var mi in members)
         {
             var concatNames = names.Concat(new[] { (string)mi.Name });
 
@@ -336,8 +332,8 @@ static EqualInfo StructuralEqual(object left, object right, IEnumerable<string>
             else
             {
                 var i = mi as PropertyInfo;
-                lv = i.GetValue(left);
-                rv = i.GetValue(right);
+                lv = i.GetValue(left, null);
+                rv = i.GetValue(right, null);
             }
 
             var result = StructuralEqual(lv, rv, concatNames);

Assets/Scripts/UnityTests/Rx/ArrayPoolTest.cs

@@ -1,4 +1,5 @@
-using NUnit.Framework;
+#if CSHARP_7_OR_LATER
+using NUnit.Framework;
 using System;
 using UniRx.Async.Internal;
 using System.Collections.Generic;
@@ -33,3 +34,5 @@ public void Rent()
         }
     }
 }
+
+#endif

Assets/Scripts/UnityTests/Rx/MicroCoroutineTest.cs

@@ -25,9 +25,9 @@ public object Current
                 return null;
             }
         }
-        public int OriginalCount => original;
+        public int OriginalCount { get { return original; } }
 
-        public int Count => count;
+        public int Count { get { return count; } }
 
         public bool MoveNext()
         {
@@ -41,7 +41,7 @@ public void Reset()
 
         public override string ToString()
         {
-            return $"{count}/{original}";
+            return count + "/" + "original";
         }
     }
 

Assets/Scripts/UnityTests/Rx/ReactivePropertyTest.cs

@@ -2,7 +2,6 @@
 using System.Collections.Generic;
 using System.Linq;
 using System.Text;
-using System.Threading.Tasks;
 using NUnit.Framework;
 
 namespace UniRx.Tests
@@ -357,7 +356,7 @@ public void FinishedSourceToReadOnlyReactiveProperty()
                 // immediate
                 {
 
-                    var ex = new Exception();
+                    // var ex = new Exception();
                     var source = new Subject<int>();
                     var rxProp = source.ToReadOnlyReactiveProperty();