PATCH: remove process.mixin - nodejs

The old Google Groups will be going away soon, but your browser is incompatible with the new version.
Message from discussion PATCH: remove process.mixin
Received-SPF: pass (google.com: domain of marak.squi...@gmail.com designates 74.125.82.41 as permitted sender) client-ip=74.125.82.41;
DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=gamma;
        h=mime-version:in-reply-to:references:date:message-id:subject:from:to
         :content-type;
        b=F2P47X1nZbF9gBlC1f5dtLHVE+HVuaF+Tappo7E7rdvhNDeSIs4lRSBRkkyVKK33Jc
         YQdJPlJUtikj62xBcQhf29BpKKnuOsovSw1QWnazmqsXRqWX1Py/7PFHvq5nJJYHuJuZ
         ALfL3iO4UuiogTPodDq5c/kz/ZFl61ldlMayQ=
MIME-Version: 1.0
In-Reply-To: <21ee31951003090907o645bb538i97acb4cbd7fc4...@mail.gmail.com>
References: <aaedcef51003081516wd5a2ccdn54e4c12ee4365...@mail.gmail.com>
	 <21ee31951003090907o645bb538i97acb4cbd7fc4...@mail.gmail.com>
Date: Tue, 9 Mar 2010 12:16:15 -0500
Message-ID: <96fa0bf91003090916s66894282mc57ded43205ea...@mail.gmail.com>
Subject: Re: [nodejs] PATCH: remove process.mixin
From: Marak Squires <marak.squi...@gmail.com>
To: nodejs@googlegroups.com
Content-Type: multipart/alternative; boundary=0016e6d7e33acfac7f0481615642

--0016e6d7e33acfac7f0481615642
Content-Type: text/plain; charset=ISO-8859-1

Errrr, fuck.

*would be a great thing to have in a lib/object.js module

*Yeah can we get started on this yesterday? I was using a lot of
process.mixin() earlier on, then it broke a bunch, then kinda worked, then
broke a bunch again. I understand the test coverage for deep-copy was never
quite there.

Are there any current modules out there that can do stuff like this?



On Tue, Mar 9, 2010 at 12:07 PM, Ryan Dahl <coldredle...@gmail.com> wrote:

> On Mon, Mar 8, 2010 at 3:16 PM, Isaac Schlueter <i...@izs.me> wrote:
> > As discussed in IRC this morning, process.mixin has a few major issues.
> >
> > Deep object copying is something that belongs in userland, not in
> > src/node.js.  These patches eliminate it.
> >
> > (In my opinion, thorough robust deep-copy-and-merge functionality
> > would be a great thing to have in a lib/object.js module or something,
> > but it has nothing to do with the *process* and thus doesn't belong in
> > node.js or on the process object.)
> >
> > See attachments, or
> http://github.com/isaacs/node/commits/mixin-deprecation
>
>
> Thanks Issac, I took patches 1-5. I think it ought to be moved to sys
> for now. (Which is where I'm putting all the utility functions.)
>
> --
> You received this message because you are subscribed to the Google Groups
> "nodejs" group.
> To post to this group, send email to nodejs@googlegroups.com.
> To unsubscribe from this group, send email to
> nodejs+unsubscribe@googlegroups.com<nodejs%2Bunsubscribe@googlegroups.com>
> .
> For more options, visit this group at
> http://groups.google.com/group/nodejs?hl=en.
>
>

--0016e6d7e33acfac7f0481615642
Content-Type: text/html; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable

Errrr, fuck. <br><br><i>would be a great thing to have in a lib/object.js m=
odule<br><br></i>Yeah can we get started on this yesterday? I was using a l=
ot of process.mixin() earlier on, then it broke a bunch, then kinda worked,=
 then broke a bunch again. I understand the test coverage for deep-copy was=
 never quite there. <br>
<br>Are there any current modules out there that can do stuff like this? <b=
r><br><br><br><div class=3D"gmail_quote">On Tue, Mar 9, 2010 at 12:07 PM, R=
yan Dahl <span dir=3D"ltr">&lt;<a href=3D"mailto:coldredle...@gmail.com">co=
ldredle...@gmail.com</a>&gt;</span> wrote:<br>
<blockquote class=3D"gmail_quote" style=3D"margin: 0pt 0pt 0pt 0.8ex; borde=
r-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div class=3D"im"=
>On Mon, Mar 8, 2010 at 3:16 PM, Isaac Schlueter &lt;<a href=3D"mailto:i@iz=
s.me">i...@izs.me</a>&gt; wrote:<br>

&gt; As discussed in IRC this morning, process.mixin has a few major issues=
.<br>
&gt;<br>
&gt; Deep object copying is something that belongs in userland, not in<br>
&gt; src/node.js. =A0These patches eliminate it.<br>
&gt;<br>
&gt; (In my opinion, thorough robust deep-copy-and-merge functionality<br>
&gt; would be a great thing to have in a lib/object.js module or something,=
<br>
&gt; but it has nothing to do with the *process* and thus doesn&#39;t belon=
g in<br>
&gt; node.js or on the process object.)<br>
&gt;<br>
&gt; See attachments, or <a href=3D"http://github.com/isaacs/node/commits/m=
ixin-deprecation" target=3D"_blank">http://github.com/isaacs/node/commits/m=
ixin-deprecation</a><br>
<br>
<br>
</div>Thanks Issac, I took patches 1-5. I think it ought to be moved to sys=
<br>
for now. (Which is where I&#39;m putting all the utility functions.)<br>
<div><div></div><div class=3D"h5"><br>
--<br>
You received this message because you are subscribed to the Google Groups &=
quot;nodejs&quot; group.<br>
To post to this group, send email to <a href=3D"mailto:nodejs@googlegroups.=
com">nodejs@googlegroups.com</a>.<br>
To unsubscribe from this group, send email to <a href=3D"mailto:nodejs%2Bun=
subscribe@googlegroups.com">nodejs+unsubscribe@googlegroups.com</a>.<br>
For more options, visit this group at <a href=3D"http://groups.google.com/g=
roup/nodejs?hl=3Den" target=3D"_blank">http://groups.google.com/group/nodej=
s?hl=3Den</a>.<br>
<br>
</div></div></blockquote></div><br>

--0016e6d7e33acfac7f0481615642--
Create a group - Google Groups - Google Home - Terms of Service - Privacy Policy
Account Options
