Finished commenting, now I can think about only testing

revid/RevidInstance.go

@@ -348,21 +348,21 @@ func (r *revidInst)readCamera() {
 	r.Log(Info, "Out of reading routine!")
 }
 
-// Stop
+// Stop halts any processing of video
 func (r *revidInst) Stop() {
 	if r.isRunning {
 		r.Log(Info, "Stopping revid!")
 		r.isRunning = false
+		if r.cmd != nil {
 			r.cmd.Process.Kill()
+		}
 	} else {
 		r.Log(Warning, "revidInst.Stop() called but revid not running!")
 	}
 }
 
-/*
+// packClips takes data segments; whether that be tsPackets or jpeg frames and
-New returns a pointer to a newly allocated RingBuffer with the parameters specified.
+// packs them into clips 1s long.
-It initialises fields and allocates the required dataMemory.
-*/
 func (r *revidInst) packClips() {
 	clipSize := 0
 	packetCount := 0
@@ -421,10 +421,8 @@ func (r *revidInst) packClips() {
 	}
 }
 
-/*
+// outputClips takes the clips produced in the packClips method and outputs them
-New returns a pointer to a newly allocated RingBuffer with the parameters specified.
+// to the desired output defined in the revidInst config
-It initialises fields and allocates the required dataMemory.
-*/
 func (r *revidInst) outputClips() {
 	now := time.Now()
 	prevTime := now
@@ -469,10 +467,7 @@ func (r *revidInst) outputClips() {
 	}
 }
 
-/*
+// sendClipToHTTP takes a clip and an output url and posts through http. 
-New returns a pointer to a newly allocated RingBuffer with the parameters specified.
-It initialises fields and allocates the required dataMemory.
-*/
 func (r *revidInst) sendClipToHTTP(clip []byte, output string) error {
 	timeout := time.Duration(httpTimeOut * time.Second)
 	client := http.Client{