[exim-cvs] cvs commit: exim/exim-src/src deliver.c

Startseite
Nachricht löschen
Nachricht beantworten
Autor: Philip Hazel
Datum:  
To: exim-cvs
Betreff: [exim-cvs] cvs commit: exim/exim-src/src deliver.c
ph10 2006/06/30 15:14:46 BST

  Modified files:
    exim-src/src         deliver.c 
  Log:
  Remove commented-out old de-duplication code. It's been over a year now.


  Revision  Changes    Path
  1.33      +1 -41     exim/exim-src/src/deliver.c


  Index: deliver.c
  ===================================================================
  RCS file: /home/cvs/exim/exim-src/src/deliver.c,v
  retrieving revision 1.32
  retrieving revision 1.33
  diff -u -r1.32 -r1.33
  --- deliver.c    27 Jun 2006 15:38:07 -0000    1.32
  +++ deliver.c    30 Jun 2006 14:14:46 -0000    1.33
  @@ -1,4 +1,4 @@
  -/* $Cambridge: exim/exim-src/src/deliver.c,v 1.32 2006/06/27 15:38:07 ph10 Exp $ */
  +/* $Cambridge: exim/exim-src/src/deliver.c,v 1.33 2006/06/30 14:14:46 ph10 Exp $ */


   /*************************************************
   *     Exim - an Internet mail transport agent    *
  @@ -4372,10 +4372,7 @@
   /* This function was introduced when the test for duplicate addresses that are
   not pipes, files, or autoreplies was moved from the middle of routing to when
   routing was complete. That was to fix obscure cases when the routing history
  -affects the subsequent routing of identical addresses. If that change has to be
  -reversed, this function is no longer needed. For a while, the old code that was
  -affected by this change is commented with !!!OLD-DE-DUP!!! so it can be found
  -easily.
  +affects the subsequent routing of identical addresses.


   This function is called after routing, to check that the final routed addresses
   are not duplicates. If we detect a duplicate, we remember what it is a
  @@ -5450,37 +5447,6 @@
         continue;
         }


  -
  -    /* !!!OLD-DE-DUP!!!  We used to test for duplicates at this point, in order
  -    to save effort on routing duplicate addresses. However, facilities have
  -    been added to Exim so that now two identical addresses that are children of
  -    other addresses may be routed differently as a result of their previous
  -    routing history. For example, different redirect routers may have given
  -    them different redirect_router values, but there are other cases too.
  -    Therefore, tests for duplicates now take place when routing is complete.
  -    This is the old code, kept for a while for the record, and in case this
  -    radical change has to be backed out for some reason. */
  -
  -    #ifdef NEVER
  -    /* If it's a duplicate, remember what it's a duplicate of */
  -
  -    if ((tnode = tree_search(tree_duplicates, addr->unique)) != NULL)
  -      {
  -      DEBUG(D_deliver|D_route)
  -        debug_printf("%s is a duplicate address: discarded\n", addr->unique);
  -      addr->dupof = tnode->data.ptr;
  -      addr->next = addr_duplicate;
  -      addr_duplicate = addr;
  -      continue;
  -      }
  -
  -    /* Record this address, so subsequent duplicates get picked up. */
  -
  -    tree_add_duplicate(addr->unique, addr);
  -    #endif
  -
  -
  -
       /* Get the routing retry status, saving the two retry keys (with and
       without the local part) for subsequent use. Ignore retry records that
       are too old. */
  @@ -5792,11 +5758,6 @@
   local_user_gid = (gid_t)(-1);
   local_user_uid = (uid_t)(-1);


-
-/* !!!OLD-DE-DUP!!! The next two statement were introduced when checking for
-duplicates was moved from within routing to afterwards. If that change has to
-be backed out, they should be removed. */
-
/* Check for any duplicate addresses. This check is delayed until after
routing, because the flexibility of the routing configuration means that
identical addresses with different parentage may end up being redirected to
@@ -5805,7 +5766,6 @@

do_duplicate_check(&addr_local);
do_duplicate_check(&addr_remote);
-

/* When acting as an MUA wrapper, we proceed only if all addresses route to a
remote transport. The check that they all end up in one transaction happens in