[generator] Always use XAPeerMembers for XAJavaInterop1 (#457)

Context: #341

When generating the `_members` field when using
`generator --codegen-target=XAJavaInterop1`, in one instance we emit:

	static JniPeerMembers _members = new XAPeerMembers ("android/text/foo", typeof (ISpannableInvoker));

while in the 3 other instances we generate:

	static JniPeerMembers _members = new JniPeerMembers ("android/text/foo", typeof (ISpannableInvoker));

Refactor all the instances of writing this line to a single method
which correctly uses `XAPeerMembers` when needed.

Author: jpobst

tools/generator/Java.Interop.Tools.Generator.CodeGeneration/JavaInteropCodeGenerator.cs

@@ -25,14 +25,12 @@ static string GetInvokeType (string type)
 
 		internal override string GetAllInterfaceImplements () => "IJavaObject, IJavaPeerable";
 
+		protected virtual string GetPeerMembersType () => "JniPeerMembers";
+
 		internal override void WriteClassHandle (ClassGen type, string indent, bool requireNew)
 		{
-			writer.WriteLine ("{0}\tinternal    {1}     static  readonly    JniPeerMembers  _members    = new {2} (\"{3}\", typeof ({4}));",
-					indent,
-					requireNew ? "new" : "   ",
-					GetPeerMembersType (),
-					type.RawJniName,
-					type.Name);
+			WritePeerMembers (indent + '\t', true, requireNew, type.RawJniName, type.Name);
+
 			writer.WriteLine ("{0}\tinternal static {1}IntPtr class_ref {{", indent, requireNew ? "new " : string.Empty);
 			writer.WriteLine ("{0}\t\tget {{", indent);
 			writer.WriteLine ("{0}\t\t\treturn _members.JniPeerType.PeerReference.Handle;", indent);
@@ -55,22 +53,15 @@ internal override void WriteClassHandle (ClassGen type, string indent, bool requ
 			}
 		}
 
-		protected virtual string GetPeerMembersType ()
-		{
-			return "JniPeerMembers";
-		}
-
 		internal override void WriteClassHandle (InterfaceGen type, string indent, string declaringType)
 		{
-			writer.WriteLine ("{0}new static JniPeerMembers _members = new JniPeerMembers (\"{1}\", typeof ({2}));",indent, type.RawJniName, declaringType);
+			WritePeerMembers (indent, false, true, type.RawJniName, declaringType);
 		}
 
 		internal override void WriteClassInvokerHandle (ClassGen type, string indent, string declaringType)
 		{
-			writer.WriteLine ("{0}internal    new     static  readonly    JniPeerMembers  _members    = new JniPeerMembers (\"{1}\", typeof ({2}));",
-					indent,
-					type.RawJniName,
-					declaringType);
+			WritePeerMembers (indent, true, true, type.RawJniName, declaringType);
+
 			writer.WriteLine ();
 			writer.WriteLine ("{0}public override global::Java.Interop.JniPeerMembers JniPeerMembers {{", indent);
 			writer.WriteLine ("{0}\tget {{ return _members; }}", indent);
@@ -84,10 +75,8 @@ internal override void WriteClassInvokerHandle (ClassGen type, string indent, st
 
 		internal override void WriteInterfaceInvokerHandle (InterfaceGen type, string indent, string declaringType)
 		{
-			writer.WriteLine ("{0}internal    new     static  readonly    JniPeerMembers  _members    = new JniPeerMembers (\"{1}\", typeof ({2}));",
-					indent,
-					type.RawJniName,
-					declaringType);
+			WritePeerMembers (indent, true, true, type.RawJniName, declaringType);
+
 			writer.WriteLine ();
 			writer.WriteLine ("{0}static IntPtr java_class_ref {{", indent);
 			writer.WriteLine ("{0}\tget {{ return _members.JniPeerType.PeerReference.Handle; }}", indent);
@@ -272,6 +261,14 @@ internal override void WriteFieldSetBody (Field field, string indent, GenBase ty
 			}
 			writer.WriteLine ("{0}}}", indent);
 		}
+
+		void WritePeerMembers (string indent, bool isInternal, bool isNew, string rawJniType, string declaringType)
+		{
+			var signature = $"{(isInternal ? "internal " : "")}static {(isNew ? "new " : "")}readonly JniPeerMembers _members = ";
+			var type = $"new {GetPeerMembersType ()} (\"{rawJniType}\", typeof ({declaringType}));";
+
+			writer.WriteLine ($"{indent}{signature}{type}");
+		}
 	}
 }
 

tools/generator/Java.Interop.Tools.Generator.CodeGeneration/XAJavaInteropCodeGenerator.cs

@@ -9,10 +9,7 @@ public XAJavaInteropCodeGenerator (TextWriter writer, CodeGenerationOptions opti
 		{
 		}
 
-		protected override string GetPeerMembersType ()
-		{
-			return "XAPeerMembers";
-		}
+		protected override string GetPeerMembersType () => "XAPeerMembers";
 	}
 }
 

tools/generator/Tests-Core/expected.ji/Android.Text.ISpannable.cs

@@ -14,7 +14,7 @@ public partial interface ISpannable : Android.Text.ISpanned {
 	[global::Android.Runtime.Register ("android/text/Spannable", DoNotGenerateAcw=true)]
 	internal partial class ISpannableInvoker : global::Java.Lang.Object, ISpannable {
 
-		internal    new     static  readonly    JniPeerMembers  _members    = new JniPeerMembers ("android/text/Spannable", typeof (ISpannableInvoker));
+		internal static new readonly JniPeerMembers _members = new JniPeerMembers ("android/text/Spannable", typeof (ISpannableInvoker));
 
 		static IntPtr java_class_ref {
 			get { return _members.JniPeerType.PeerReference.Handle; }

tools/generator/Tests-Core/expected.ji/Android.Text.ISpanned.cs

@@ -19,7 +19,7 @@ public partial interface ISpanned : IJavaObject, IJavaPeerable {
 	[global::Android.Runtime.Register ("android/text/Spanned", DoNotGenerateAcw=true)]
 	internal partial class ISpannedInvoker : global::Java.Lang.Object, ISpanned {
 
-		internal    new     static  readonly    JniPeerMembers  _members    = new JniPeerMembers ("android/text/Spanned", typeof (ISpannedInvoker));
+		internal static new readonly JniPeerMembers _members = new JniPeerMembers ("android/text/Spanned", typeof (ISpannedInvoker));
 
 		static IntPtr java_class_ref {
 			get { return _members.JniPeerType.PeerReference.Handle; }

tools/generator/Tests-Core/expected.ji/Android.Text.SpannableString.cs

@@ -9,7 +9,7 @@ namespace Android.Text {
 	[global::Android.Runtime.Register ("android/text/SpannableString", DoNotGenerateAcw=true)]
 	public partial class SpannableString : Android.Text.SpannableStringInternal, Android.Text.ISpannable {
 
-		internal    new     static  readonly    JniPeerMembers  _members    = new JniPeerMembers ("android/text/SpannableString", typeof (SpannableString));
+		internal static new readonly JniPeerMembers _members = new JniPeerMembers ("android/text/SpannableString", typeof (SpannableString));
 		internal static new IntPtr class_ref {
 			get {
 				return _members.JniPeerType.PeerReference.Handle;

tools/generator/Tests-Core/expected.ji/Android.Text.SpannableStringInternal.cs

@@ -9,7 +9,7 @@ namespace Android.Text {
 	[global::Android.Runtime.Register ("android/text/SpannableStringInternal", DoNotGenerateAcw=true)]
 	public abstract partial class SpannableStringInternal : Java.Lang.Object {
 
-		internal    new     static  readonly    JniPeerMembers  _members    = new JniPeerMembers ("android/text/SpannableStringInternal", typeof (SpannableStringInternal));
+		internal static new readonly JniPeerMembers _members = new JniPeerMembers ("android/text/SpannableStringInternal", typeof (SpannableStringInternal));
 		internal static new IntPtr class_ref {
 			get {
 				return _members.JniPeerType.PeerReference.Handle;
@@ -70,7 +70,7 @@ internal partial class SpannableStringInternalInvoker : SpannableStringInternal
 
 		public SpannableStringInternalInvoker (IntPtr handle, JniHandleOwnership transfer) : base (handle, transfer) {}
 
-		internal    new     static  readonly    JniPeerMembers  _members    = new JniPeerMembers ("android/text/SpannableStringInternal", typeof (SpannableStringInternalInvoker));
+		internal static new readonly JniPeerMembers _members = new JniPeerMembers ("android/text/SpannableStringInternal", typeof (SpannableStringInternalInvoker));
 
 		public override global::Java.Interop.JniPeerMembers JniPeerMembers {
 			get { return _members; }

tools/generator/Tests-Core/expected.ji/Android.Views.View.cs

@@ -22,7 +22,7 @@ public partial interface IOnClickListener : IJavaObject, IJavaPeerable {
 		[global::Android.Runtime.Register ("android/view/View$OnClickListener", DoNotGenerateAcw=true)]
 		internal partial class IOnClickListenerInvoker : global::Java.Lang.Object, IOnClickListener {
 
-			internal    new     static  readonly    JniPeerMembers  _members    = new JniPeerMembers ("android/view/View$OnClickListener", typeof (IOnClickListenerInvoker));
+			internal static new readonly JniPeerMembers _members = new JniPeerMembers ("android/view/View$OnClickListener", typeof (IOnClickListenerInvoker));
 
 			static IntPtr java_class_ref {
 				get { return _members.JniPeerType.PeerReference.Handle; }
@@ -128,7 +128,7 @@ internal static bool __IsEmpty (IOnClickListenerImplementor value)
 		}
 
 
-		internal    new     static  readonly    JniPeerMembers  _members    = new JniPeerMembers ("android/view/View", typeof (View));
+		internal static new readonly JniPeerMembers _members = new JniPeerMembers ("android/view/View", typeof (View));
 		internal static new IntPtr class_ref {
 			get {
 				return _members.JniPeerType.PeerReference.Handle;

tools/generator/Tests-Core/expected.ji/Java.Lang.Object.cs

@@ -9,7 +9,7 @@ namespace Java.Lang {
 	[global::Android.Runtime.Register ("java/lang/Object", DoNotGenerateAcw=true)]
 	public partial class Object  {
 
-		internal            static  readonly    JniPeerMembers  _members    = new JniPeerMembers ("java/lang/Object", typeof (Object));
+		internal static readonly JniPeerMembers _members = new JniPeerMembers ("java/lang/Object", typeof (Object));
 		internal static IntPtr class_ref {
 			get {
 				return _members.JniPeerType.PeerReference.Handle;

tools/generator/Tests/Integration-Tests/CSharpKeywords.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using NUnit.Framework;
 
 namespace generatortests

tools/generator/Tests/Unit-Tests/CodeGeneratorExpectedResults/JavaInterop1/WriteClass.txt

@@ -2,7 +2,7 @@
 [global::Android.Runtime.Register ("java/code/MyClass", DoNotGenerateAcw=true)]
 public partial class MyClass  {
 
-	internal    new     static  readonly    JniPeerMembers  _members    = new JniPeerMembers ("java/code/MyClass", typeof (MyClass));
+	internal static new readonly JniPeerMembers _members = new JniPeerMembers ("java/code/MyClass", typeof (MyClass));
 	internal static new IntPtr class_ref {
 		get {
 			return _members.JniPeerType.PeerReference.Handle;