Skip to content

Conversation

@Ettemlevest
Copy link
Contributor

@avbdr
Copy link
Collaborator

avbdr commented Apr 22, 2016

can you please update readme as well? thank you

@Ettemlevest
Copy link
Contributor Author

Thanks for the reminder. I added a small description to the readme.

@avbdr
Copy link
Collaborator

avbdr commented Apr 22, 2016

as well, I noticed you are not cleaning up errno variable in reset() function. Its getting called after each query execution to remove all the data

@Ettemlevest
Copy link
Contributor Author

I just added the errno parts after every $_stmtError lines but I did not check to reset the values. In this case we should also reset the $_stmtError string.

@avbdr
Copy link
Collaborator

avbdr commented Apr 22, 2016

sorry, my bad, this variables shouldnt be reset :) as you are using them after execution
fine. ill land this as is

@avbdr avbdr merged commit a62e491 into ThingEngineer:master Apr 22, 2016
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.

2 participants