Skip to content

added int return value#56

Open
lefty01 wants to merge 1 commit intoWifWaf:masterfrom
lefty01:main.add_ret_val_on_begin
Open

added int return value#56
lefty01 wants to merge 1 commit intoWifWaf:masterfrom
lefty01:main.add_ret_val_on_begin

Conversation

@lefty01
Copy link
Copy Markdown

@lefty01 lefty01 commented Dec 8, 2023

added int return value to verify and begin functions to check if sensor connection failed.
I found it useful to have some indication whether a sensor might be attached or not . with this change one could use code like this:

  co2Serial.begin(9600);
  if (0 == co2MHZ19.begin(co2Serial)) {
    co2MHZ19OK = true;
    co2MHZ19.autoCalibration();
  }

not sure if you want to integrate this or not?

@lefty01
Copy link
Copy Markdown
Author

lefty01 commented Dec 8, 2023

I also noticed that the version string on top of the .h/.cpp files do not match tag and library.properties

@lefty01 lefty01 force-pushed the main.add_ret_val_on_begin branch from 9cc58c0 to 3a463a1 Compare December 8, 2023 18:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant