Merge lp:~rvb/gwacl/missing-err-check into lp:gwacl

Proposed by Raphaël Badin on 2013-07-17
Status: Merged
Approved by: Raphaël Badin on 2013-07-17
Approved revision: 187
Merged at revision: 188
Proposed branch: lp:~rvb/gwacl/missing-err-check
Merge into: lp:gwacl
Diff against target: 12 lines (+2/-1)
1 file modified
example/management/run.go (+2/-1)
To merge this branch: bzr merge lp:~rvb/gwacl/missing-err-check
Reviewer Review Type Date Requested Status
Julian Edwards (community) 2013-07-17 Approve on 2013-07-17
Review via email: mp+175265@code.launchpad.net

Commit message

Check error return in example script.

To post a comment you must log in.
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'example/management/run.go'
2--- example/management/run.go 2013-07-16 10:21:07 +0000
3+++ example/management/run.go 2013-07-17 12:16:28 +0000
4@@ -121,7 +121,8 @@
5
6 defer func() {
7 fmt.Printf("Deleting virtual network %s...\n", virtualNetworkName)
8- api.RemoveVirtualNetworkSite(virtualNetworkName)
9+ err := api.RemoveVirtualNetworkSite(virtualNetworkName)
10+ checkError(err)
11 fmt.Printf("Done deleting virtual network %s\n", virtualNetworkName)
12 }()
13

Subscribers

People subscribed via source and target branches

to all changes: