[rfc][icedtea-web] a new reproducer for LiveConnect J->JS "get" tests
Jana Fabrikova
jfabriko at redhat.com
Thu Apr 11 05:58:22 PDT 2013
Hi Adam,
On Wed, 2013-03-06 at 12:50 -0500, Adam Domurad wrote:
> This isn't what your subject line says it is :-)
>
you are right, this was an exceptional time when all the reproducers
were a little bit simillar and I tried to send them at once, which was
not such a good idea, since I forgot to change the subject;)
> On 03/06/2013 10:58 AM, Jana Fabrikova wrote:
> > Hello,
> >
> > please see the attached patch of new reproducer for J->JS "evaluating
> > by calling JS eval from J".
>
> The patch itself looks good, but what was the motivation for this ? IMHO
> we do not need more than one reproducer that tests eval, which we
> already have (JSObjectFromEval). From our point-of-view it's sending a
> string to the javascript engine. All the specific object conversion
> stuff should be handled with unit tests (this may require a bit of
> refactoring to support, but should be do-able).
Ok, I will not push this reproducer, since the sending of string to JS
eval is already covered by JSObjectFromEval.
Thank you for the review,
cheers,
Jana
>
> Are you working on all these patches concurrently ? If not it'd be
> better to post them to list for feedback as you finish them, instead of
> all at once. Not a huge deal, though.
>
> > [..snip..]
>
> Happy hacking,
> -Adam
More information about the distro-pkg-dev
mailing list