search

YUKAI / konashi-android-sdk

konashi Android SDK
http://konashi.ux-xu.com
Apache License 2.0
7 stars 7 forks source link

Fix value length #109

Closed e10dokup closed 9 years ago

e10dokup commented 9 years ago

value 0byte目のlengthの長さにはaddressも含まれるので+1して対応

izumin5210 commented 9 years ago

wercker.ymlが古すぎてCI走らないので，最新のdevelopをmergeしてください

izumin5210 commented 9 years ago

I2cWriteAction#hasValidParams()（実体はI2cHelper.isValidDataLength(int length)）も修正したほうがいいかもしれない．

izumin5210 commented 9 years ago 
I2C_DATA_MAX_LENGTH =16
I2C_MAX_VALUE_SIZE = 20

に対して

int size = (mData.length < Konashi.I2C_DATA_MAX_LENGTH) ? mData.length + 2 : Konashi.MAX_VALUE_SIZE;

は違和感

izumin5210 commented 9 years ago

実装はLGTM 寿司ゆき:sigh

あとはテストですね

e10dokup commented 9 years ago

実機挙動がおかしい（setValue()開始直後にmDataの中身がすでに変？）なのでmergeは待って下さい

e10dokup commented 9 years ago

（adxl345用の処理になっていたのに気づかなかった）

izumin5210 commented 9 years ago

:ok_woman: