- changed title to revid: simplify config.Config field types
- edited description
revid: simplify config.Config field types
Issue #248
resolved
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)
-
reporter -
Saxon
-
reporter - changed status to resolved
We are going to approach this differently.
- Log in to comment