From a8081b52b298359ae8d50ae5ee2572f8af6a1faf Mon Sep 17 00:00:00 2001
From: Saxon <saxon@ausocean.org>
Date: Mon, 26 Aug 2019 13:43:45 +0930
Subject: [PATCH] revid/config.go: MinPeriod => MinFrames

---
 revid/config.go | 20 ++++++++++----------
 revid/revid.go  |  8 ++++----
 2 files changed, 14 insertions(+), 14 deletions(-)

diff --git a/revid/config.go b/revid/config.go
index 7807b52c..87fb9139 100644
--- a/revid/config.go
+++ b/revid/config.go
@@ -109,7 +109,7 @@ const (
 	defaultRotation         = 0 // Degrees
 	defaultWidth            = 1280
 	defaultHeight           = 720
-	defaultMinPeriod        = 100
+	defaultMinFrames        = 100
 	defaultClipDuration     = 0
 	defaultQuantization     = 30
 	defaultBitrate          = 400000
@@ -204,14 +204,14 @@ type Config struct {
 	// are using Raspivid input.
 	Quantization uint
 
-	// MinPeriod defines the frequency of key NAL units SPS, PPS and IDR in
+	// MinFrames defines the frequency of key NAL units SPS, PPS and IDR in
 	// number of NAL units. This will also determine the frequency of PSI if the
-	// output container is MPEG-TS. If ClipDuration is less than MinPeriod,
-	// ClipDuration will default to MinPeriod.
-	MinPeriod uint
+	// output container is MPEG-TS. If ClipDuration is less than MinFrames,
+	// ClipDuration will default to MinFrames.
+	MinFrames uint
 
 	// ClipDuration is the duration of MTS data that is sent using HTTP or RTP
-	// output. This defaults to 0, therefore MinPeriod will determine the length of
+	// output. This defaults to 0, therefore MinFrames will determine the length of
 	// clips by default.
 	ClipDuration time.Duration
 
@@ -392,10 +392,10 @@ func (c *Config) Validate() error {
 		return errors.New("invalid bitrate")
 	}
 
-	if c.MinPeriod == 0 {
-		c.Logger.Log(logger.Info, pkg+"no min period defined, defaulting", "MinPeriod", defaultMinPeriod)
-		c.MinPeriod = defaultMinPeriod
-	} else if c.MinPeriod < 0 {
+	if c.MinFrames == 0 {
+		c.Logger.Log(logger.Info, pkg+"no min period defined, defaulting", "MinFrames", defaultMinFrames)
+		c.MinFrames = defaultMinFrames
+	} else if c.MinFrames < 0 {
 		return errors.New("refresh period is less than 0")
 	}
 
diff --git a/revid/revid.go b/revid/revid.go
index 13efdd7b..fa1156b1 100644
--- a/revid/revid.go
+++ b/revid/revid.go
@@ -456,13 +456,13 @@ func (r *Revid) Update(vars map[string]string) error {
 				break
 			}
 			r.config.Quantization = uint(v)
-		case "MinPeriod":
+		case "MinFrames":
 			v, err := strconv.Atoi(value)
 			if err != nil {
-				r.config.Logger.Log(logger.Warning, pkg+"invalid MinPeriod param", "value", value)
+				r.config.Logger.Log(logger.Warning, pkg+"invalid MinFrames param", "value", value)
 				break
 			}
-			r.config.MinPeriod = uint(v)
+			r.config.MinFrames = uint(v)
 
 		case "ClipDuration":
 			v, err := strconv.Atoi(value)
@@ -585,7 +585,7 @@ func (r *Revid) startRaspivid() (func() error, error) {
 		args = append(args,
 			"--codec", "H264",
 			"--inline",
-			"--intra", fmt.Sprint(r.config.MinPeriod),
+			"--intra", fmt.Sprint(r.config.MinFrames),
 		)
 		if r.config.Quantization != 0 {
 			args = append(args, "-qp", fmt.Sprint(r.config.Quantization))