adafruit / Adafruit_TCS34725

Arduino library driver for Adafruit's TCS34725 RGB Color Sensor Breakout
http://www.adafruit.com/products/1334
Other
149 stars 149 forks source link

Disable function is private. #13

Closed tgidave closed 5 years ago

tgidave commented 8 years ago

Is there a reason the disable function is private?

Humancell commented 7 years ago

I just ran into this same issue. I am using this in a battery powered device, and wanted to use the "sleep" mode ... which seems to be what the disable() function does.

I edited the header, and moved disable(void) up under enable(void) to make it public, and am now calling it for testing ... we'll see if this does what I need ...

But I am also very curious ... why was disable() made private, and not used?

a18rhodes commented 5 years ago

Same issue here. Seems that this is a very old issue, perhaps Adafruit could update.

a18rhodes commented 5 years ago

I forked this and updated then submitted a pull request. Hopefully that helps get it fixed!