[go: up one dir, main page]

Menu

Commit [r1355]  Maximize  Restore  History

review-2: rejected delwin commit 1329 (CloseSocketWraper)

- typo (misspelled)
- broken: removing virtual CHost::Close() layer bypasses subclass-specific cleanup code, can cause fd-leaks
- added utility method CloseSocketRef() analogous to r1329 'CloseSocketWraper' in string_socket.(h|cpp)
- re-factored default CHost::Close() to just call new CloseSocketRef()
- virtual method & other (virtual) (CHost*)->Close() calls unchanged

mukau 2019-11-13

changed /ddc/branches/delwin-merge2/src/CommonLib/CHost.cpp
changed /ddc/branches/delwin-merge2/src/CommonLib/CHost.h
changed /ddc/branches/delwin-merge2/src/CommonLib/string_socket.cpp
changed /ddc/branches/delwin-merge2/src/CommonLib/string_socket.h