------ art_5493_32152128.1209587062209
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Content-Disposition: inline
It said on the web site to throw ones patches this direction.
Here's one for a bug submitted [and fixed] about 6 months ago. Guess
I forgot to submit the patch itself.
This is a fix for the following leaks:
def leak
"name".split(/::/)
end
loop {leak} # leaks
def grow
for i in 1..100
eval "b#{i}
end
end
1500000.times {grow} # leaks
[this style of leak is reported on several web sites as a ruby leak, as well]
Ref:
http://rubyforge.org/tracker/?funcÛÕail&atid98&aid425&group_idB6
Reservations:
I just added this patch from someone else's post--I'm not positive
there aren't side-effects, though I haven't seen any. I'm not scope
pro here :)
Also for some reason this worked on my Linux machine but not my mac
machine [still leaks]. Maybe because of my linking to some old
'legacy' libs or something.
Let me know what I can do to help it get accepted.
-R
PS Here's a copy in case the attachment doesn't get through.
Index: parse.y
--- parse.y (revision 16244)
+++ parse.y (working copy)
@@ -5821,7 +5821,8 @@
if (!(ruby_scope->flags & SCOPE_CLONE))
xfree(ruby_scope->local_tbl);
}
- ruby_scope->local_tbl ocal_tbl();
+ ruby_scope->local_vars[-1] ;
+ ruby_scope->local_tbl ocal_tbl();
}
}
local_pop();
------ art_5493_32152128.1209587062209
Content-Type: application/octet-stream; name=unleak.diff
Content-Transfer-Encoding: base64
X-Attachment-Id: f_ffocoqdr0
Content-Disposition: attachment; filename=unleak.diff
SW5kZXg6IHBhcnNlLnkKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gcGFyc2UueQkocmV2aXNpb24gMTYyNDQpCisr
KyBwYXJzZS55CSh3b3JraW5nIGNvcHkpCkBAIC01ODIxLDcgKzU4MjEsOCBAQAogICAgICAgICAg
ICAgICAgaWYgKCEocnVieV9zY29wZS0+ZmxhZ3MgJiBTQ09QRV9DTE9ORSkpCiAgICAgICAgICAg
ICAgICAgICAgeGZyZWUocnVieV9zY29wZS0+bG9jYWxfdGJsKTsKIAkgICAgfQotCSAgICBydWJ5
X3Njb3BlLT5sb2NhbF90YmwgPSBsb2NhbF90YmwoKTsKKyAgICAgICAgICAgIHJ1Ynlfc2NvcGUt
PmxvY2FsX3ZhcnNbLTFdID0gMDsKKyAgICAgICAgICAgIHJ1Ynlfc2NvcGUtPmxvY2FsX3RibCA9
IGxvY2FsX3RibCgpOwogCX0KICAgICB9CiAgICAgbG9jYWxfcG9wKCk7Cg ------ art_5493_32152128.1209587062209--