Skip to content

Commit e352a4e

Browse files
tniessenmarco-ippolito
authored andcommitted
src: update outdated references to spec sections
The exact section has changed in recent versions of ECMA-262, so fix the section number and explicitly mark the edition of the standard to avoid having to update it in the future. PR-URL: #53832 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Marco Ippolito <[email protected]>
1 parent 206c668 commit e352a4e

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

src/util-inl.h

+1-1
Original file line numberDiff line numberDiff line change
@@ -582,7 +582,7 @@ void ArrayBufferViewContents<T, S>::ReadValue(v8::Local<v8::Value> buf) {
582582
}
583583
}
584584

585-
// ECMA262 20.1.2.5
585+
// ECMA-262, 15th edition, 21.1.2.5. Number.isSafeInteger
586586
inline bool IsSafeJsInt(v8::Local<v8::Value> v) {
587587
if (!v->IsNumber()) return false;
588588
double v_d = v.As<v8::Number>()->Value();

src/util.h

+1-1
Original file line numberDiff line numberDiff line change
@@ -217,7 +217,7 @@ void DumpJavaScriptBacktrace(FILE* fp);
217217
#define UNREACHABLE(...) \
218218
ERROR_AND_ABORT("Unreachable code reached" __VA_OPT__(": ") __VA_ARGS__)
219219

220-
// ECMA262 20.1.2.6 Number.MAX_SAFE_INTEGER (2^53-1)
220+
// ECMA-262, 15th edition, 21.1.2.6. Number.MAX_SAFE_INTEGER (2^53-1)
221221
constexpr int64_t kMaxSafeJsInteger = 9007199254740991;
222222

223223
inline bool IsSafeJsInt(v8::Local<v8::Value> v);

0 commit comments

Comments
 (0)