<html>
<head>
<style><!--
.hmmessage P
{
margin:0px;
padding:0px
}
body.hmmessage
{
font-size: 12pt;
font-family:Calibri
}
--></style></head>
<body class='hmmessage'><div dir='ltr'><font color="#000000" face="Calibri,sans-serif">Here's the diff between my trimmed version and the original: <a href="http://www.mergely.com/9E70e5Ls/">http://www.mergely.com/9E70e5Ls/</a></font><BR><font color="#000000" face="Calibri,sans-serif"></font> <BR><font color="#000000" face="Calibri,sans-serif">Hmm, I actually think that most of the removed methods don't matter (as they aren't called anyway). I just wanted to make the file only contain the code that is necessary.</font><BR><font color="#000000" face="Calibri,sans-serif">The only "real" changes I had to make where accesses to internal fields in NumberFormatInfo, I just replaced them with accesses to the public properties that read them.<br id="FontBreak"></font><BR>Maybe it makes more sense to correct those accesses directly in the referencesource, so we can reference it from there?<BR>As a general guidance, when should we copy something to the Mono tree vs. editing referencesource? And if we copy, should we keep unneeded methods or remove them? <BR> <BR>-- Alex<br> <BR><div><hr id="stopSpelling">From: miguel@xamarin.com<br>Date: Sat, 29 Nov 2014 10:40:10 -0500<br>Subject: Re: [Mono-dev] Porting System.Numerics from referencesource, approaches for copying code?<br>To: alex.koeplinger@outlook.com<br>CC: mono-devel-list@lists.ximian.com<br><br><div dir="ltr">Hey Alex,<div><br></div><div>In general, the approach works.</div><div><br></div><div>I am curious as to what is that you had to remove, and why.   Can you describe the needed changes?   (I saw the diff, but it is hard to tell what happened with it).</div><div><br></div><div>I thought that Numerics was pretty much standalone.</div></div><div class="ecxgmail_extra"><br><div class="ecxgmail_quote">On Sat, Nov 29, 2014 at 10:32 AM, Alexander Köplinger <span dir="ltr"><<a href="mailto:alex.koeplinger@outlook.com" target="_blank">alex.koeplinger@outlook.com</a>></span> wrote:<br><blockquote class="ecxgmail_quote" style="padding-left: 1ex; border-left-color: rgb(204, 204, 204); border-left-width: 1px; border-left-style: solid;">


<div><div dir="ltr"><font color="#000000" face="Calibri,sans-serif">Hey, I've been trying to port System.Numerics (BigInteger, Complex classes) from referencesource to see and feel what the process looks like.</font><br><font color="#000000" face="Calibri,sans-serif">It mostly went fine, but there was one file (number.cs) I couldn't reference directly from the submodule and had to copy to the assembly </font>folder in the Mono tree.<br> <br>I had to remove a bunch of code from that file to trim it down to only what is needed for System.Numerics to compile (all the tests pass now as well).<br>Question: is this a valid approach or is there something better? Note that ifdefing the file wasn't feasible as it originally sits inside corlib and has many other unrelated methods in it.<br> <br>Here's the commit with the WIP: <a href="https://github.com/akoeplinger/mono/commit/d7c461155a975a35f76b79ebcc3fe33407bd5dce" target="_blank">https://github.com/akoeplinger/mono/commit/d7c461155a975a35f76b79ebcc3fe33407bd5dce</a><br> <br>-- Alex<br>                                       </div></div>
<br>_______________________________________________<br>
Mono-devel-list mailing list<br>
<a href="mailto:Mono-devel-list@lists.ximian.com">Mono-devel-list@lists.ximian.com</a><br>
<a href="http://lists.ximian.com/mailman/listinfo/mono-devel-list" target="_blank">http://lists.ximian.com/mailman/listinfo/mono-devel-list</a><br>
<br></blockquote></div><br></div></div>                                       </div></body>
</html>