Swap destination and source in NIB binding connector.

git-svn-id: svn+ssh://svn.gna.org/svn/gnustep/libs/gui/trunk@25727 72102866-910b-0410-8b05-ffd578937521

ChangeLog

@@ -1,3 +1,9 @@
+2007-12-12 Fred Kiefer <FredKiefer@gmx.de>
+
+	* Source/NSNibBindingConnector.m (-establishConnection): Swap
+	destination and source.
+	* Source/NSKeyValueBinding.m: Started rework.
+
 2007-12-12 Fred Kiefer <FredKiefer@gmx.de>
 
 	* Source/NSKeyValueBinding.m (GSBindingUnbindAll): Return if table

Source/NSKeyValueBinding.m

@@ -167,7 +167,9 @@ static inline void setup()
   id observedObject;
   NSString *keyPath;
 
-  setup();
+  if (!objectTable)
+    return;
+
   [bindingLock lock];
   bindings = (NSMutableDictionary *)NSMapGet(objectTable, (void *)self);
   if (bindings != nil)
@@ -184,6 +186,10 @@ static inline void setup()
   [bindingLock unlock];
 }
 
+// FIXME: This method should not be defined on this class, as it make all
+// other value observation impossible. Better add a new GSBinding class
+// to handle this. Perhaps with plenty of specific subclasses for the 
+// different special cases?
 - (void) observeValueForKeyPath: (NSString *)keyPath
                        ofObject: (id)object
                          change: (NSDictionary *)change

Source/NSNibBindingConnector.m

@@ -94,8 +94,8 @@
 
 - (void) establishConnection
 {
-  [_dst bind: _binding 
-        toObject: _src 
+  [_src bind: _binding 
+        toObject: _dst
         withKeyPath: _keyPath
         options: _options];
 }