Simplify git_revparse_ext usage

[nulltoken]

No description provided.

[nulltoken]

@jamill What's your opinion about this?

[jamill]

Should we call GC.SuppressFinalize() at the end of this method?

[nulltoken]

Oh! Indeed. Thanks for this

[jamill]

Actually, I think I was mistaken here - I don't think this is needed.

From Object.Finalize documentation:

The Object class provides no implementation for the Finalize method, and the garbage collector does not mark types derived from Object for finalization unless they override the Finalize method.

[nulltoken]

Fixed

[jamill]

DisposableTuple feels a bit awkward to me - it is kindof a general construct, but is tailored to a specific use case here. What would you think about just creating a private class that encapsulates the return value of git_revparse_ext and has 2 specific members with friendly names?

[nulltoken]

What would you think about just creating a private class that encapsulates the return value of git_revparse_ext and has 2 specific members with friendly names?

I'll try this out and report back when it's done.