Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add cursor type for more safety. #647

Closed
wants to merge 2 commits into from

Conversation

nickmshelley
Copy link
Collaborator

No description provided.

@nickmshelley
Copy link
Collaborator Author

This gives a much better diff (adding ?w=1 to the URL): https://github.com/stephencelis/SQLite.swift/pull/647/files?w=1

@nickmshelley
Copy link
Collaborator Author

Most of the discussion around this PR happened in #569.

@stephanheilner
Copy link
Contributor

👍

@nickmshelley nickmshelley requested review from kujenga and removed request for kujenga May 5, 2017 21:19
@stephanheilner
Copy link
Contributor

What is the status of this PR? I'd love to have this crash fix in my apps. @jberkel @stephencelis

@nickmshelley
Copy link
Collaborator Author

Is there anything else that should be done to get this merged @jberkel? If it's just that you don't have time to look into it more, I can just go ahead and merge it.

@jberkel
Copy link
Collaborator

jberkel commented Sep 16, 2017

@nickmshelley can you rebase this PR against the release branch?

@nickmshelley
Copy link
Collaborator Author

@jberkel I opened #726 against the release branch.

Thanks for taking the time to update the library and deal with PRs and issues.

@jberkel
Copy link
Collaborator

jberkel commented Sep 25, 2017

replaced w/ #726

@jberkel jberkel closed this Sep 25, 2017
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants