[rfc][icedtea-web] Fix wrongly placed object construction helpers in IcedTeaScriptablePluginObject
adomurad at redhat.com
Wed Jun 5 12:49:51 PDT 2013
On 06/05/2013 03:48 PM, Adam Domurad wrote:
> On 06/05/2013 10:34 AM, Adam Domurad wrote:
>> Whoever wrote these static helpers switched up which class they are
>> associated with -- it took me a while of being confused to notice it,
>> too. This patch places the object creation helper methods in the
>> right class.
>> I added some more type safety too to prevent future confusion.
>> No changelog yet, I'll wait for an ack on the patch first.
> Here is additional refactoring on top of this patch and the
> '[rfc][icedtea-web] More testing for IcedTeaScriptablePluginObject.cc
> + memory leak fixes' fix patch. This stuff is necessary cleanup in my
> opinion; this code has been neglected.
> Again changelog waiting for ack, I recommend using a tool like eg meld
> to look at the differences here. They are all safe and minor IMO.
> The reason for touching this code is to try to make the 'getvalue'
> npapi call non-blocking, so it will never hang firefox.
To clarify, the reason for the refactoring is to make way for a nicer
patch for this. This patch does not accomplish it yet.
More information about the distro-pkg-dev