<div dir="ltr">Hi,<div><br></div><div>It already is on master (but I just had some git issues, so you might have to refetch).</div><div><br></div><div>/Vlad</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">

On Fri, Feb 1, 2013 at 11:24 AM, Fredrik <span dir="ltr"><<a href="mailto:fredrik@erlang.org" target="_blank">fredrik@erlang.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="HOEnZb"><div class="h5">On 02/01/2013 11:13 AM, Nico Kruber wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Friday 01 Feb 2013 09:58:54 Vlad Dumitrescu wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi!<br>
<br>
I found a small problem in jinterface. It has been around for a while now,<br>
it happens in a corner case that probably nobody ever uses.<br>
<br>
In OtpErlangTuple(<u></u>OtpErlangObject[], int, int), when the size of the slice<br>
used is 0, then the elems fields remains uninitialized, it is the parameter<br>
that gets assigned.<br>
<br>
I used master as base, I hope it's still possible to do that.<br>
<br>
git fetch git://<a href="http://github.com/vladdu/otp.git" target="_blank">github.com/vladdu/otp.<u></u>git</a> tuple_constructor_bug<br>
<br>
<a href="https://github.com/vladdu/otp/compare/tuple_constructor_bug" target="_blank">https://github.com/vladdu/otp/<u></u>compare/tuple_constructor_bug</a><br>
<a href="https://github.com/vladdu/otp/compare/tuple_constructor_bug.patch" target="_blank">https://github.com/vladdu/otp/<u></u>compare/tuple_constructor_bug.<u></u>patch</a><br>
<br>
Given that there are no other tests for jinterface at this level, do you<br>
want me to write one for this case?<br>
<br>
best regards,<br>
Vlad<br>
</blockquote>
your branch is outdated so the diff is not clean, i.e. contains a lot of other<br>
commits<br>
->  please rebase your branch<br>
see <a href="https://github.com/erlang/otp/wiki/Submitting-patches" target="_blank">https://github.com/erlang/otp/<u></u>wiki/Submitting-patches</a><br>
<br>
it would probably also be good to base it on maint - but someone from the otp<br>
team needs to decide here :)<br>
<br>
your commit though seems reasonable<br>
<a href="https://github.com/vladdu/otp/commit/ee29a8aa733fbf1a1666b2f85a9fd6ff19d777b3" target="_blank">https://github.com/vladdu/otp/<u></u>commit/<u></u>ee29a8aa733fbf1a1666b2f85a9fd6<u></u>ff19d777b3</a><br>
<br>
<br>
Nico<br>
______________________________<u></u>_________________<br>
erlang-patches mailing list<br>
<a href="mailto:erlang-patches@erlang.org" target="_blank">erlang-patches@erlang.org</a><br>
<a href="http://erlang.org/mailman/listinfo/erlang-patches" target="_blank">http://erlang.org/mailman/<u></u>listinfo/erlang-patches</a><br>
</blockquote></div></div>
Hello,<br>
Just rebase it on master will work fine for me.<span class="HOEnZb"><font color="#888888"><br>
<br>
-- <br>
<br>
BR Fredrik Gustafsson<br>
Erlang OTP Team</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
______________________________<u></u>_________________<br>
erlang-patches mailing list<br>
<a href="mailto:erlang-patches@erlang.org" target="_blank">erlang-patches@erlang.org</a><br>
<a href="http://erlang.org/mailman/listinfo/erlang-patches" target="_blank">http://erlang.org/mailman/<u></u>listinfo/erlang-patches</a><br>
</div></div></blockquote></div><br></div>