[Webkit-unassigned] [Bug 65414] DequeIterator lacks default constructor

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=65414





--- Comment #4 from Darin Adler <darin at apple.com>  2011-09-06 08:47:07 PST ---
(From update of attachment 106405)
View in context: https://bugs.webkit.org/attachment.cgi?id=106405&action=review

> Source/JavaScriptCore/wtf/Deque.h:598
> +        , m_next(0)
> +        , m_previous(0)

I’m not sure I understand the full rationale for setting these in a debug build but not a release build. Will this help us catch mistakes we would otherwise miss? Or perhaps it will hide mistakes we might otherwise catch.

> Source/JavaScriptCore/wtf/MainThread.cpp:-213
> -        // We must redefine 'i' each pass, because the itererator's operator= 
> -        // requires 'this' to be valid, and remove() invalidates all iterators

The substance of this comment still seems correct, even if the word redefine is no longer accurate, so it’s not clear to me that it’s good to remove it.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.