Right you are. I will take care of that later this morning.
On Dec 27, 2013 8:59 AM, "Tomaz Muraus" <notifications@github.com> wrote:

> In libcloud/compute/drivers/ec2.py:
>
> >          """
> >          params = {'Action': 'DisassociateAddress'}
> >
> > -        params.update({'PublicIp': elastic_ip_address})
> > +        if elastic_ip_address:
>
> You should also throw here if both arguments are specified, right?
>
> Please also add a test case for this.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/apache/libcloud/pull/208/files#r8567553>
> .
>