Bug 22061 – create script to check for exit-time destructors

Bug 22061 - create script to check for exit-time destructors


Summary:	create script to check for exit-time destructors

Status:	RESOLVED FIXED

Product:	WebKit
Component:	Tools / Tests
Version:	528+ (Nightly build)
Platform:	Macintosh Mac OS X 10.5

Importance:	P2 Normal
Assigned To:	Darin Adler

URL:
Keywords:

Depends on:
Blocks:	21810
	Show dependency tree / graph

Reported:	2008-11-03 18:16 PST by Darin Adler
Modified:	2008-11-04 11:28 PST (History)

Attachments
patch (48.23 KB, patch) 2008-11-03 18:25 PST, Darin Adler	timothy: review+	Review Patch \| Details \| Formatted Diff \| Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note

You need to log in before you can comment on or make changes to this bug.

Description From Darin Adler 2008-11-03 18:16:21 PST

We'd like to eliminate all exit-time destructors from WebKit for faster process exit.

------- Comment #1 From Darin Adler 2008-11-03 18:25:08 PST -------

Created an attachment (id=24876) [details]
patch

------- Comment #2 From Timothy Hatcher 2008-11-04 05:26:59 PST -------

(From update of attachment 24876 [details])
> +void lockAtomicallyInitializedStaticMutex() {}
> +void unlockAtomicallyInitializedStaticMutex() { }

Space between the braces vs no space in the previous function.

> +    static String& key =  *new String("WebKitDatabaseVersionKey");

Extra space between the assignment.

r=me

------- Comment #3 From Greg Bolsinga 2008-11-04 11:09:56 PST -------

I think this fixes Bug 21810

------- Comment #4 From Greg Bolsinga 2008-11-04 11:11:44 PST -------

*** Bug 21810 has been marked as a duplicate of this bug. ***

------- Comment #5 From Darin Adler 2008-11-04 11:19:59 PST -------

(In reply to comment #3)
> I think this fixes Bug 21810

No, it doesn't fix it. It's just the tool to help us track the additional work needed.