Deploy OrdinalNumber in JSC::SourceCode
Created attachment 296703 [details] Patch
Comment on attachment 296703 [details] Patch r=me
Comment on attachment 296703 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=296703&action=review > Source/JavaScriptCore/parser/SourceCode.h:40 > + , m_firstLine(OrdinalNumber::beforeFirst()) > + , m_startColumn(OrdinalNumber::beforeFirst()) Is this just to indicate that this value doesn't mean anything? Why not just have a default empty constructor that does that?
> > Source/JavaScriptCore/parser/SourceCode.h:40 > > + , m_firstLine(OrdinalNumber::beforeFirst()) > > + , m_startColumn(OrdinalNumber::beforeFirst()) > > Is this just to indicate that this value doesn't mean anything? Why not just > have a default empty constructor that does that? The empty constructor constructs zero. Just like int() or unsigned() would. Eventually, we'll switch off of beforeFirst() and just have clients use std::optional.
Committed r209632: <http://trac.webkit.org/changeset/209632>