Skip to content

Commit 758b0df

Browse files
committed
Guarding fixes and don't default NRT to on.
1 parent c199e1f commit 758b0df

File tree

3 files changed

+3
-5
lines changed

3 files changed

+3
-5
lines changed

tools/generator/CodeGeneratorOptions.cs

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -173,8 +173,6 @@ public static CodeGeneratorOptions Parse (string[] args)
173173
return null;
174174
}
175175

176-
opts.SupportNullableReferenceTypes = true;
177-
178176
return opts;
179177
}
180178

tools/generator/Java.Interop.Tools.Generator.ObjectModel/Symbols/ArraySymbol.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ public string GetGenericType (Dictionary<string, string> mappings)
7878

7979
public string FromNative (CodeGenerationOptions opt, string var_name, bool owned)
8080
{
81-
return String.Format ("({0}[]?) JNIEnv.GetArray ({1}, {2}, typeof ({3}))", opt.GetOutputName (ElementType), var_name, owned ? "JniHandleOwnership.TransferLocalRef" : "JniHandleOwnership.DoNotTransfer", opt.GetOutputName (sym.FullName));
81+
return String.Format ("({0}[]{4}) JNIEnv.GetArray ({1}, {2}, typeof ({3}))", opt.GetOutputName (ElementType), var_name, owned ? "JniHandleOwnership.TransferLocalRef" : "JniHandleOwnership.DoNotTransfer", opt.GetOutputName (sym.FullName), opt.NullableOperator);
8282
}
8383

8484
public string ToNative (CodeGenerationOptions opt, string var_name, Dictionary<string, string> mappings = null)
@@ -117,7 +117,7 @@ public string[] PostCall (CodeGenerationOptions opt, string var_name)
117117

118118
public string[] PreCallback (CodeGenerationOptions opt, string var_name, bool owned)
119119
{
120-
return new string[] { String.Format ("var {1} = ({0}[]?) JNIEnv.GetArray ({2}, JniHandleOwnership.DoNotTransfer, typeof ({3}));", opt.GetOutputName (ElementType), opt.GetSafeIdentifier (var_name), opt.GetSafeIdentifier (TypeNameUtilities.GetNativeName (var_name)), opt.GetOutputName (sym.FullName)) };
120+
return new string[] { String.Format ("var {1} = ({0}[]{4}) JNIEnv.GetArray ({2}, JniHandleOwnership.DoNotTransfer, typeof ({3}));", opt.GetOutputName (ElementType), opt.GetSafeIdentifier (var_name), opt.GetSafeIdentifier (TypeNameUtilities.GetNativeName (var_name)), opt.GetOutputName (sym.FullName), opt.NullableOperator) };
121121
}
122122

123123
public string[] PreCall (CodeGenerationOptions opt, string var_name)

tools/generator/Java.Interop.Tools.Generator.ObjectModel/Symbols/GeneratedEnumSymbol.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ public string GetGenericType (Dictionary<string, string> mappings)
2727
public string FromNative (CodeGenerationOptions opt, string var_name, bool owned)
2828
{
2929
if (IsArray)
30-
return String.Format ("({0}[]?) JNIEnv.GetArray ({1}, {2}, typeof ({0}))", opt.GetOutputName (enum_type), var_name, owned ? "JniHandleOwnership.TransferLocalRef" : "JniHandleOwnership.DoNotTransfer");
30+
return String.Format ("({0}[]{3}) JNIEnv.GetArray ({1}, {2}, typeof ({0}))", opt.GetOutputName (enum_type), var_name, owned ? "JniHandleOwnership.TransferLocalRef" : "JniHandleOwnership.DoNotTransfer", opt.NullableOperator);
3131
else
3232
return String.Format ("({0}) {1}", opt.GetOutputName (enum_type), var_name);
3333
}

0 commit comments

Comments
 (0)