search

ARMmbed / sockets

mbed sockets library abstraction layer
Other
6 stars 18 forks source link

TCPAsynch.cpp: Operator usage in destructor doesn't stick to correct type #52

Closed adbridge closed 8 years ago

adbridge commented 8 years ago
  1. Numerical variables such as _TCPSockets should not be checked for being zero by using the boolean ! operator. Should instead use == 0
  2. _tick_handle should be checked for being non-NULL before being used
  3. _tick_handle should be reset to NULL rather than 0
ciarmcom commented 8 years ago

ARM Internal Ref: IOTSFW-1601

adbridge commented 8 years ago

@bremoran just waiting review I believe/ closing

bremoran commented 8 years ago

This issue has been resolved in https://github.com/ARMmbed/sockets/pull/49