Ubuntu Push Notifications

Merge lp:~pedronis/ubuntu-push/acceptance-panicked into lp:ubuntu-push

  1. acceptance-panicked
  2. Merge into trunk
Proposed by Samuele Pedroni
Status: Merged
Approved by: Samuele Pedroni
Approved revision: 77
Merged at revision: 77
Proposed branch: lp:~pedronis/ubuntu-push/acceptance-panicked
Merge into: lp:ubuntu-push
Diff against target: 24 lines (+4/-4)
1 file modified
server/acceptance/suites/helpers.go (+4/-4)
To merge this branch: bzr merge lp:~pedronis/ubuntu-push/acceptance-panicked
Reviewer Review Type Date Requested Status
Samuele Pedroni Approve
Review via email: mp+209097@code.launchpad.net

Commit message

[trivial] variable spelling

Description of the change

trivial, variable spelling

To post a comment you must log in.
Revision history for this message
Samuele Pedroni (pedronis) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'server/acceptance/suites/helpers.go'
2--- server/acceptance/suites/helpers.go 2014-02-26 19:50:35 +0000
3+++ server/acceptance/suites/helpers.go 2014-03-03 16:56:29 +0000
4@@ -111,16 +111,16 @@
5 }
6 logs := make(chan string, 10)
7 go func() {
8- paniced := false
9+ panicked := false
10 for {
11- info, err := getLineInfo(paniced)
12+ info, err := getLineInfo(panicked)
13 if err != nil {
14 logs <- fmt.Sprintf("%s capture: %v", cmdName, err)
15 close(logs)
16 return
17 }
18- if paniced || strings.HasPrefix(info, "ERROR(PANIC") {
19- paniced = true
20+ if panicked || strings.HasPrefix(info, "ERROR(PANIC") {
21+ panicked = true
22 c.Log(info)
23 continue
24 }

Subscribers

People subscribed via source and target branches