Merged in define_MaxMin_frames (pull request #291)

revid/config.go: define max value for MinFrames

Approved-by: Saxon Milton <saxon.milton@gmail.com>
This commit is contained in:
Frank Cui 2019-12-03 06:00:01 +00:00 committed by Saxon Milton
commit 9f077b3799
1 changed files with 6 additions and 2 deletions

View File

@ -327,11 +327,15 @@ func (c *Config) Validate() error {
c.BurstPeriod = defaultBurstPeriod
}
if c.MinFrames == 0 {
const maxMinFrames = 1000
switch {
case c.MinFrames == 0:
c.Logger.Log(logger.Info, pkg+"no min period defined, defaulting", "MinFrames", defaultMinFrames)
c.MinFrames = defaultMinFrames
} else if c.MinFrames < 0 {
case c.MinFrames < 0:
return errors.New("refresh period is less than 0")
case c.MinFrames > maxMinFrames:
return errors.New("refresh period is greater than 1000")
}
if c.FrameRate <= 0 {