Skip to content

Constructor argument matching supports simple int to (float|double) types #239

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

Closed
sigbjorn wants to merge 16 commits into from
Closed

Constructor argument matching supports simple int to (float|double) types #239

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
2a2c9c7
added intial test that illustrates how int->double conversion works i…
sigbjorn Jun 30, 2016
78dca29
ease windows development, - default settings after checkout reflectin…
sigbjorn Jul 3, 2016
af1b937
added method tests for simple promotions int->float, int->double
sigbjorn Jul 3, 2016
24baadf
extending test to cover negative outcome of bool->double|float
sigbjorn Jul 3, 2016
bde4258
adding default ct to challenge the matching mechanism
sigbjorn Jul 3, 2016
c8c8d8c
merged extra tests
sigbjorn Jul 3, 2016
a65d793
adding test, and print-out warning for the not yet implemented feature
sigbjorn Jul 3, 2016
e698b57
added more test for illustrating super/sub-class and argument passing…
sigbjorn Jul 3, 2016
1fd75b3
fixed py 2.7 syntax for super, plus fixed typo
sigbjorn Jul 4, 2016
fb9b21e
reverted the .sln to unmodified state
sigbjorn Jul 4, 2016
9e1d72b
reverting changes to runtime and clrmodule projects
sigbjorn Jul 4, 2016
0e2cd12
Reverting changes to Console, embed_tests and testing .csproj files
sigbjorn Jul 4, 2016
e58d5b2
Hopefully fixed py 2.6 issues
sigbjorn Jul 4, 2016
8602823
Merge branch 'ct_arg_match_fix' of https://github.com/sigbjorn/python…
sigbjorn Jul 4, 2016
bdbaa62
SimplePromotableType simplified, returns true if src is int-type and …
sigbjorn Jul 23, 2016
478b632
Merge branch 'master' into ct_arg_match_fix
sigbjorn Jul 30, 2016
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
SimplePromotableType simplified, returns true if src is int-type and … 
…destination type is float or double, leaving overflow checking entirely to the conversion routine
  • Loading branch information
@sigbjorn
sigbjorn committed Jul 23, 2016
commit bdbaa626a496354d1eaf2f5f2d6739f4dbe627b9
7 changes: 4 additions & 3 deletions src/runtime/methodbinder.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -469,15 +469,16 @@ internal Binding Bind(IntPtr inst, IntPtr args, IntPtr kw,
}
/// <summary>
/// Returns true if src type (int -type) is promotable to a higher resolution type (float|double)
/// <note>no sanity check for overflow is done, this is done elsewhere when performing
/// the actual conversion.
/// </note>
/// </summary>
/// <param name="src"></param>
/// <param name="dst"></param>
/// <returns></returns>
private bool SimplePromotableType(Type src, Type dst)
{
return (((src == typeof(int) || src == typeof(short)) && (dst == typeof(double) || dst == typeof(float))))
|| ((src == typeof(long)) && (dst == typeof(double)))
|| ((src ==typeof(int) || src == typeof(short)) && (dst == typeof(long)));
return (((src == typeof(int) || src == typeof(short) || src == typeof(long) ) && (dst == typeof(double) || dst == typeof(float))));
}

internal virtual IntPtr Invoke(IntPtr inst, IntPtr args, IntPtr kw)
Expand Down