search

washo4evr / Socket.io-v1.x-Library

Socket.io Library for Arduino
108 stars 58 forks source link

Updated return values to true & false as function return type is bool #73

Closed jbteja closed 4 years ago

jbteja commented 4 years ago

Hi Washo4evr,

I realized the monitor function return type is 'bool' and In my yesterday's commit mentioned it as int.

So, I made changes accordingly and Created the PR. Please review it.

Changes:

  1. Took privilege of combining all if conditions into one (Hoping you will be okay with it :|).
  2. The char variable 'which' is nowhere used. So removed it.
  3. Removed white-spaces at two places.
washo4evr commented 4 years ago

Thanks for taking the time. Havent really played with node.js in a long time but glad to see you are finding value in this library :)

jbteja commented 4 years ago

Thanks for merging the changes.. :)