nervousnet / nervousnet-HUB-iOS

Nervousnet HUB client for iOS
GNU General Public License v3.0
2 stars 0 forks source link

in 'SensorReading' just point to 'GeneralSensorConfig' #11

Closed a-enz closed 7 years ago

a-enz commented 7 years ago

SensorReading shares data with GeneralSensorConfig which needs to be copied back and forth, it would be nicer to just point to the Config object instead which makes creation of SensorReadings cleaner and shorter

e.g: better: var reading = SensorReading(config) current: var reading = SensorReading(sensorID: config.sensorID, sensorName: config.sensorName, parameterNames: config.parameterNames)

a-enz commented 7 years ago

I implemented this