Merge pull request #1203 from weaveworks/reportcard

Fix a couple of minor issue for goreportcard and add badge for it.

README.md

@@ -1,6 +1,6 @@
 # Scope
 
-[![Circle CI](https://circleci.com/gh/weaveworks/scope/tree/master.svg?style=shield)](https://circleci.com/gh/weaveworks/scope/tree/master) [![Coverage Status](https://coveralls.io/repos/weaveworks/scope/badge.svg)](https://coveralls.io/r/weaveworks/scope) [![Slack Status](https://weave-scope-slack.herokuapp.com/badge.svg)](https://weave-scope-slack.herokuapp.com)
+[![Circle CI](https://circleci.com/gh/weaveworks/scope/tree/master.svg?style=shield)](https://circleci.com/gh/weaveworks/scope/tree/master) [![Coverage Status](https://coveralls.io/repos/weaveworks/scope/badge.svg)](https://coveralls.io/r/weaveworks/scope) [![Slack Status](https://weave-scope-slack.herokuapp.com/badge.svg)](https://weave-scope-slack.herokuapp.com) [![Go Report Card](https://goreportcard.com/badge/github.com/weaveworks/scope)](https://goreportcard.com/report/github.com/weaveworks/scope)
 
 Weave Scope automatically generates a map of your application, enabling you to
 intuitively understand, monitor, and control your containerized, microservices based application.

experimental/sniff/testclient/main.go

@@ -62,6 +62,10 @@ func main() {
 			}
 			log.Println(ci.Timestamp.String())
 			err = parser.DecodeLayers(data, &decoded)
+			if err != nil {
+				log.Printf("Error in DecodeLayers: %v", err)
+				continue
+			}
 			for _, t := range decoded {
 				switch t {
 				case layers.LayerTypeEthernet:

experimental/tracer/ptrace/ptracer.go

@@ -76,15 +76,16 @@ func (t *PTracer) Stop() {
 }
 
 // TraceProcess starts tracing the given pid
-func (t *PTracer) TraceProcess(pid int) *process {
-	result := make(chan *process)
+func (t *PTracer) TraceProcess(pid int) {
+	result := make(chan struct{})
 	t.ops <- func() {
 		process := newProcess(pid, t)
 		t.processes[pid] = process
 		process.trace()
-		result <- process
+		result <- struct{}{}
 	}
-	return <-result
+	<-result
+	return
 }
 
 // StopTracing stops tracing all threads for the given pid