Merge lp:~julian-edwards/gwacl/remove-spurious-err into lp:gwacl

Proposed by Julian Edwards
Status: Merged
Approved by: Julian Edwards
Approved revision: 195
Merged at revision: 195
Proposed branch: lp:~julian-edwards/gwacl/remove-spurious-err
Merge into: lp:gwacl
Diff against target: 16 lines (+0/-4)
1 file modified
management_base.go (+0/-4)
To merge this branch: bzr merge lp:~julian-edwards/gwacl/remove-spurious-err
Reviewer Review Type Date Requested Status
Julian Edwards (community) Approve
Review via email: mp+176103@code.launchpad.net

Commit message

Remove a spurious error check in DeleteAffinityGroup

To post a comment you must log in.
Revision history for this message
Julian Edwards (julian-edwards) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'management_base.go'
2--- management_base.go 2013-07-19 13:18:56 +0000
3+++ management_base.go 2013-07-22 02:46:25 +0000
4@@ -483,12 +483,8 @@
5 // DeleteAffinityGroup requests a deletion of the named affinity group.
6 // See http://msdn.microsoft.com/en-us/library/windowsazure/gg715314.aspx
7 func (api *ManagementAPI) DeleteAffinityGroup(request *DeleteAffinityGroupRequest) error {
8- var err error
9 checkPathComponents(request.Name)
10 url := "affinitygroups/" + request.Name
11- if err != nil {
12- return err
13- }
14 response, err := api.session.delete(url, "2011-02-25")
15 if err != nil {
16 if IsNotFoundError(err) {

Subscribers

People subscribed via source and target branches

to all changes: