From a5b7643b9e0999ce332c1a7f319796b17135900a Mon Sep 17 00:00:00 2001 From: Jon Coppeard Date: Fri, 12 Apr 2013 09:31:43 +0100 Subject: [PATCH] Bug 860704 - GC: Comment says RawT converts to RootedT r=terrence --HG-- extra : rebase_source : 2f5388adb27a97ee3762d6cab7ec0591b0f718b3 --- js/public/RootingAPI.h | 22 +++++++++++----------- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/js/public/RootingAPI.h b/js/public/RootingAPI.h index 0f1e6b459c5..9f9fa9df53f 100644 --- a/js/public/RootingAPI.h +++ b/js/public/RootingAPI.h @@ -49,7 +49,7 @@ * dangerous-looking actions cannot trigger a GC: js_malloc, cx->malloc_, * rt->malloc_, and friends and JS_ReportOutOfMemory. * - * The following family of four classes will exactly root a stack location. + * The following family of three classes will exactly root a stack location. * Incorrect usage of these classes will result in a compile error in almost * all cases. Therefore, it is very hard to be incorrectly rooted if you use * these classes exclusively. These classes are all templated on the type T of @@ -58,7 +58,7 @@ * - Rooted declares a variable of type T, whose value is always rooted. * Rooted may be automatically coerced to a Handle, below. Rooted * should be used whenever a local variable's value may be held live across a - * call which can trigger a GC. This is generally true of + * call which can trigger a GC. * * - Handle is a const reference to a Rooted. Functions which take GC * things or values as arguments and need to root those arguments should @@ -90,16 +90,16 @@ * The following diagram explains the list of supported, implicit type * conversions between classes of this family: * - * RawT -----> Rooted ----> Handle - * | ^ - * | | - * | | - * +---> MutableHandle - * (via &) + * Rooted ----> Handle + * | ^ + * | | + * | | + * +---> MutableHandle + * (via &) * - * Currently all of these types implicit conversion to RawT. These are present - * only for the purpose of bootstrapping exact rooting and will be removed in - * the future (Bug 817164). + * Currently, all of these types have an implicit conversion to RawT. These are + * present only for the purpose of bootstrapping exact rooting and will be + * removed in the future (Bug 817164). */ namespace js {