[rfc] [icedtea-web] Reproducer for clipboard (reopened PR708)
Adam Domurad
adomurad at redhat.com
Thu Oct 25 08:14:48 PDT 2012
Note: Most of message snipped.
>>> + clipboard.setContents(strSel, null);
>>> + }
>>
>> Its not an issue at all, but the 'public' is a little funny for
>> something that has no external
>> existence :)
>
> hmhm left public ook?
For sure, I like to comment like this, but I'd have to stop if they were
all taken as must-do's :),
>>> +}
>>
>> Content-wise the patch is good, nice work handling this
>> tricky-to-test feature!
>> Apologies for the nitpicking :)
>>
> I'm Glad for them!
>
>
> J
Thanks for the update, I'm sure it took a little jogging of memory!
Good for HEAD.
Thanks,
- Adam
More information about the distro-pkg-dev
mailing list