Skip to content

Various fixes and features#1

Merged
aburgm merged 4 commits intogobby:masterfrom
polarina:master
Jun 30, 2014
Merged

Various fixes and features#1
aburgm merged 4 commits intogobby:masterfrom
polarina:master

Conversation

@polarina
Copy link
Copy Markdown
Contributor

These patches fix bug #623, implements enhancement #612, and implements features required for enhancement #626 in Gobby.

@aburgm
Copy link
Copy Markdown
Contributor

aburgm commented Jun 30, 2014

Thanks very much for the contribution! I have added some comments to some individual commits. Do you think you can change the pull request accordingly?

@aburgm
Copy link
Copy Markdown
Contributor

aburgm commented Jun 30, 2014

By the way, I have seen that you have also added more patches to the Gobby bugtracker. Do you plan to create pull requests for those as well, or should I have a look at the individual patches in the bugtracker?

@polarina
Copy link
Copy Markdown
Contributor Author

The patches I've added in the Gobby bugtracker are identical to the commits in this pull request. I only added them for completeness.

polarina added 3 commits June 30, 2014 13:54
…e_user_area

The bug manifests itself when show_remote_cursors and show_remote_selections
evaluate to FALSE and show_remote_current_lines evaluates to TRUE.

Usual symptoms are crashes.
@polarina
Copy link
Copy Markdown
Contributor Author

I've resolved the issues you've pointed out, to the best of my knowledge. How do the patches look now?

@aburgm aburgm merged commit 27ef5d4 into gobby:master Jun 30, 2014
@aburgm
Copy link
Copy Markdown
Contributor

aburgm commented Jun 30, 2014

Very good, thanks!

@nirik nirik mentioned this pull request Oct 11, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants