(dormant account) Reporter
	Description • 14 years ago

Need a *hydra script to detect these. I think it's a matter of looking for generated functions that are destined for .ctors section.

	Ehren Metcalfe Assignee
	Comment 1 • 14 years ago

It's a bit messy to hardcode the name __static_initialization_and_destruction but this is what's done by gcc.

	Ehren Metcalfe Assignee
	Comment 2 • 14 years ago

note that the calls to __cxa_atexit are to ensure that the destructor for a statically initialized object is called on library unload/exit.

	(dormant account) Reporter
	Comment 3 • 14 years ago

(In reply to comment #2)
> Created an attachment (id=453310) [details]
> results
> 
> note that the calls to __cxa_atexit are to ensure that the destructor for a
> statically initialized object is called on library unload/exit.

Yeah, those are often harmful too. https://bugzilla.mozilla.org/show_bug.cgi?id=569629#c11

Very nice. You even found a few nsCOMPtr globals. Excellent work!

	(dormant account) Reporter
	Comment 4 • 14 years ago

Comment on attachment 453309 [details] [diff] [review]
script

>+          warning(pretty_var(arg) + " defined by call to constructor " + pretty_func(decl) +

>+          warning(pretty_func(decl) + " called during static initialization", location_of(decl));

Nice detailed error messages. This is even better than I expected.

	Ehren Metcalfe Assignee
	Comment 5 • 14 years ago

changing to use cfg_isn_iterator, carrying over r+.

	Ehren Metcalfe Assignee
	Comment 6 • 14 years ago

missed a testcase, carrying over r+

	Ehren Metcalfe Assignee
	Comment 7 • 14 years ago

added commit message to patch

	dwitte@gmail.com
	Comment 8 • 14 years ago

Comment on attachment 466456 [details] [diff] [review]
patch

http://hg.mozilla.org/mozilla-central/rev/b240b82abfaf