Minor XXX fixups

1 files changed, 5 insertions, 1 deletions

diff --git a/src/declarative/qml/v8/qv8contextwrapper_p.h b/src/declarative/qml/v8/qv8contextwrapper_p.h
index 3149569285..12e01bab76 100644
--- a/src/declarative/qml/v8/qv8contextwrapper_p.h
+++ b/src/declarative/qml/v8/qv8contextwrapper_p.h
@@ -79,7 +79,11 @@ public:
     void addSubContext(v8::Handle<v8::Object> qmlglobal, v8::Handle<v8::Script>, 
                        QDeclarativeContextData *ctxt);
 
-    // XXX aakenned - remove this abomination
+    // XXX We only use the secondary scope to pass the "arguments" of the signal to
+    // on<SignalName> properties.  Instead of doing this we should rewrite the 
+    // JavaScript closure function to accept these arguments as named parameters.
+    // To keep backwards compatibility we have to check that the argument names are
+    // not members of the QV8Engine::illegalNames() set.
     QObject *setSecondaryScope(v8::Handle<v8::Object>, QObject *);
 
     QDeclarativeContextData *callingContext();