Skip to content
This repository was archived by the owner on Sep 6, 2020. It is now read-only.

Conversation

@pefoley2
Copy link

@pefoley2 pefoley2 commented Apr 4, 2013

Here is a change I had to make while converting my application from AutobahnAndroid to android-websockets to take advantage of the latter's ssl support.

I converted my application from AutobahnAndroid to android-websockets
to take advantage of ssl support.
android-websockets did not have a isConnected method, which is used in
my application, so I implemented one.
@koush
Copy link
Owner

koush commented Apr 4, 2013

java 6 (which is the target on this project) uses @OverRide on interfaces.

@koush
Copy link
Owner

koush commented Apr 4, 2013

Please split the changes into two pulls, as I would happily accept the first, but I fail to see the point of the second

@pefoley2
Copy link
Author

pefoley2 commented Apr 7, 2013

It appears that my java target version was erroneously set to 1.5.
I'll drop the second patch.

koush added a commit that referenced this pull request Apr 8, 2013
add isConnected method and remove incorrect annotations.
@koush koush merged commit 84119d1 into koush:master Apr 8, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants