| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
| |
Consistency is good.
|
|\
| |
| | |
Profiling with scottg
|
| | |
|
| |
| |
| |
| |
| | |
This fixes a warning left by the earlier optimization and
addresses one of the other hotspots identified by GProf.
|
| |
| |
| |
| |
| |
| |
| |
| | |
GProf shows `git_text_gather_stats` as the most expensive call
in large diffs. The function calculates a lot of information
that is not actually used and does not do so in a optimal
order. This introduces a tuned `git_buf_is_binary` function
that executes the same algorithm in a fraction of the time.
|
| |
| |
| |
| |
| | |
This fixes the examples so they will build and adds a PROFILE
option to the CMakeFile that enabled gprof info on non-Windows
|
|\ \
| |/
|/| |
Fix workdir iterators on empty directories
|
|/
|
|
|
|
|
| |
Creating a workdir iterator on a directory with absolutely
no files was returning an error (GIT_ENOTFOUND) instead of
an iterator for nothing. This fixes that and includes two
new tests that cover that case.
|
|\
| |
| | |
Fix status for files under ignored dirs
|
|/
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
There was a bug where tracked files inside directories that were
inside ignored directories where not being found by status. To
make that a little clearer, if you have a .gitignore with:
ignore/
And then have the following files:
ignore/dir/tracked <-- actually a tracked file
ignore/dir/untracked <-- should be ignored
Then we would show the tracked file as being removed (because
when we got the to contained item "dir/" inside the ignored
directory, we decided it was safe to skip -- bzzt, wrong!).
This update is much more careful about checking that we are
not skipping over any prefix of a tracked item, regardless of
whether it is ignored or not.
As documented in diff.c, this commit does create behavior that
still differs from core git with regards to the handling of
untracked files contained inside ignored directories. With
libgit2, those files will just not show up in status or diff.
With core git, those files don't show up in status or diff
either *unless* they are explicitly ignored by a .gitignore
pattern in which case they show up as ignored files.
Needless to say, this is a local behavior difference only, so
it should not be important and (to me) the libgit2 behavior
seems more consistent.
|
|\
| |
| | |
notes: make git_note_foreach() callback signature easier to cope with from a binding perspective
|
|/
|
|
| |
binding perspective
|
| |
|
| |
|
|
|
|
| |
These are deprecated and replaced with the diffing code in git2/diff.h
|
|\
| |
| | |
really reset walker with git_revwalk_reset
|
| |
| |
| |
| |
| |
| |
| |
| | |
From the description of git_revwalk_reset in revwalk.h the function should
clear all pushed and hidden commits, and leave the walker in a blank state (just like at creation).
Apparently everything gets reseted appart of pushed commits (walk->one and walk->twos)
This fix should reset the walker properly.
|
|\ \
| | |
| | | |
Update git_status_file and add ranged iterators
|
| | |
| | |
| | |
| | |
| | | |
This function fills in a git_buf with the common prefix of
an array of strings, but let's make that a little more clear.
|
| | | |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The goal of this work is to rewrite git_status_file to use the
same underlying code as git_status_foreach.
This is done in 3 phases:
1. Extend iterators to allow ranged iteration with start and
end prefixes for the range of file names to be covered.
2. Improve diff so that when there is a pathspec and there is
a common non-wildcard prefix of the pathspec, it will use
ranged iterators to minimize excess iteration.
3. Rewrite git_status_file to call git_status_foreach_ext
with a pathspec that covers just the one file being checked.
Since ranged iterators underlie the status & diff implementation,
this is actually fairly efficient. The workdir iterator does
end up loading the contents of all the directories down to the
single file, which should ideally be avoided, but it is pretty
good.
|
|\ \ \
| |/ /
|/| | |
A few indexer consistency checks
|
| | |
| | |
| | |
| | |
| | | |
Error out in finalize if there is junk after the packfile hash or we
couldn't process all the objects.
|
|/ / |
|
|\ \
| | |
| | | |
Fix compilation warnings
|
|/ / |
|
|\ \
| | |
| | | |
Add git_note_foreach()
|
| | | |
|
|/ /
| |
| |
| | |
This reverts commit 1093e2de22f6ca245b09d758a3510899a8362048.
|
|\ \
| | |
| | | |
branch: cover with test that moving a non existing branch returns ENOTFOUND
|
| |/ |
|
|\ \
| | |
| | | |
branch: make git_branch_delete() return GIT_ENOTFOUND when the branch doesn't exist
|
| |/
| |
| |
| | |
doesn't exist
|
|\ \
| | |
| | | |
See issue https://github.com/libgit2/libgit2/issues/680
|
| |/ |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Building a "shared object" (DLL) in Windows includes 2 steps:
- specify __declspec(dllexport)
when building the library itself. MSVC will disallow itself from
optimizing these symbols out and reference them in the PE's
Exports-Table.
Further, a static link library will be generated. This library
contains the symbols which are exported via the declsepc above.
The __declspec(dllexport) becomes part of the symbol-signature
(like parameter types in C++ are 'mangled' into the symbol name,
the export specifier is mingled with the name)
- specify __declspec(dllimport)
when using the library. This again mingles the declspec into the
name and declares the function / variable with external linkage.
cmake automatically adds -Dgit2_EXPORTS to the compiler arguments
when compiling the libgit2 project.
The 'git2' is the name specified via PROJECT() in CMakeLists.txt.
|
|\ \
| | |
| | | |
Add git_blob_create_fromdisk()
|
| | | |
|
| |/
| |
| |
| | |
This function will create blobs in the object database from files anywhere on the filesystem. This can be run against bare and non-bare repositories.
|
|\ \
| | |
| | | |
Fix build/runtime issues on Solaris
|
| | |
| | |
| | |
| | |
| | |
| | | |
Since we now rely on it (at least under Solaris), I figured we probably
want to make sure it's accurate. The new test makes sure that creating a
file with a name of length FILENAME_MAX+1 fails.
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
On Solaris, struct dirent is defined differently than Linux. The field
containing the path name is of size 0, rather than NAME_MAX. So, we need to
use a properly sized buffer on Solaris to avoid a stack overflow.
Also fix some DIR* leaks on cleanup.
|
| | | |
|
| | |
| | |
| | |
| | |
| | | |
On Solaris, rmdir() throws EEXIST if the folder is not empty, so just add
one more case to check for that, alongside ENOTEMPTY.
|