| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| | |
|
| |
|
|
|
|
|
|
|
|
| |
References and their logs are logically coupled, let's make it so in
the code by moving the fs-based reflog implementation to live next to
the fs-based refs one.
As part of the change, make the function take names rather than
references, as only the names are relevant when looking up and
handling reflogs.
|
| |
|
|
| |
...before the helper's cleanup method tries to delete their files.
|
| | |
|
| |
|
|
|
|
|
|
| |
There were a lot of places in the test code base that were creating
a commit from the index on the current branch. This just adds a
helper to handle that case pretty easily. There was only one test
where this change ended up tweaking the test data, so pretty easy
and mostly just a cleanup.
|
| |
|
|
|
|
|
|
|
|
|
| |
Unfortunately git-core uses the term "unborn branch" and "orphan
branch" interchangeably. However, "orphan" is only really there for
the checkout command, which has the `--orphan` option so it doesn't
actually create the branch.
Branches never have parents, so the distinction of a branch with no
parents is odd to begin with. Crucially, the error messages deal with
unborn branches, so let's use that.
|
| | |
|
| |
|
|
| |
This will probably prevent many lookup/free
operations in calling code.
|
| | |
|
| | |
|
| |
|
|
|
|
|
|
|
| |
`git_diff_get_patch()` would unconditionally load the patch object and
then simply leak it if the user hadn't requested it. Short-circuit
loading the object if the user doesn't want it.
The rest of the plugs are simply calling the free functions of objects
allocated during the tests.
|
| | |
|
| | |
|
| |
|
|
| |
to match
|
| |
|
|
|
|
|
|
|
|
|
|
| |
Stash was sometimes obscuring the actual error code, replacing it
with a -1 when there was more descriptive value. This updates
stash to preserve the original error code more reliably along
with a variety of other error handling tweaks.
I believe this is an improvement, but arguably, preserving the
underlying error code may result in values that are harder to
interpret by the caller who does not understand the internals.
Discussion is welcome!
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
|
|
|
|
|
|
|
|
|
|
| |
Without this change, any failed assertion in the second (or a later) test
inside a test suite has a chance of double deleting memory, resulting in
a heap corruption. See #1096 for details.
This leaves alone the test cases where we "just" use cl_git_sandbox_init()
and cl_git_sandbox_cleanup(). These methods already take good care to not
double delete a repository.
Fixes #1096
|
| |
|
|
|
| |
Passing 0 as the index now retrieves the most recent entry instead
of the oldest one.
|
| |\
| |
| | |
Update checkout with new strategies & behavior
|
| | |
| |
| |
| |
| | |
This fixes some various warnings that showed up in Travis and
a couple uses of uninitialized memory and one memory leak.
|
| | | |
|
| |/ |
|
| | |
|
| |\
| |
| | |
Index changes
|
| | | |
|
| |/ |
|
| | |
|
| | |
|
| |
|
