<!DOCTYPE html>
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
</head>
<body>
<p>It was suggested in that PR that the discussion be brought to
this mailing list to gather any inputs and interest in the change.
Except for your current mail, there hasn't been any other
discussion here in the mailing list about it.</p>
<p>In the PR itself I haven't seen any objections to this change,
but it isn't clear what motivated the new constructor. Does it
fall into the good-to-have category? How frequently the
SequenceInputStream gets used and whether this new constructor
would help such usages wasn't explained either. </p>
<p>If you do pick this up, I think it would be useful to find out
what kind of usage the SequenceInputStream class sees in libraries
and whether this new constructor is going to be helpful for such
usages.</p>
<p>-Jaikiran<br>
</p>
<div class="moz-cite-prefix">On 17/10/24 10:36 am, David Alayachew
wrote:<br>
</div>
<blockquote type="cite"
cite="mid:CAA9v-_NMsvL+QvTi36kYgkX1CbaTVmr2WV6Ek-o6OXBTtJs3pA@mail.gmail.com">
<meta http-equiv="content-type" content="text/html; charset=UTF-8">
<p dir="ltr">Hello Jaikiran,</p>
<p dir="ltr">Thanks for the response. This is exactly what I was
looking for.</p>
<p dir="ltr">Does this mean that this is up for grabs? There was a
comment near the bottom about more evaluations, but that was
also in 2023.</p>
<p dir="ltr">If all I need to do is continue the work that the
other person was doing, I am more than happy to.</p>
<p dir="ltr">Looks like in the pr attempted but closed, all
Aleksey Shipilëv wanted was for the developer to activate the
test suite, so that they could see that the tests passed. It
sounds like all I need to do is do that and then we should be
good for review?</p>
<br>
<div class="gmail_quote">
<div dir="ltr" class="gmail_attr">On Thu, Oct 17, 2024, 12:54 AM
Jaikiran Pai <<a href="mailto:jai.forums2013@gmail.com"
moz-do-not-send="true" class="moz-txt-link-freetext">jai.forums2013@gmail.com</a>>
wrote:<br>
</div>
<blockquote class="gmail_quote"
style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello
David,<br>
<br>
There's an enhancement request for this here <br>
<a href="https://bugs.openjdk.org/browse/JDK-8305947"
rel="noreferrer noreferrer" target="_blank"
moz-do-not-send="true" class="moz-txt-link-freetext">https://bugs.openjdk.org/browse/JDK-8305947</a><br>
<br>
-Jaikiran<br>
<br>
On 17/10/24 9:17 am, David Alayachew wrote:<br>
> Hello Core Libs Dev Team,<br>
><br>
> Currently, java.io.SequenceInputStream only has 2
constructors -- one <br>
> that takes in 2 instances of InputStream, and another
that takes in an <br>
> Enumeration of type InputStream.<br>
><br>
> Would it make sense to add a third constructor that takes
in multiple <br>
> instances of InputStream? I don't know if a list, an
array, varargs, <br>
> etc makes more sense. But I would think a 3rd constructor
would <br>
> improve this API.<br>
><br>
> Thank you for your time and help.<br>
> David Alayachew<br>
</blockquote>
</div>
</blockquote>
</body>
</html>