From b5cc93b9094f6fd20cbb566dcf7fe547b6bead5c Mon Sep 17 00:00:00 2001 From: Ted Campbell Date: Tue, 20 Nov 2018 10:07:16 +0000 Subject: [PATCH] Bug 1508255 - Truncate '***' lines in js/src comments r=jandem Depends on D12385 Differential Revision: https://phabricator.services.mozilla.com/D12386 --HG-- extra : moz-landing-system : lando --- js/src/builtin/MapObject.cpp | 14 ++++++------- js/src/builtin/Reflect.cpp | 4 ++-- js/src/builtin/Stream.cpp | 26 ++++++++++++------------ js/src/gc/Marking.cpp | 20 +++++++++--------- js/src/gc/Tracer.cpp | 8 ++++---- js/src/jsapi.cpp | 2 +- js/src/jsapi.h | 6 +++--- js/src/vm/Debugger.cpp | 38 +++++++++++++++++------------------ js/src/vm/Debugger.h | 4 ++-- js/src/vm/EnvironmentObject.h | 8 ++++---- js/src/vm/Iteration.cpp | 2 +- js/src/vm/JSObject.cpp | 12 +++++------ js/src/vm/JSObject.h | 2 +- js/src/vm/NativeObject.cpp | 12 +++++------ js/src/vm/NativeObject.h | 4 ++-- js/src/vm/ObjectOperations.h | 2 +- js/src/vm/StringType.cpp | 2 +- js/src/vm/StringType.h | 2 +- js/src/vm/UbiNodeCensus.cpp | 6 +++--- 19 files changed, 87 insertions(+), 87 deletions(-) diff --git a/js/src/builtin/MapObject.cpp b/js/src/builtin/MapObject.cpp index 93dc6c87c176..32115767423a 100644 --- a/js/src/builtin/MapObject.cpp +++ b/js/src/builtin/MapObject.cpp @@ -29,7 +29,7 @@ using mozilla::NumberEqualsInt32; using JS::DoubleNaNValue; -/*** HashableValue *******************************************************************************/ +/*** HashableValue **********************************************************/ bool HashableValue::setValue(JSContext* cx, HandleValue v) @@ -138,7 +138,7 @@ HashableValue::trace(JSTracer* trc) const } -/*** MapIterator *********************************************************************************/ +/*** MapIterator ************************************************************/ namespace { @@ -410,7 +410,7 @@ MapIteratorObject::createResultPair(JSContext* cx) } -/*** Map *****************************************************************************************/ +/*** Map ********************************************************************/ const ClassOps MapObject::classOps_ = { nullptr, // addProperty @@ -1037,7 +1037,7 @@ MapObject::clear(JSContext* cx, HandleObject obj) } -/*** SetIterator *********************************************************************************/ +/*** SetIterator ************************************************************/ static const ClassOps SetIteratorObjectClassOps = { nullptr, /* addProperty */ @@ -1268,7 +1268,7 @@ SetIteratorObject::createResult(JSContext* cx) } -/*** Set *****************************************************************************************/ +/*** Set ********************************************************************/ const ClassOps SetObject::classOps_ = { nullptr, // addProperty @@ -1749,7 +1749,7 @@ SetObject::clear(JSContext* cx, unsigned argc, Value* vp) return CallNonGenericMethod(cx, is, clear_impl, args); } -/*** JS static utility functions *********************************************/ +/*** JS static utility functions ********************************************/ static bool forEach(const char* funcName, JSContext *cx, HandleObject obj, HandleValue callbackFn, HandleValue thisArg) @@ -1840,7 +1840,7 @@ CallObjFunc(bool(*ObjFunc)(JSContext* cx, Iter kind, return true; } -/*** JS public APIs **********************************************************/ +/*** JS public APIs *********************************************************/ JS_PUBLIC_API JSObject* JS::NewMapObject(JSContext* cx) diff --git a/js/src/builtin/Reflect.cpp b/js/src/builtin/Reflect.cpp index 567e6b5a70fa..8d6656a65ad5 100644 --- a/js/src/builtin/Reflect.cpp +++ b/js/src/builtin/Reflect.cpp @@ -18,7 +18,7 @@ using namespace js; -/*** Reflect methods *****************************************************************************/ +/*** Reflect methods ********************************************************/ /* ES6 26.1.4 Reflect.deleteProperty (target, propertyKey) */ static bool @@ -217,7 +217,7 @@ static const JSFunctionSpec methods[] = { }; -/*** Setup **************************************************************************************/ +/*** Setup ******************************************************************/ JSObject* js::InitReflect(JSContext* cx, Handle global) diff --git a/js/src/builtin/Stream.cpp b/js/src/builtin/Stream.cpp index bc652ce5671a..b772d405603a 100644 --- a/js/src/builtin/Stream.cpp +++ b/js/src/builtin/Stream.cpp @@ -520,7 +520,7 @@ const Class cls::protoClass_ = { \ }; -/*** 3.2. Class ReadableStream *******************************************************************/ +/*** 3.2. Class ReadableStream **********************************************/ /** * Streams spec, 3.2.3., steps 1-4. @@ -871,7 +871,7 @@ static const JSPropertySpec ReadableStream_properties[] = { CLASS_SPEC(ReadableStream, 0, SlotCount, 0, 0, JS_NULL_CLASS_OPS); -/*** 3.3. General readable stream abstract operations ********************************************/ +/*** 3.3. General readable stream abstract operations ***********************/ // Streams spec, 3.3.1. AcquireReadableStreamBYOBReader ( stream ) // Always inlined. @@ -1307,7 +1307,7 @@ ReadableStreamTee(JSContext* cx, } -/*** 3.4. The interface between readable streams and controllers *********************************/ +/*** 3.4. The interface between readable streams and controllers ************/ inline static MOZ_MUST_USE bool AppendToListAtSlot(JSContext* cx, @@ -1748,7 +1748,7 @@ ReadableStreamHasDefaultReader(JSContext* cx, } -/*** 3.5. Class ReadableStreamDefaultReader ******************************************************/ +/*** 3.5. Class ReadableStreamDefaultReader *********************************/ static MOZ_MUST_USE bool ReadableStreamReaderGenericInitialize(JSContext* cx, @@ -1985,7 +1985,7 @@ CLASS_SPEC(ReadableStreamDefaultReader, 1, SlotCount, ClassSpec::DontDefineConst JS_NULL_CLASS_OPS); -/*** 3.7. Readable stream reader abstract operations *********************************************/ +/*** 3.7. Readable stream reader abstract operations ************************/ // Streams spec, 3.7.1. IsReadableStreamDefaultReader ( x ) // Implemented via is() @@ -2202,7 +2202,7 @@ ReadableStreamDefaultReaderRead(JSContext* cx, } -/*** 3.8. Class ReadableStreamDefaultController **************************************************/ +/*** 3.8. Class ReadableStreamDefaultController *****************************/ inline static MOZ_MUST_USE bool ReadableStreamControllerCallPullIfNeeded(JSContext* cx, @@ -2743,7 +2743,7 @@ ReadableStreamDefaultControllerPullSteps(JSContext* cx, } -/*** 3.9. Readable stream default controller abstract operations *********************************/ +/*** 3.9. Readable stream default controller abstract operations ************/ // Streams spec, 3.9.1. IsReadableStreamDefaultController ( x ) // Implemented via is() @@ -3134,7 +3134,7 @@ ReadableStreamControllerGetDesiredSizeUnchecked(ReadableStreamController* contro } -/*** 3.10. Class ReadableByteStreamController ****************************************************/ +/*** 3.10. Class ReadableByteStreamController *******************************/ #if 0 // disable user-defined byte streams @@ -3580,7 +3580,7 @@ ReadableStreamControllerPullSteps(JSContext* cx, Handle() @@ -3747,7 +3747,7 @@ ReadableByteStreamControllerInvalidateBYOBRequest(JSContext* cx, // Unified with 3.9.3 above. -/*** 6.1. Queuing strategies *********************************************************************/ +/*** 6.1. Queuing strategies ************************************************/ // Streams spec, 6.1.2.2. new ByteLengthQueuingStrategy({ highWaterMark }) bool @@ -3853,7 +3853,7 @@ CLASS_SPEC(CountQueuingStrategy, 1, 0, 0, 0, JS_NULL_CLASS_OPS); #undef CLASS_SPEC -/*** 6.2. Queue-with-sizes operations ************************************************************/ +/*** 6.2. Queue-with-sizes operations ***************************************/ /** * Streams spec, 6.2.1. DequeueValue ( container ) nothrow @@ -3973,7 +3973,7 @@ ResetQueue(JSContext* cx, Handle unwrappedContainer) } -/*** 6.3. Miscellaneous operations ***************************************************************/ +/*** 6.3. Miscellaneous operations ******************************************/ /** * Appends the given |obj| to the given list |container|'s list. @@ -4092,7 +4092,7 @@ ValidateAndNormalizeQueuingStrategy(JSContext* cx, HandleValue size, } -/*** API entry points ****************************************************************************/ +/*** API entry points *******************************************************/ JS_FRIEND_API JSObject* js::UnwrapReadableStream(JSObject* obj) diff --git a/js/src/gc/Marking.cpp b/js/src/gc/Marking.cpp index 4204e5dd2b09..bbe49d6ef4fe 100644 --- a/js/src/gc/Marking.cpp +++ b/js/src/gc/Marking.cpp @@ -123,7 +123,7 @@ using mozilla::PodCopy; /* clang-format on */ -/*** Tracing Invariants **************************************************************************/ +/*** Tracing Invariants *****************************************************/ #if defined(DEBUG) template @@ -398,7 +398,7 @@ js::gc::AssertRootMarkingPhase(JSTracer* trc) #endif -/*** Tracing Interface ***************************************************************************/ +/*** Tracing Interface ******************************************************/ template T DoCallback(JS::CallbackTracer* trc, T* thingp, const char* name); template void DoMarking(GCMarker* gcmarker, T* thing); @@ -599,7 +599,7 @@ js::gc::TraceRangeInternal(JSTracer* trc, size_t len, T* vec, const char* name) } -/*** GC Marking Interface *************************************************************************/ +/*** GC Marking Interface ***************************************************/ namespace js { @@ -962,7 +962,7 @@ js::GCMarker::mark(T* thing) } -/*** Inline, Eager GC Marking *********************************************************************/ +/*** Inline, Eager GC Marking ***********************************************/ // Each of the eager, inline marking paths is directly preceeded by the // out-of-line, generic tracing code for comparison. Both paths must end up @@ -1622,7 +1622,7 @@ VisitTraceList(F f, const int32_t* traceList, uint8_t* memory, Args&&... args) } -/*** Mark-stack Marking ***************************************************************************/ +/*** Mark-stack Marking *****************************************************/ bool GCMarker::markUntilBudgetExhaused(SliceBudget& budget) @@ -2014,7 +2014,7 @@ GCMarker::restoreValueArray(const MarkStack::SavedValueArray& savedArray) } -/*** Mark Stack ***********************************************************************************/ +/*** Mark Stack *************************************************************/ static_assert(sizeof(MarkStack::TaggedPtr) == sizeof(uintptr_t), "A TaggedPtr should be the same size as a pointer"); @@ -2466,7 +2466,7 @@ MarkStackIter::saveValueArray(const MarkStack::SavedValueArray& savedArray) } -/*** GCMarker *************************************************************************************/ +/*** GCMarker ***************************************************************/ /* * ExpandWeakMaps: the GC is recomputing the liveness of WeakMap entries by @@ -2729,7 +2729,7 @@ GCMarker::sizeOfExcludingThis(mozilla::MallocSizeOf mallocSizeOf) const } -/*** Tenuring Tracer *****************************************************************************/ +/*** Tenuring Tracer ********************************************************/ namespace js { template @@ -3324,7 +3324,7 @@ js::TenuringTracer::moveStringToTenured(JSString* dst, JSString* src, AllocKind } -/*** IsMarked / IsAboutToBeFinalized **************************************************************/ +/*** IsMarked / IsAboutToBeFinalized ****************************************/ template static inline void @@ -3504,7 +3504,7 @@ FOR_EACH_PUBLIC_TAGGED_GC_POINTER_TYPE(INSTANTIATE_INTERNAL_MARKING_FUNCTIONS) } /* namespace js */ -/*** Cycle Collector Barrier Implementation *******************************************************/ +/*** Cycle Collector Barrier Implementation *********************************/ /* * The GC and CC are run independently. Consequently, the following sequence of diff --git a/js/src/gc/Tracer.cpp b/js/src/gc/Tracer.cpp index 33bbdee047ac..c132721ae7e5 100644 --- a/js/src/gc/Tracer.cpp +++ b/js/src/gc/Tracer.cpp @@ -40,7 +40,7 @@ CheckTracedThing(JSTracer* trc, T thing); } // namespace js -/*** Callback Tracer Dispatch ********************************************************************/ +/*** Callback Tracer Dispatch ***********************************************/ template T @@ -115,7 +115,7 @@ JS::CallbackTracer::getTracingEdgeName(char* buffer, size_t bufferSize) } -/*** Public Tracing API **************************************************************************/ +/*** Public Tracing API *****************************************************/ JS_PUBLIC_API void JS::TraceChildren(JSTracer* trc, GCCellPtr thing) @@ -181,7 +181,7 @@ JS::TraceIncomingCCWs(JSTracer* trc, const JS::CompartmentSet& compartments) } -/*** Cycle Collector Helpers **********************************************************************/ +/*** Cycle Collector Helpers ************************************************/ // This function is used by the Cycle Collector (CC) to trace through -- or in // CC parlance, traverse -- a Shape tree. The CC does not care about Shapes or @@ -298,7 +298,7 @@ gc::TraceCycleCollectorChildren(JS::CallbackTracer* trc, ObjectGroup* group) } -/*** Traced Edge Printer *************************************************************************/ +/*** Traced Edge Printer ****************************************************/ static size_t CountDecimalDigits(size_t num) diff --git a/js/src/jsapi.cpp b/js/src/jsapi.cpp index 64773a2f67fd..c85b4d589819 100644 --- a/js/src/jsapi.cpp +++ b/js/src/jsapi.cpp @@ -2024,7 +2024,7 @@ JS::AssertObjectBelongsToCurrentThread(JSObject* obj) } -/*** Standard internal methods *******************************************************************/ +/*** Standard internal methods **********************************************/ JS_PUBLIC_API bool JS_GetPrototype(JSContext* cx, HandleObject obj, MutableHandleObject result) diff --git a/js/src/jsapi.h b/js/src/jsapi.h index 92035941aa60..7b950569fe45 100644 --- a/js/src/jsapi.h +++ b/js/src/jsapi.h @@ -1888,7 +1888,7 @@ extern JS_PUBLIC_API bool JS_FreezeObject(JSContext* cx, JS::Handle obj); -/*** Property descriptors ************************************************************************/ +/*** Property descriptors ***************************************************/ namespace JS { @@ -2160,7 +2160,7 @@ FromPropertyDescriptor(JSContext* cx, } // namespace JS -/*** Standard internal methods ******************************************************************** +/*** Standard internal methods ********************************************** * * The functions below are the fundamental operations on objects. * @@ -2741,7 +2741,7 @@ extern JS_PUBLIC_API JSObject* JS_New(JSContext* cx, JS::HandleObject ctor, const JS::HandleValueArray& args); -/*** Other property-defining functions ***********************************************************/ +/*** Other property-defining functions **************************************/ extern JS_PUBLIC_API JSObject* JS_DefineObject(JSContext* cx, JS::HandleObject obj, const char* name, diff --git a/js/src/vm/Debugger.cpp b/js/src/vm/Debugger.cpp index b35a17c18e43..b8d76bc47436 100644 --- a/js/src/vm/Debugger.cpp +++ b/js/src/vm/Debugger.cpp @@ -75,7 +75,7 @@ using mozilla::TimeDuration; using mozilla::TimeStamp; -/*** Forward declarations, ClassOps and Classes **************************************************/ +/*** Forward declarations, ClassOps and Classes *****************************/ static void DebuggerFrame_finalize(FreeOp* fop, JSObject* obj); static void DebuggerFrame_trace(JSTracer* trc, JSObject* obj); @@ -232,7 +232,7 @@ static const Class DebuggerSource_class = { }; -/*** Utils ***************************************************************************************/ +/*** Utils ******************************************************************/ /* * If fun is an interpreted function, remove any async function/generator @@ -564,7 +564,7 @@ RequireGlobalObject(JSContext* cx, HandleValue dbgobj, HandleObject referent) } -/*** Breakpoints *********************************************************************************/ +/*** Breakpoints ************************************************************/ BreakpointSite::BreakpointSite(Type type) : type_(type), enabledCount(0) @@ -695,7 +695,7 @@ WasmBreakpointSite::destroyIfEmpty(FreeOp* fop) } } -/*** Debugger hook dispatch **********************************************************************/ +/*** Debugger hook dispatch *************************************************/ Debugger::Debugger(JSContext* cx, NativeObject* dbg) : object(dbg), @@ -1597,7 +1597,7 @@ Debugger::unwrapPropertyDescriptor(JSContext* cx, HandleObject obj, } -/*** Debuggee resumption values and debugger error handling **************************************/ +/*** Debuggee resumption values and debugger error handling *****************/ static bool GetResumptionProperty(JSContext* cx, HandleObject obj, HandlePropertyName name, ResumeMode namedMode, @@ -1902,7 +1902,7 @@ Debugger::processHandlerResult(Maybe& ar, bool success, const Value& } -/*** Debuggee completion values ******************************************************************/ +/*** Debuggee completion values *********************************************/ /* static */ void Debugger::resultToCompletion(JSContext* cx, bool ok, const Value& rv, @@ -1982,7 +1982,7 @@ Debugger::receiveCompletionValue(Maybe& ar, bool ok, } -/*** Firing debugger hooks ***********************************************************************/ +/*** Firing debugger hooks **************************************************/ static bool CallMethodIfPresent(JSContext* cx, HandleObject obj, const char* name, size_t argc, Value* argv, @@ -2663,7 +2663,7 @@ Debugger::slowPathPromiseHook(JSContext* cx, Hook hook, Handle p } -/*** Debugger code invalidation for observing execution ******************************************/ +/*** Debugger code invalidation for observing execution *********************/ class MOZ_RAII ExecutionObservableRealms : public Debugger::ExecutionObservableSet { @@ -3199,7 +3199,7 @@ Debugger::updateObservesAsmJSOnDebuggees(IsObserving observing) } -/*** Allocations Tracking *************************************************************************/ +/*** Allocations Tracking ***************************************************/ /* static */ bool Debugger::cannotTrackAllocations(const GlobalObject& global) @@ -3295,7 +3295,7 @@ Debugger::removeAllocationsTrackingForAllDebuggees() -/*** Debugger JSObjects **************************************************************************/ +/*** Debugger JSObjects *****************************************************/ void Debugger::traceCrossCompartmentEdges(JSTracer* trc) @@ -5793,7 +5793,7 @@ const JSFunctionSpec Debugger::static_methods[] { JS_FS_END }; -/*** Debugger.Script *****************************************************************************/ +/*** Debugger.Script ********************************************************/ // Get the Debugger.Script referent as bare Cell. This should only be used for // GC operations like tracing. Please use GetScriptReferent below. @@ -7777,7 +7777,7 @@ static const JSFunctionSpec DebuggerScript_methods[] = { }; -/*** Debugger.Source *****************************************************************************/ +/*** Debugger.Source ********************************************************/ // For internal use only. static inline NativeObject* @@ -8370,7 +8370,7 @@ static const JSFunctionSpec DebuggerSource_methods[] = { }; -/*** Debugger.Frame ******************************************************************************/ +/*** Debugger.Frame *********************************************************/ ScriptedOnStepHandler::ScriptedOnStepHandler(JSObject* object) : object_(object) @@ -9688,7 +9688,7 @@ const JSFunctionSpec DebuggerFrame::methods_[] = { }; -/*** Debugger.Object *****************************************************************************/ +/*** Debugger.Object ********************************************************/ void DebuggerObject_trace(JSTracer* trc, JSObject* obj) @@ -11982,7 +11982,7 @@ DebuggerObject::getScriptedProxyHandler(JSContext* cx, HandleDebuggerObject obje } -/*** Debugger.Environment ************************************************************************/ +/*** Debugger.Environment ***************************************************/ void DebuggerEnv_trace(JSTracer* trc, JSObject* obj) @@ -12578,7 +12578,7 @@ DebuggerEnvironment::setVariable(JSContext* cx, HandleDebuggerEnvironment enviro } -/*** JS::dbg::Builder ****************************************************************************/ +/*** JS::dbg::Builder *******************************************************/ Builder::Builder(JSContext* cx, js::Debugger* debugger) : debuggerObject(cx, debugger->toJSObject().get()), @@ -12655,7 +12655,7 @@ Builder::newObject(JSContext* cx) } -/*** JS::dbg::AutoEntryMonitor ******************************************************************/ +/*** JS::dbg::AutoEntryMonitor **********************************************/ AutoEntryMonitor::AutoEntryMonitor(JSContext* cx) : cx_(cx), @@ -12670,7 +12670,7 @@ AutoEntryMonitor::~AutoEntryMonitor() } -/*** Glue ****************************************************************************************/ +/*** Glue *******************************************************************/ extern JS_PUBLIC_API bool JS_DefineDebuggerObject(JSContext* cx, HandleObject obj) @@ -12836,7 +12836,7 @@ js::CheckDebuggeeThing(JSObject* obj, bool invisibleOk) #endif // DEBUG -/*** JS::dbg::GarbageCollectionEvent **************************************************************/ +/*** JS::dbg::GarbageCollectionEvent ****************************************/ namespace JS { namespace dbg { diff --git a/js/src/vm/Debugger.h b/js/src/vm/Debugger.h index 638904c02a1f..29b4a6353e8d 100644 --- a/js/src/vm/Debugger.h +++ b/js/src/vm/Debugger.h @@ -940,7 +940,7 @@ class Debugger : private mozilla::LinkedListElement WeakGlobalObjectSet::Range allDebuggees() const { return debuggees.all(); } - /*********************************** Methods for interaction with the GC. */ + /*** Methods for interaction with the GC. *******************************/ /* * A Debugger object is live if: @@ -1066,7 +1066,7 @@ class Debugger : private mozilla::LinkedListElement // promise can only make the transition from unsettled to settled once. static inline void onPromiseSettled(JSContext* cx, Handle promise); - /************************************* Functions for use by Debugger.cpp. */ + /*** Functions for use by Debugger.cpp. *********************************/ inline bool observesEnterFrame() const; inline bool observesNewScript() const; diff --git a/js/src/vm/EnvironmentObject.h b/js/src/vm/EnvironmentObject.h index 4c3cd92352a0..da3bf9e9f19a 100644 --- a/js/src/vm/EnvironmentObject.h +++ b/js/src/vm/EnvironmentObject.h @@ -40,10 +40,10 @@ extern JSScript* EnvironmentCoordinateFunctionScript(JSScript* script, jsbytecode* pc); -/*** Environment objects *****************************************************/ +/*** Environment objects ****************************************************/ -/*** Environment objects *****************************************************/ +/*** Environment objects ****************************************************/ // clang-format off /* @@ -696,7 +696,7 @@ class RuntimeLexicalErrorObject : public EnvironmentObject }; -/*****************************************************************************/ +/****************************************************************************/ // A environment iterator describes the active environments starting from an // environment, scope pair. This pair may be derived from the current point of @@ -882,7 +882,7 @@ class LiveEnvironmentVal }; -/*****************************************************************************/ +/****************************************************************************/ /* * Debug environment objects diff --git a/js/src/vm/Iteration.cpp b/js/src/vm/Iteration.cpp index 9f63e3770e84..e4fb07e35e9a 100644 --- a/js/src/vm/Iteration.cpp +++ b/js/src/vm/Iteration.cpp @@ -1115,7 +1115,7 @@ Realm::createIterResultTemplateObject(JSContext* cx, WithObjectPrototype withPro return templateObject; } -/*** Iterator objects ****************************************************************************/ +/*** Iterator objects *******************************************************/ size_t PropertyIteratorObject::sizeOfMisc(mozilla::MallocSizeOf mallocSizeOf) const diff --git a/js/src/vm/JSObject.cpp b/js/src/vm/JSObject.cpp index d9ee14258cec..14c60e2f4484 100644 --- a/js/src/vm/JSObject.cpp +++ b/js/src/vm/JSObject.cpp @@ -291,7 +291,7 @@ js::Throw(JSContext* cx, HandleId id, unsigned errorNumber, const char* details) } -/*** PropertyDescriptor operations and DefineProperties ******************************************/ +/*** PropertyDescriptor operations and DefineProperties *********************/ static const char js_getter_str[] = "getter"; static const char js_setter_str[] = "setter"; @@ -496,7 +496,7 @@ js::ReadPropertyDescriptors(JSContext* cx, HandleObject props, bool checkAccesso return true; } -/*** Seal and freeze *****************************************************************************/ +/*** Seal and freeze ********************************************************/ static unsigned GetSealedOrFrozenAttributes(unsigned attrs, IntegrityLevel level) @@ -2874,7 +2874,7 @@ js::GetPrototypeIfOrdinary(JSContext* cx, HandleObject obj, bool* isOrdinary, return true; } -/*** ES6 standard internal methods ***************************************************************/ +/*** ES6 standard internal methods ******************************************/ bool js::SetPrototype(JSContext* cx, HandleObject obj, HandleObject proto, JS::ObjectOpResult& result) @@ -3156,7 +3156,7 @@ js::DefineDataElement(JSContext* cx, HandleObject obj, uint32_t index, HandleVal return DefineDataProperty(cx, obj, id, value, attrs); } -/*** SpiderMonkey nonstandard internal methods ***************************************************/ +/*** SpiderMonkey nonstandard internal methods ******************************/ // Mark an object as having an immutable prototype // @@ -3250,7 +3250,7 @@ js::DefineFunctions(JSContext* cx, HandleObject obj, const JSFunctionSpec* fs, } -/*** ToPrimitive *************************************************************/ +/*** ToPrimitive ************************************************************/ /* * Gets |obj[id]|. If that value's not callable, returns true and stores an @@ -3668,7 +3668,7 @@ GetObjectSlotNameFunctor::operator()(JS::CallbackTracer* trc, char* buf, size_t } } -/*** Debugging routines **************************************************************************/ +/*** Debugging routines *****************************************************/ #if defined(DEBUG) || defined(JS_JITSPEW) diff --git a/js/src/vm/JSObject.h b/js/src/vm/JSObject.h index e21354bea8df..e3aa68ab7f83 100644 --- a/js/src/vm/JSObject.h +++ b/js/src/vm/JSObject.h @@ -33,7 +33,7 @@ namespace gc { class RelocationOverlay; } // namespace gc -/******************************************************************************/ +/****************************************************************************/ class GlobalObject; class NewObjectCache; diff --git a/js/src/vm/NativeObject.cpp b/js/src/vm/NativeObject.cpp index 9174253b9591..c1b02102834e 100644 --- a/js/src/vm/NativeObject.cpp +++ b/js/src/vm/NativeObject.cpp @@ -1207,7 +1207,7 @@ template bool js::NativeLookupOwnProperty(JSContext* cx, NativeObject* const& obj, const jsid& id, FakeMutableHandle propp); -/*** [[DefineOwnProperty]] ***********************************************************************/ +/*** [[DefineOwnProperty]] **************************************************/ static MOZ_ALWAYS_INLINE bool CallAddPropertyHook(JSContext* cx, HandleNativeObject obj, HandleId id, HandleValue value) @@ -2172,7 +2172,7 @@ js::AddOrUpdateSparseElementHelper(JSContext* cx, HandleArrayObject obj, int32_t } -/*** [[HasProperty]] *****************************************************************************/ +/*** [[HasProperty]] ********************************************************/ // ES6 draft rev31 9.1.7.1 OrdinaryHasProperty bool @@ -2228,7 +2228,7 @@ js::NativeHasProperty(JSContext* cx, HandleNativeObject obj, HandleId id, bool* } -/*** [[GetOwnPropertyDescriptor]] ****************************************************************/ +/*** [[GetOwnPropertyDescriptor]] *******************************************/ bool js::NativeGetOwnPropertyDescriptor(JSContext* cx, HandleNativeObject obj, HandleId id, @@ -2291,7 +2291,7 @@ js::NativeGetOwnPropertyDescriptor(JSContext* cx, HandleNativeObject obj, Handle } -/*** [[Get]] *************************************************************************************/ +/*** [[Get]] ****************************************************************/ static inline bool CallGetter(JSContext* cx, HandleObject obj, HandleValue receiver, HandleShape shape, @@ -2702,7 +2702,7 @@ js::GetNameBoundInEnvironment(JSContext* cx, HandleObject envArg, HandleId id, M } -/*** [[Set]] *************************************************************************************/ +/*** [[Set]] ****************************************************************/ static bool MaybeReportUndeclaredVarAssignment(JSContext* cx, HandleId id) @@ -3108,7 +3108,7 @@ js::NativeSetElement(JSContext* cx, HandleNativeObject obj, uint32_t index, Hand return NativeSetProperty(cx, obj, id, v, receiver, result); } -/*** [[Delete]] **********************************************************************************/ +/*** [[Delete]] *************************************************************/ // ES6 draft rev31 9.1.10 [[Delete]] bool diff --git a/js/src/vm/NativeObject.h b/js/src/vm/NativeObject.h index ae9704da9954..d153f415c950 100644 --- a/js/src/vm/NativeObject.h +++ b/js/src/vm/NativeObject.h @@ -1553,7 +1553,7 @@ NativeObject::privateWriteBarrierPre(void** oldval) } -/*** Standard internal methods *******************************************************************/ +/*** Standard internal methods **********************************************/ /* * These functions should follow the algorithms in ES6 draft rev 29 section 9.1 @@ -1656,7 +1656,7 @@ extern bool NativeDeleteProperty(JSContext* cx, HandleNativeObject obj, HandleId id, ObjectOpResult& result); -/*** SpiderMonkey nonstandard internal methods ***************************************************/ +/*** SpiderMonkey nonstandard internal methods ******************************/ template extern bool diff --git a/js/src/vm/ObjectOperations.h b/js/src/vm/ObjectOperations.h index 94a9c479b7d5..06483467b504 100644 --- a/js/src/vm/ObjectOperations.h +++ b/js/src/vm/ObjectOperations.h @@ -250,7 +250,7 @@ inline bool DeleteElement(JSContext* cx, JS::Handle obj, uint32_t index, JS::ObjectOpResult& result); -/*** SpiderMonkey nonstandard internal methods ***************************************************/ +/*** SpiderMonkey nonstandard internal methods ******************************/ /** * If |obj| (underneath any functionally-transparent wrapper proxies) has as diff --git a/js/src/vm/StringType.cpp b/js/src/vm/StringType.cpp index 588476b69b0a..5d3b56f82e3b 100644 --- a/js/src/vm/StringType.cpp +++ b/js/src/vm/StringType.cpp @@ -2165,7 +2165,7 @@ JSString::fillWithRepresentatives(JSContext* cx, HandleArrayObject array) } -/*** Conversions *********************************************************************************/ +/*** Conversions ************************************************************/ UniqueChars js::EncodeLatin1(JSContext* cx, JSString* str) diff --git a/js/src/vm/StringType.h b/js/src/vm/StringType.h index 4e666d0bab52..9eaca4566d5e 100644 --- a/js/src/vm/StringType.h +++ b/js/src/vm/StringType.h @@ -1690,7 +1690,7 @@ JSString* SubstringKernel(JSContext* cx, HandleString str, int32_t beginInt, int32_t lengthInt); -/*** Conversions *********************************************************************************/ +/*** Conversions ************************************************************/ /* * Convert a string to a printable C string. diff --git a/js/src/vm/UbiNodeCensus.cpp b/js/src/vm/UbiNodeCensus.cpp index e351e9827699..d90c57d3ef58 100644 --- a/js/src/vm/UbiNodeCensus.cpp +++ b/js/src/vm/UbiNodeCensus.cpp @@ -34,7 +34,7 @@ CountDeleter::operator()(CountBase* ptr) js_free(ptr); } -/*** Count Types ***********************************************************************************/ +/*** Count Types ************************************************************/ // The simplest type: just count everything. class SimpleCount : public CountType { @@ -1134,7 +1134,7 @@ ByFilename::report(JSContext* cx, CountBase& countBase, MutableHandleValue repor } -/*** Census Handler *******************************************************************************/ +/*** Census Handler *********************************************************/ JS_PUBLIC_API bool CensusHandler::operator() (BreadthFirst& traversal, @@ -1170,7 +1170,7 @@ CensusHandler::operator() (BreadthFirst& traversal, } -/*** Parsing Breakdowns ***************************************************************************/ +/*** Parsing Breakdowns *****************************************************/ static CountTypePtr ParseChildBreakdown(JSContext* cx, HandleObject breakdown, PropertyName* prop)