Richard Yen dba at richyen.com
Mon Jan 25 15:29:57 PST 2010
Hello,

While parsing through remote_worker.c, I found that there's an "if"  
statement where it seems possible that it should be an "else if"  
statement.

My patch:
Index: slon/remote_worker.c
===================================================================
--- slon/remote_worker.c	(revision 806)
+++ slon/remote_worker.c	(working copy)
@@ -825,7 +825,7 @@

  				need_reloadListen = true;
  			}
-			if (strcmp(event->ev_type, "CLONE_NODE") == 0)
+			else if (strcmp(event->ev_type, "CLONE_NODE") == 0)
  			{
  				int			no_id = (int) strtol(event->ev_data1, NULL, 10);
  				int			no_provider = (int) strtol(event->ev_data2, NULL, 10);

Could someone verify that the "else" actually belongs there?  If so,  
could you please apply the patch (I don't have commit access to the  
repository)?

Thanks!
--Richard


More information about the Slony1-general mailing list