Protocol:

    Don't send new-membership when a membership is changed.

Documentation:
    Document protocol change.

Test suite:
    Do not expect new-membership when membership is changed.

ChangeLog

+1999-06-26  David Byers  <davby@ida.liu.se>
+
+	Send new-membership only for new memberships.
+	* doc/Protocol-A.texi (async-new-membership): Remove
+	documentation that says this message is sent when a membership is
+	reprioritized.
+
+	* src/server/testsuite/lyskomd.0/06.exp (shutdown_06): Don't
+	expect new-membership when membership is changed.
+
+	* src/server/membership.c (add_member_common): Only send
+	new-membership for new memberships, not changed memberships.
+
 1999-06-25  Per Cederqvist  <ceder@lysator.liu.se>
 
 	* Release 2.0.0.

doc/Protocol-A.texi

@@ -2,7 +2,7 @@
 @c
 @c FIXME: Explain how the garb works with nice and keep-commented
 @c
-@c $Id: Protocol-A.texi,v 1.79 1999/06/25 21:40:55 ceder Exp $
+@c $Id: Protocol-A.texi,v 1.80 1999/06/26 08:52:32 byers Exp $
 @c %**start of header
 @setfilename protocol-a.info
 @settitle LysKOM Protocol A
@@ -7964,9 +7964,8 @@ the recipient.
 @end example
 
 This message indicates that the membership for @code{pers-no} in
-conference @code{conf-no} has been added or reprioritized. This message
-is currently sent only to @code{pers-no}, but that may change in the
-future.
+conference @code{conf-no} has been added. This message is currently sent
+only to @code{pers-no}, but that may change in the future.
 
 See also @pxref{async-leave-conf}.
 

src/server/membership.c

 /*
- * $Id: membership.c,v 0.49 1999/06/24 12:58:01 ceder Exp $
+ * $Id: membership.c,v 0.50 1999/06/26 08:53:13 byers Exp $
  * Copyright (C) 1991-1999  Lysator Academic Computer Association.
  *
  * This file is part of the LysKOM server.
@@ -38,7 +38,7 @@
 #define DEBUG_MARK_AS_READ
 
 static const char *
-rcsid = "$Id: membership.c,v 0.49 1999/06/24 12:58:01 ceder Exp $";
+rcsid = "$Id: membership.c,v 0.50 1999/06/26 08:53:13 byers Exp $";
 #include "rcs.h"
 USE(rcsid);
 
@@ -1032,9 +1032,9 @@ add_member_common(Conf_no              conf_no,
 	do_add_member(conf_no, conf_c, pers_no, pers_p, ACTPERS,
                       priority, where, type, fake_passive);
 
+        send_async_new_membership(pers_no, conf_no);
     }
 
-    send_async_new_membership(pers_no, conf_no);
     return OK;
 }
 

src/server/testsuite/lyskomd.0/06.exp

@@ -472,9 +472,8 @@ simple_expect "=1051"
 talk_to client 1
 kom_ping_server
 
-# Check that async message is sent to person
+# Check that async message is not sent to person
 talk_to client 0
-simple_expect ":2 18 8 9"
 kom_ping_server
 
 # Check that accept-async can block the message