Agreed, it's a mistake on my part. Feel free to correct it :). Sorry I haven't had time to do another review.

Sent from my phone

> On Jun 3, 2014, at 7:04 PM, Gwildor Sok <notifications@github.com> wrote:
> 
> In post_office/mail.py:
> 
> > @@ -38,7 +38,7 @@ def parse_priority(priority):
> >      return priority
> >  
> >  
> > -def create(sender, recipient, subject='', message='', html_message='',
> > +def create(sender, recipients=[], cc=[], bcc=[], subject='', message='', html_message='',
> I see this on other places on your changes as well, but it is generally not a good idea to have parameters of a function default to a mutable such as a list. When the value is changed in the content of the function later on, later calls will have the new value by default as well, because the reference to the value stays the same.
> 
> Example:
> 
> >>> def a(b=[]):
> ...     b.append(1)
> ...     print b
> ...
> >>> a()
> [1]
> >>> a()
> [1, 1]
> Best and accepted way to circumvent this, is by doing something like this:
> 
> def foo(bar=None):
>     if bar is None:
>         bar = []
> 
>     # The rest here
> —
> Reply to this email directly or view it on GitHub.