TextPosition and OrdinalNumber should be more like idiomatic numbers
Created attachment 296689 [details] Patch
Comment on attachment 296689 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=296689&action=review > Source/WebCore/xml/XMLErrors.cpp:63 > + if (type == fatal || (m_errorCount < maxErrors && !(m_lastErrorPosition || (m_lastErrorPosition->m_line != position.m_line && m_lastErrorPosition->m_column != position.m_column)))) { Shouldn't this be (!m_lastErrorPosition || (m_lastErrorPosition->... ?
> > Source/WebCore/xml/XMLErrors.cpp:63 > > + if (type == fatal || (m_errorCount < maxErrors && !(m_lastErrorPosition || (m_lastErrorPosition->m_line != position.m_line && m_lastErrorPosition->m_column != position.m_column)))) { > > Shouldn't this be (!m_lastErrorPosition || (m_lastErrorPosition->... ? Oops! Yup.
Created attachment 296693 [details] Patch
Committed r209627: <http://trac.webkit.org/changeset/209627>