Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Modernization stage1 #1117

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
wants to merge 5 commits into from
Closed

Modernization stage1 #1117

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
3b88515
Drop PYTHON3 flag (default now)
filmor Dec 14, 2019
7d6c4ac
Stop using Runtime.IsPython2/3
filmor Dec 15, 2019
14230b0
drop remoting
koubaa Apr 17, 2020
32d4f0a
update changelog
koubaa Apr 17, 2020
5bdb090
fix ucs2
koubaa Apr 17, 2020
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
drop remoting
  • Loading branch information
@koubaa
koubaa committed Apr 17, 2020
commit 14230b0979a71e94afbcc4c24cc9246450c3b8f6
8 changes: 0 additions & 8 deletions src/runtime/converter.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -167,15 +167,7 @@ internal static IntPtr ToPython(object value, Type type)
var pyderived = value as IPythonDerivedType;
if (null != pyderived)
{
#if NETSTANDARD
return ClassDerivedObject.ToPython(pyderived);
#else
// if object is remote don't do this
if (!System.Runtime.Remoting.RemotingServices.IsTransparentProxy(pyderived))
{
return ClassDerivedObject.ToPython(pyderived);
}
#endif
}

// hmm - from Python, we almost never care what the declared
Expand Down