revid: simplify config.Config field types

Issue #248 resolved
Saxon Milton created an issue

This is in preparation for changes to be made to achieve issue #241.

Firstly, considering that a lot of configuration is done using constant string literals, we may as well have the enum fields in the Config struct e.g. Input (can be File, Raspivid, Webcam etc) just be string literals instead of uint8 consts. This will reduce the need for conversion from the map[string]string we get from the cloud, to the config.Config struct. As well as this, our enum consts for Inputs and Outputs etc should also just be string literal values, then we can easily compare to Config fields in logic.

Secondly, integer fields in the Config struct should all be int.

Comments (3)

  1. Log in to comment