Make the network code use handles #212

Merged
carlosmn merged 1 commits from remote-handle into master 2015-06-23 06:33:34 -05:00
carlosmn commented 2015-06-10 06:38:44 -05:00 (Migrated from github.com)

This wasn't ported together with the rest, but it does exhibit the same
issues, so let's port it over now.

This wasn't ported together with the rest, but it does exhibit the same issues, so let's port it over now.
carlosmn commented 2015-06-10 06:40:07 -05:00 (Migrated from github.com)

/cc @michaeledgar @radeksimko this should be enough to get the code in master to work. The code in next requires a somewhat different handling dealing more with the C allocator.

/cc @michaeledgar @radeksimko this should be enough to get the code in `master` to work. The code in `next` requires a somewhat different handling dealing more with the C allocator.
michaeledgar commented 2015-06-18 13:56:10 -05:00 (Migrated from github.com)

This patch is good, and resolves my issues. Thanks!

Sorry it took so long to respond to this - I had a separate bug in my code which presented nearly identically during remote operations. (I was trying to reuse *git.Cred objects, but the underlying *C.git_cred is freed by libgit2 after the first use)

This patch is good, and resolves my issues. Thanks! Sorry it took so long to respond to this - I had a separate bug in my code which presented nearly identically during remote operations. (I was trying to reuse *git.Cred objects, but the underlying *C.git_cred is freed by libgit2 after the first use)
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: jcarr/git2go#212
No description provided.