# HG changeset patch
# User Dan Villiom Podlaski Christiansen <danchr@gmail.com>
# Date 1241369489 -7200
# Node ID 15b8bab035041265c580ec965bc4abe382f1a47a
# Parent  9ad5cf45e30c60d352fc7ef17d0e5ca244d9f2b8
Change default author substitution to avoid updating test hashes.

Rather than '%s <%s%s>', just use '%s%s'. The two should be
functionally equivalent. All tests pass.

Added a comment about the handling of missing authors. This is tested
by test_push_to_default in tests/test_push_command.py; it is treated
the same as before.

diff --git a/maps.py b/maps.py
--- a/maps.py
+++ b/maps.py
@@ -70,8 +70,8 @@ class AuthorMap(dict):
         if author in self:
             result = self.super.__getitem__(author)
         elif self.ui.configbool('hgsubversion', 'defaultauthors', True):
-            self[author] = result = \
-                '%s <%s%s>' % (author, author, self.defaulthost)
+            # TODO: should we treat missing authors specially?
+            self[author] = result = '%s%s' % (author, self.defaulthost)
             self.ui.warn('Substituting author "%s" for default "%s"\n'
                          % (author, result))
         else: