mozilla
/
pjs
зеркало из https://github.com/mozilla/pjs.git
1
0
Форкнуть 0
Код Задачи Пакеты Проекты Релизы Вики Активность

Use GC-safe vector of JSFunction* to hold getters/setters.

This commit is contained in:
beard%netscape.com 2000-12-30 07:46:18 +00:00
Родитель 99f9432582
Коммит df3abcca9b
2 изменённых файлов: 16 добавлений и 14 удалений
Показать статистику Скачать Patch файл Скачать Diff файл Показать все файлы Свернуть все файлы

15
js/js2/jsclasses.h
Просмотреть файл

@ -73,7 +73,8 @@ namespace JSClasses {
typedef std::pair<String, JSFunction*> MethodEntry;
typedef std::vector<MethodEntry> JSMethods;
typedef std::vector<MethodEntry, gc_allocator<MethodEntry> > JSMethods;
typedef std::vector<JSFunction*, gc_allocator<JSFunction*> > JSFunctions;
/**
* Represents a class in the JavaScript 2 (ECMA 4) language.
@ -92,8 +93,8 @@ namespace JSClasses {
JSMethods mMethods;
bool mHasGetters; // tracks whether any getters/setters get assigned
bool mHasSetters;
JSFunction **mGetters; // allocated at 'complete()' time
JSFunction **mSetters;
JSFunctions mGetters; // allocated at 'complete()' time
JSFunctions mSetters;
public:
JSClass(JSScope* scope, const String& name, JSClass* superClass = 0)
: JSType(name, superClass),
@ -180,12 +181,12 @@ namespace JSClasses {
bool hasGetter(uint32 index)
{
return (mGetters && mGetters[index]);
return (index < mGetters.size() && mGetters[index]);
}
bool hasSetter(uint32 index)
{
return (mSetters && mSetters[index]);
return (index < mSetters.size() && mSetters[index]);
}
JSFunction* getter(uint32 index)
@ -270,8 +271,8 @@ namespace JSClasses {
bool complete()
{
if (mHasGetters || mHasSetters) {
if (mHasGetters) mGetters = new JSFunction*[mSlotCount];
if (mHasSetters) mSetters = new JSFunction*[mSlotCount];
if (mHasGetters) mGetters.resize(mSlotCount);
if (mHasSetters) mSetters.resize(mSlotCount);
JSSlots::iterator end = mSlots.end();
for (JSSlots::iterator i = mSlots.begin(); i != end; i++) {
if (mHasGetters) mGetters[i->second.mIndex] = i->second.mGetter;

15
js2/src/jsclasses.h
Просмотреть файл

@ -73,7 +73,8 @@ namespace JSClasses {
typedef std::pair<String, JSFunction*> MethodEntry;
typedef std::vector<MethodEntry> JSMethods;
typedef std::vector<MethodEntry, gc_allocator<MethodEntry> > JSMethods;
typedef std::vector<JSFunction*, gc_allocator<JSFunction*> > JSFunctions;
/**
* Represents a class in the JavaScript 2 (ECMA 4) language.
@ -92,8 +93,8 @@ namespace JSClasses {
JSMethods mMethods;
bool mHasGetters; // tracks whether any getters/setters get assigned
bool mHasSetters;
JSFunction **mGetters; // allocated at 'complete()' time
JSFunction **mSetters;
JSFunctions mGetters; // allocated at 'complete()' time
JSFunctions mSetters;
public:
JSClass(JSScope* scope, const String& name, JSClass* superClass = 0)
: JSType(name, superClass),
@ -180,12 +181,12 @@ namespace JSClasses {
bool hasGetter(uint32 index)
{
return (mGetters && mGetters[index]);
return (index < mGetters.size() && mGetters[index]);
}
bool hasSetter(uint32 index)
{
return (mSetters && mSetters[index]);
return (index < mSetters.size() && mSetters[index]);
}
JSFunction* getter(uint32 index)
@ -270,8 +271,8 @@ namespace JSClasses {
bool complete()
{
if (mHasGetters || mHasSetters) {
if (mHasGetters) mGetters = new JSFunction*[mSlotCount];
if (mHasSetters) mSetters = new JSFunction*[mSlotCount];
if (mHasGetters) mGetters.resize(mSlotCount);
if (mHasSetters) mSetters.resize(mSlotCount);
JSSlots::iterator end = mSlots.end();
for (JSSlots::iterator i = mSlots.begin(); i != end; i++) {
if (mHasGetters) mGetters[i->second.mIndex] = i->second.mGetter;