http://bugs.webkit.org/show_bug.cgi?id=15800
REGRESSION (r27303): RegExp leaks
* kjs/regexp_object.h:
(KJS::RegExpImp::setRegExp):
(KJS::RegExpImp::regExp):
(KJS::RegExpImp::classInfo):
* kjs/regexp_object.cpp:
(RegExpImp::RegExpImp):
(RegExpImp::~RegExpImp):
Renamed reg member variable to m_regExp, changed it to use OwnPtr.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@27393
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
+2007-11-03 Alexey Proskuryakov <ap@webkit.org>
+
+ Reviewed by Darin.
+
+ http://bugs.webkit.org/show_bug.cgi?id=15800
+ REGRESSION (r27303): RegExp leaks
+
+ * kjs/regexp_object.h:
+ (KJS::RegExpImp::setRegExp):
+ (KJS::RegExpImp::regExp):
+ (KJS::RegExpImp::classInfo):
+ * kjs/regexp_object.cpp:
+ (RegExpImp::RegExpImp):
+ (RegExpImp::~RegExpImp):
+ Renamed reg member variable to m_regExp, changed it to use OwnPtr.
+
2007-11-02 Maciej Stachowiak <mjs@apple.com>
Reviewed by Oliver.
const ClassInfo RegExpImp::info = {"RegExp", 0, 0, 0};
-RegExpImp::RegExpImp(RegExpPrototype *regexpProto)
- : JSObject(regexpProto), reg(0L)
+RegExpImp::RegExpImp(RegExpPrototype* regexpProto)
+ : JSObject(regexpProto)
{
}
RegExpImp::~RegExpImp()
{
- delete reg;
}
// ------------------------------ RegExpObjectImp ------------------------------
class RegExpImp : public JSObject {
public:
- RegExpImp(RegExpPrototype *regexpProto);
- ~RegExpImp();
- void setRegExp(RegExp *r) { reg = r; }
- RegExp* regExp() const { return reg; }
+ RegExpImp(RegExpPrototype* regexpProto);
+ virtual ~RegExpImp();
+ void setRegExp(RegExp* r) { m_regExp.set(r); }
+ RegExp* regExp() const { return m_regExp.get(); }
- virtual const ClassInfo *classInfo() const { return &info; }
+ virtual const ClassInfo* classInfo() const { return &info; }
static const ClassInfo info;
private:
- RegExp *reg;
+ OwnPtr<RegExp> m_regExp;
};
struct RegExpObjectImpPrivate;