changelog: removed bad default arguments in add method

The arguments defaulted to None, but

- user cannot be None since it is immediately stripped.

- p1 and p2 cannot be None since they are passed directly to
  revlog.addrevision, where they are mandatory.

mercurial/changelog.py

@@ -198,9 +198,8 b' class changelog(revlog.revlog):'
         files = l[3:]
         return (manifest, user, (time, timezone), files, desc, extra)
 
-    def add(self, manifest, files, desc, transaction, p1=None, p2=None,
-                  user=None, date=None, extra={}):
-
+    def add(self, manifest, files, desc, transaction, p1, p2,
+                  user, date=None, extra={}):
         user = user.strip()
         if "\n" in user:
             raise error.RevlogError(_("username %s contains a newline")