search

maarten-pennings / CCS811

Arduino library for the CCS811 gas sensor for monitoring indoor air quality.
MIT License
165 stars 46 forks source link

constructor not working #1

Closed holgerlembke closed 5 years ago

holgerlembke commented 6 years ago

Hi.

I see the idea you had with your constructor using default values. On the other hand, if you use CCS811() as a constructor call, this will call the default constructor, because it is still there and fits better than the constructor you created.

So just a hint, that it might not work the intended way...

  public: // Main interface
    CCS811() {
        CCS811(-1,CCS811_SLAVADDR_0);
    }
    CCS811(int nwake=-1, int slaveaddr=CCS811_SLAVADDR_0);
maarten-pennings commented 5 years ago

Hi - sorry for the very late reaction. I didn't known that github projects have active Issues list be default.

I tried to understand you comment, but I don't. Maybe you can give me some more help. I tried the attached test-cons.ino script, it gave the attached test-cons.txt output. And that seems to do what I want.

Thanks for the feedback.

test-cons.ino.txt test-cons.txt

maarten-pennings commented 5 years ago

Shall I close the issue?

holgerlembke commented 5 years ago

yes please.

as far as I see, I'm wrong...