Fix arduino#380 characteristic written returns incorrect value after first connect

sgbihu · sys_maker · commit fe14b8bd2600 · 2017-02-08T13:34:26.000-08:00
1. Not set the flage when local device try to set the value.
2. Changed file
    libraries/CurieBLE/src/internal/BLECharacteristicImp.cpp
diff --git a/libraries/CurieBLE/src/internal/BLECharacteristicImp.cpp b/libraries/CurieBLE/src/internal/BLECharacteristicImp.cpp
@@ -260,6 +260,7 @@ bool
 BLECharacteristicImp::setValue(const unsigned char value[], uint16_t length)
 {
     _setValue(value, length, 0);
+    _value_updated = true;
     if (BLEUtils::isLocalBLE(_ble_device) == true)
     {
         // GATT server
@@ -555,7 +556,6 @@ BLECharacteristicImp::_setValue(const uint8_t value[], uint16_t length, uint16_t
         }
     }
     
-    _value_updated = true;
     memcpy(_value + offset, value, length);
     _value_length = length;
 }

Original file line number	Diff line number	Diff line change
`@@ -260,6 +260,7 @@ bool`
`260`	`260`	`BLECharacteristicImp::setValue(const unsigned char value[], uint16_t length)`
`261`	`261`	`{`
`262`	`262`	`_setValue(value, length, 0);`
	`263`	`+ _value_updated = true;`
`263`	`264`	`if (BLEUtils::isLocalBLE(_ble_device) == true)`
`264`	`265`	`{`
`265`	`266`	`// GATT server`
`@@ -555,7 +556,6 @@ BLECharacteristicImp::_setValue(const uint8_t value[], uint16_t length, uint16_t`
`555`	`556`	`}`
`556`	`557`	`}`
`557`	`558`
`558`		`- _value_updated = true;`
`559`	`559`	`memcpy(_value + offset, value, length);`
`560`	`560`	`_value_length = length;`
`561`	`561`	`}`