Skip to content

[release/7.0] [hot_reload] implement param reflection #77734

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
carlossanlop merged 15 commits into from
Nov 3, 2022
Merged

[release/7.0] [hot_reload] implement param reflection #77734

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
9e3c042
Add new test ReflectionAddNewMethod
lambdageek Oct 27, 2022
a50e6a5
FIXME: get_param_names, get_marshal_info and custom_attrs need work
lambdageek Oct 27, 2022
130b191
WIP - add a method param reverse lookup
lambdageek Oct 27, 2022
ddb5e1f
look up params from added methods
lambdageek Oct 27, 2022
1e94e62
Remove FIXMEs and unused field
lambdageek Oct 27, 2022
01a653a
remove writelines from test
lambdageek Oct 27, 2022
c935dfa
fix test on coreclr
lambdageek Oct 27, 2022
f4a58e1
why does coreclr have 2 attributes here??
lambdageek Oct 28, 2022
50aa8f9
There should be 2 attributes on the 4th param
lambdageek Oct 28, 2022
b3f903f
one more place that looks at params
lambdageek Oct 29, 2022
8b3c7e9
A couple more places where we look at the Params table
lambdageek Oct 29, 2022
81d4e6d
Check default values on params on added method
lambdageek Oct 29, 2022
9fbb7bd
fix lookup if table is empty
lambdageek Oct 29, 2022
2d191c2
add a gratuitious typeof assert
lambdageek Oct 29, 2022
a29fdb2
Add a single mono_metadata_get_method_params function
lambdageek Oct 31, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Check default values on params on added method
  • Loading branch information
@lambdageek
lambdageek authored and github-actions committed Nov 1, 2022
commit 81d4e6d7c0d77b345cdc432e41b8cce91273dc6e
6 changes: 5 additions & 1 deletion src/libraries/System.Runtime.Loader/tests/ApplyUpdateTest.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -702,7 +702,11 @@ public static void TestReflectionAddNewMethod()
Assert.True(foundCallerMemberName);
Assert.True(foundOptional);

// TODO: check the default values of the last two params
Assert.True(parms[3].HasDefaultValue);
Assert.True(parms[4].HasDefaultValue);

Assert.Null(parms[3].DefaultValue);
Assert.Equal(string.Empty, parms[4].DefaultValue);
});
}
}
Expand Down