cordova/sockets-for-cordova
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #11 from KoenLav/re-order-close

Browse Source 
First close then invoke close event handler
This commit is contained in:
Nikita 2021-01-23 14:20:05 +03:00 committed by GitHub
commit 62d86857b9
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/android/src/cz/blocshop/socketsforcordova/SocketAdapterImpl.java
View File

@ -76,8 +76,8 @@ public class SocketAdapterImpl implements SocketAdapter {
@Override @Override
public void close() throws IOException { public void close() throws IOException {
this.invokeCloseEventHandler(false);
this.socket.close(); this.socket.close();
this.invokeCloseEventHandler(false);
} }
@Override @Override

2
src/ios/SocketsForCordova/Classes/SocketAdapter.m
View File

@ -266,8 +266,8 @@ int writeTimeoutSeconds = 5.0;
} }
- (void)close { - (void)close {
self.closeEventHandler(FALSE);
[self closeStreams]; [self closeStreams];
self.closeEventHandler(FALSE);
} }
- (void)closeStreams { - (void)closeStreams {

2
src/wp8/src/SocketAdapter.cs
View File

@ -80,8 +80,8 @@ namespace Blocshop.ScoketsForCordova
public void Close() public void Close()
{ {
this.CloseEventHandler(false);
this.socket.Close(); this.socket.Close();
this.CloseEventHandler(false);
} }
private void StartReadTask() private void StartReadTask()