state: RelationEndpoint => Endpoint

state: RelationEndpoint => Endpoint

...also added an ImplementedBy  method, for determining whether a given
charm offers a compatible endpoint, and tests which had apparently somehow
not been written.

https://code.launchpad.net/~fwereade/juju-core/state-relation-endpoint/+merge/130594

(do not edit description out of merge proposal)

[fwereade, 2012-10-19 16:08:56]

Please take a look.

[niemeyer, 2012-10-25 08:23:07]

LGTM assuming a better name for Match.

https://codereview.appspot.com/6734049/diff/1/state/endpoint.go
File state/endpoint.go (right):

https://codereview.appspot.com/6734049/diff/1/state/endpoint.go#newcode61
state/endpoint.go:61: // Match returns whether the endpoint matches the supplied
charm.
endpoint.ImplementedBy(charm) sounds about right. WDYT?

[fwereade, 2012-10-25 09:00:46]

*** Submitted:

state: RelationEndpoint => Endpoint

...also added an ImplementedBy  method, for determining whether a given
charm offers a compatible endpoint, and tests which had apparently somehow
not been written.

R=niemeyer
CC=
https://codereview.appspot.com/6734049

[fwereade, 2012-10-25 09:01:08]

On 2012/10/25 08:23:07, niemeyer wrote:
> LGTM assuming a better name for Match.
> 
> https://codereview.appspot.com/6734049/diff/1/state/endpoint.go
> File state/endpoint.go (right):
> 
> https://codereview.appspot.com/6734049/diff/1/state/endpoint.go#newcode61
> state/endpoint.go:61: // Match returns whether the endpoint matches the
supplied
> charm.
> endpoint.ImplementedBy(charm) sounds about right. WDYT?

Sounds perfect, thanks :).