Copy all Strings received from agentcore instead of using raw pointer (#126)

sjanuary · mattcolegate · commit 463c5235f78f · 2017-05-22T12:27:25.000+01:00
* Add debug to diagnose test crash on Travis

* Try copying bad string

* remove debug
diff --git a/Sources/SwiftMetrics/SwiftMetrics.swift b/Sources/SwiftMetrics/SwiftMetrics.swift
@@ -66,7 +66,10 @@ private func receiveAgentCoreData(cSourceId: UnsafePointer<CChar>, cSize: CUnsig
   }
   let source = String(cString: cSourceId)
   if source != "api" {
-    let message = String(bytesNoCopy: data, length: size, encoding: String.Encoding.utf8, freeWhenDone: false) ?? ""
+    let opaquePointer = OpaquePointer(data)
+    let cstrPointer = UnsafePointer<CChar>(opaquePointer)
+
+    let message = String(cString:cstrPointer) ?? ""
     if swiftMon != nil {
       swiftMon!.raiseCoreEvent(topic: source, message: message)
     }

Original file line number	Diff line number	Diff line change
`@@ -66,7 +66,10 @@ private func receiveAgentCoreData(cSourceId: UnsafePointer<CChar>, cSize: CUnsig`
`66`	`66`	`}`
`67`	`67`	`let source = String(cString: cSourceId)`
`68`	`68`	`if source != "api" {`
`69`		`- let message = String(bytesNoCopy: data, length: size, encoding: String.Encoding.utf8, freeWhenDone: false) ?? ""`
	`69`	`+ let opaquePointer = OpaquePointer(data)`
	`70`	`+ let cstrPointer = UnsafePointer<CChar>(opaquePointer)`
	`71`	`+`
	`72`	`+ let message = String(cString:cstrPointer) ?? ""`
`70`	`73`	`if swiftMon != nil {`
`71`	`74`	`swiftMon!.raiseCoreEvent(topic: source, message: message)`
`72`	`75`	`}`