Clean up inplace binary operations for subscripts.

Remove a lot of duplicated code, and make it a lot
easier to implement JS compliant semantics.

Change-Id: Ic5ecb58d34a5df8fe05e86c5f906d70c50c51b54
Reviewed-by: Erik Verbruggen <erik.verbruggen@digia.com>

1 files changed, 3 insertions, 0 deletions

diff --git a/qmljs_objects.h b/qmljs_objects.h
index ebb2291586..6fae34d79f 100644
--- a/qmljs_objects.h
+++ b/qmljs_objects.h
@@ -413,6 +413,7 @@ struct Object {
 
     Value getValue(Context *ctx, PropertyDescriptor *p) const;
     bool inplaceBinOp(Value rhs, Context *ctx, String *name, BinOp op);
+    virtual bool inplaceBinOp(Value rhs, Value index, BinOp op, Context *ctx);
 };
 
 struct ForEachIteratorObject: Object {
@@ -460,6 +461,8 @@ struct ArrayObject: Object {
     virtual QString className() { return QStringLiteral("Array"); }
     virtual ArrayObject *asArrayObject() { return this; }
     virtual Value __get__(Context *ctx, String *name);
+
+    virtual bool inplaceBinOp(Value rhs, Value index, BinOp op, Context *ctx);
 };
 
 struct FunctionObject: Object {