[MERGE] Change FileNames allocation strategy

[Robert Collins]

Robert Collins has voted approve.
Status is now: Resubmit
Comment:
Oh, I didn't mean to leave the keyword there. So this should be
+       def allocate(self):
+               """Allocate name in the names collection.
+
+               :param name: The name to allocate.
+               :raises: bzrlib.errors.DuplicateKey if the name is 
already allocated.
+               """
+               if len(self._names) >= self._cap:
+                       raise errors.BzrError('too many files')
+               if name in self._names:
+                       raise errors.DuplicateKey(name)
+               self._names.add(name)
+               return name


For details, see: 
http://bundlebuggy.aaronbentley.com/request/%3C1186444328.23776.74.camel%40localhost.localdomain%3E