Code review comment for lp:~free.ekanayaka/landscape-client/method-call-defer

Revision history for this message
Free Ekanayaka (free.ekanayaka) wrote :

Thanks Duncan! It was awesome to get your Landscape "goodbye review" :)

[1]

Yay, thanks buildbot.

[2]

The thing is that this is a very general method call, so in principle any exception could be raised. However whatever gets raised is wrapped around a MethodCallError, so the remote caller can possibly cope with it.

[3]

Great idea! That indeed simplified the logic quite a bit. Thanks.

« Back to merge proposal