Revert back to reflect.Ptr

api/beacon/beacon.go

@@ -58,24 +58,22 @@ func (b *Beacon) WatchDeviceChanges(ctx context.Context) (chan bool, error) {
 	ch := make(chan bool)
 
 	go func() {
+		defer close(ch)
 		for {
 			select {
 			case changed := <-b.propchanged:
 
 				if changed == nil {
-					ctx.Done()
 					return
 				}
 
 				if changed.Name == "ManufacturerData" || changed.Name == "ServiceData" {
 					ch <- b.Parse()
 				}
 
-				break
 			case <-ctx.Done():
 				b.propchanged <- nil
-				close(ch)
-				break
+				return
 			}
 		}
 	}()

util/map_struct.go

@@ -90,11 +90,11 @@ func mapStructField(obj interface{}, name string, value dbus.Variant) error {
 	if val.Type().Kind() == reflect.Map {
 
 		if structFieldType.Kind() == reflect.Struct ||
-			(structFieldType.Kind() == reflect.Pointer && structFieldType.Elem().Kind() == reflect.Struct) {
+			(structFieldType.Kind() == reflect.Ptr && structFieldType.Elem().Kind() == reflect.Struct) {
 
 			variantMap, ok := value.Value().(map[string]dbus.Variant)
 			if ok {
-				if structFieldType.Kind() == reflect.Pointer {
+				if structFieldType.Kind() == reflect.Ptr {
 					if structFieldValue.IsZero() {
 						return fmt.Errorf("Pointer field %s: uninitialized", name)
 					}