Unverified Commit c3bbbcf5 by Patrick Steinhardt Committed by GitHub

Merge pull request #5117 from libgit2/ethomson/to_from

Change API instances of `fromnoun` to `from_noun` (with an underscore)
parents 2bdb617a e45350fe
...@@ -72,7 +72,7 @@ int lg2_blame(git_repository *repo, int argc, char *argv[]) ...@@ -72,7 +72,7 @@ int lg2_blame(git_repository *repo, int argc, char *argv[])
* Get the raw data inside the blob for output. We use the * Get the raw data inside the blob for output. We use the
* `commitish:path/to/file.txt` format to find it. * `commitish:path/to/file.txt` format to find it.
*/ */
if (git_oid_iszero(&blameopts.newest_commit)) if (git_oid_is_zero(&blameopts.newest_commit))
strcpy(spec, "HEAD"); strcpy(spec, "HEAD");
else else
git_oid_tostr(spec, sizeof(spec), &blameopts.newest_commit); git_oid_tostr(spec, sizeof(spec), &blameopts.newest_commit);
......
...@@ -21,7 +21,7 @@ static int update_cb(const char *refname, const git_oid *a, const git_oid *b, vo ...@@ -21,7 +21,7 @@ static int update_cb(const char *refname, const git_oid *a, const git_oid *b, vo
git_oid_fmt(b_str, b); git_oid_fmt(b_str, b);
b_str[GIT_OID_HEXSZ] = '\0'; b_str[GIT_OID_HEXSZ] = '\0';
if (git_oid_iszero(a)) { if (git_oid_is_zero(a)) {
printf("[new] %.20s %s\n", b_str, refname); printf("[new] %.20s %s\n", b_str, refname);
} else { } else {
git_oid_fmt(a_str, a); git_oid_fmt(a_str, a);
......
...@@ -136,7 +136,7 @@ GIT_EXTERN(int) git_blob_filtered_content( ...@@ -136,7 +136,7 @@ GIT_EXTERN(int) git_blob_filtered_content(
* relative to the repository's working dir * relative to the repository's working dir
* @return 0 or an error code * @return 0 or an error code
*/ */
GIT_EXTERN(int) git_blob_create_fromworkdir(git_oid *id, git_repository *repo, const char *relative_path); GIT_EXTERN(int) git_blob_create_from_workdir(git_oid *id, git_repository *repo, const char *relative_path);
/** /**
* Read a file from the filesystem and write its content * Read a file from the filesystem and write its content
...@@ -148,7 +148,7 @@ GIT_EXTERN(int) git_blob_create_fromworkdir(git_oid *id, git_repository *repo, c ...@@ -148,7 +148,7 @@ GIT_EXTERN(int) git_blob_create_fromworkdir(git_oid *id, git_repository *repo, c
* @param path file from which the blob will be created * @param path file from which the blob will be created
* @return 0 or an error code * @return 0 or an error code
*/ */
GIT_EXTERN(int) git_blob_create_fromdisk(git_oid *id, git_repository *repo, const char *path); GIT_EXTERN(int) git_blob_create_from_disk(git_oid *id, git_repository *repo, const char *path);
/** /**
* Create a stream to write a new blob into the object db * Create a stream to write a new blob into the object db
...@@ -156,12 +156,12 @@ GIT_EXTERN(int) git_blob_create_fromdisk(git_oid *id, git_repository *repo, cons ...@@ -156,12 +156,12 @@ GIT_EXTERN(int) git_blob_create_fromdisk(git_oid *id, git_repository *repo, cons
* This function may need to buffer the data on disk and will in * This function may need to buffer the data on disk and will in
* general not be the right choice if you know the size of the data * general not be the right choice if you know the size of the data
* to write. If you have data in memory, use * to write. If you have data in memory, use
* `git_blob_create_frombuffer()`. If you do not, but know the size of * `git_blob_create_from_buffer()`. If you do not, but know the size of
* the contents (and don't want/need to perform filtering), use * the contents (and don't want/need to perform filtering), use
* `git_odb_open_wstream()`. * `git_odb_open_wstream()`.
* *
* Don't close this stream yourself but pass it to * Don't close this stream yourself but pass it to
* `git_blob_create_fromstream_commit()` to commit the write to the * `git_blob_create_from_stream_commit()` to commit the write to the
* object db and get the object id. * object db and get the object id.
* *
* If the `hintpath` parameter is filled, it will be used to determine * If the `hintpath` parameter is filled, it will be used to determine
...@@ -175,7 +175,7 @@ GIT_EXTERN(int) git_blob_create_fromdisk(git_oid *id, git_repository *repo, cons ...@@ -175,7 +175,7 @@ GIT_EXTERN(int) git_blob_create_fromdisk(git_oid *id, git_repository *repo, cons
* to apply onto the content of the blob to be created. * to apply onto the content of the blob to be created.
* @return 0 or error code * @return 0 or error code
*/ */
GIT_EXTERN(int) git_blob_create_fromstream( GIT_EXTERN(int) git_blob_create_from_stream(
git_writestream **out, git_writestream **out,
git_repository *repo, git_repository *repo,
const char *hintpath); const char *hintpath);
...@@ -189,7 +189,7 @@ GIT_EXTERN(int) git_blob_create_fromstream( ...@@ -189,7 +189,7 @@ GIT_EXTERN(int) git_blob_create_fromstream(
* @param stream the stream to close * @param stream the stream to close
* @return 0 or an error code * @return 0 or an error code
*/ */
GIT_EXTERN(int) git_blob_create_fromstream_commit( GIT_EXTERN(int) git_blob_create_from_stream_commit(
git_oid *out, git_oid *out,
git_writestream *stream); git_writestream *stream);
...@@ -202,7 +202,7 @@ GIT_EXTERN(int) git_blob_create_fromstream_commit( ...@@ -202,7 +202,7 @@ GIT_EXTERN(int) git_blob_create_fromstream_commit(
* @param len length of the data * @param len length of the data
* @return 0 or an error code * @return 0 or an error code
*/ */
GIT_EXTERN(int) git_blob_create_frombuffer( GIT_EXTERN(int) git_blob_create_from_buffer(
git_oid *id, git_repository *repo, const void *buffer, size_t len); git_oid *id, git_repository *repo, const void *buffer, size_t len);
/** /**
......
...@@ -45,6 +45,30 @@ ...@@ -45,6 +45,30 @@
*/ */
GIT_BEGIN_DECL GIT_BEGIN_DECL
/** @name Deprecated Blob Functions
*
* These functions are retained for backward compatibility. The newer
* versions of these functions should be preferred in all new code.
*
* There is no plan to remove these backward compatibility values at
* this time.
*/
/**@{*/
GIT_EXTERN(int) git_blob_create_fromworkdir(git_oid *id, git_repository *repo, const char *relative_path);
GIT_EXTERN(int) git_blob_create_fromdisk(git_oid *id, git_repository *repo, const char *path);
GIT_EXTERN(int) git_blob_create_fromstream(
git_writestream **out,
git_repository *repo,
const char *hintpath);
GIT_EXTERN(int) git_blob_create_fromstream_commit(
git_oid *out,
git_writestream *stream);
GIT_EXTERN(int) git_blob_create_frombuffer(
git_oid *id, git_repository *repo, const void *buffer, size_t len);
/**@}*/
/** @name Deprecated Buffer Functions /** @name Deprecated Buffer Functions
* *
* These functions and enumeration values are retained for backward * These functions and enumeration values are retained for backward
...@@ -167,10 +191,11 @@ GIT_EXTERN(void) giterr_set_oom(void); ...@@ -167,10 +191,11 @@ GIT_EXTERN(void) giterr_set_oom(void);
/**@}*/ /**@}*/
/** @name Deprecated Index Constants /** @name Deprecated Index Functions and Constants
* *
* These enumeration values are retained for backward compatibility. * These functions and enumeration values are retained for backward
* The newer versions of these values should be preferred in all new code. * compatibility. The newer versions of these values should be
* preferred in all new code.
* *
* There is no plan to remove these backward compatibility values at * There is no plan to remove these backward compatibility values at
* this time. * this time.
...@@ -210,6 +235,11 @@ GIT_EXTERN(void) giterr_set_oom(void); ...@@ -210,6 +235,11 @@ GIT_EXTERN(void) giterr_set_oom(void);
#define GIT_INDEXCAP_NO_SYMLINKS GIT_INDEX_CAPABILITY_NO_SYMLINKS #define GIT_INDEXCAP_NO_SYMLINKS GIT_INDEX_CAPABILITY_NO_SYMLINKS
#define GIT_INDEXCAP_FROM_OWNER GIT_INDEX_CAPABILITY_FROM_OWNER #define GIT_INDEXCAP_FROM_OWNER GIT_INDEX_CAPABILITY_FROM_OWNER
GIT_EXTERN(int) git_index_add_frombuffer(
git_index *index,
const git_index_entry *entry,
const void *buffer, size_t len);
/**@}*/ /**@}*/
/** @name Deprecated Object Constants /** @name Deprecated Object Constants
...@@ -263,6 +293,12 @@ GIT_EXTERN(size_t) git_object__size(git_object_t type); ...@@ -263,6 +293,12 @@ GIT_EXTERN(size_t) git_object__size(git_object_t type);
#define GIT_REF_FORMAT_REFSPEC_PATTERN GIT_REFERENCE_FORMAT_REFSPEC_PATTERN #define GIT_REF_FORMAT_REFSPEC_PATTERN GIT_REFERENCE_FORMAT_REFSPEC_PATTERN
#define GIT_REF_FORMAT_REFSPEC_SHORTHAND GIT_REFERENCE_FORMAT_REFSPEC_SHORTHAND #define GIT_REF_FORMAT_REFSPEC_SHORTHAND GIT_REFERENCE_FORMAT_REFSPEC_SHORTHAND
GIT_EXTERN(int) git_tag_create_frombuffer(
git_oid *oid,
git_repository *repo,
const char *buffer,
int force);
/**@}*/ /**@}*/
/** @name Deprecated Credential Callback Types /** @name Deprecated Credential Callback Types
...@@ -273,7 +309,6 @@ GIT_EXTERN(size_t) git_object__size(git_object_t type); ...@@ -273,7 +309,6 @@ GIT_EXTERN(size_t) git_object__size(git_object_t type);
* There is no plan to remove these backward compatibility values at * There is no plan to remove these backward compatibility values at
* this time. * this time.
*/ */
/**@{*/
typedef git_cred_sign_cb git_cred_sign_callback; typedef git_cred_sign_cb git_cred_sign_callback;
typedef git_cred_ssh_interactive_cb git_cred_ssh_interactive_callback; typedef git_cred_ssh_interactive_cb git_cred_ssh_interactive_callback;
...@@ -294,6 +329,20 @@ typedef git_trace_cb git_trace_callback; ...@@ -294,6 +329,20 @@ typedef git_trace_cb git_trace_callback;
/**@}*/ /**@}*/
/** @name Deprecated Object ID Types
*
* These types are retained for backward compatibility. The newer
* versions of these values should be preferred in all new code.
*
* There is no plan to remove these backward compatibility values at
* this time.
*/
/**@{*/
GIT_EXTERN(int) git_oid_iszero(const git_oid *id);
/**@}*/
/** @name Deprecated Transfer Progress Types /** @name Deprecated Transfer Progress Types
* *
* These types are retained for backward compatibility. The newer * These types are retained for backward compatibility. The newer
......
...@@ -572,7 +572,7 @@ GIT_EXTERN(int) git_index_add_bypath(git_index *index, const char *path); ...@@ -572,7 +572,7 @@ GIT_EXTERN(int) git_index_add_bypath(git_index *index, const char *path);
* @param len length of the data * @param len length of the data
* @return 0 or an error code * @return 0 or an error code
*/ */
GIT_EXTERN(int) git_index_add_frombuffer( GIT_EXTERN(int) git_index_add_from_buffer(
git_index *index, git_index *index,
const git_index_entry *entry, const git_index_entry *entry,
const void *buffer, size_t len); const void *buffer, size_t len);
......
...@@ -207,7 +207,7 @@ GIT_EXTERN(int) git_oid_strcmp(const git_oid *id, const char *str); ...@@ -207,7 +207,7 @@ GIT_EXTERN(int) git_oid_strcmp(const git_oid *id, const char *str);
* *
* @return 1 if all zeros, 0 otherwise. * @return 1 if all zeros, 0 otherwise.
*/ */
GIT_EXTERN(int) git_oid_iszero(const git_oid *id); GIT_EXTERN(int) git_oid_is_zero(const git_oid *id);
/** /**
* OID Shortener object * OID Shortener object
......
...@@ -217,7 +217,7 @@ GIT_EXTERN(int) git_tag_annotation_create( ...@@ -217,7 +217,7 @@ GIT_EXTERN(int) git_tag_annotation_create(
* @param force Overwrite existing tags * @param force Overwrite existing tags
* @return 0 on success; error code otherwise * @return 0 on success; error code otherwise
*/ */
GIT_EXTERN(int) git_tag_create_frombuffer( GIT_EXTERN(int) git_tag_create_from_buffer(
git_oid *oid, git_oid *oid,
git_repository *repo, git_repository *repo,
const char *buffer, const char *buffer,
......
...@@ -532,7 +532,7 @@ static int apply_one( ...@@ -532,7 +532,7 @@ static int apply_one(
if (delta->status != GIT_DELTA_DELETED) { if (delta->status != GIT_DELTA_DELETED) {
if ((error = git_apply__patch(&post_contents, &filename, &mode, if ((error = git_apply__patch(&post_contents, &filename, &mode,
pre_contents.ptr, pre_contents.size, patch, opts)) < 0 || pre_contents.ptr, pre_contents.size, patch, opts)) < 0 ||
(error = git_blob_create_frombuffer(&post_id, repo, (error = git_blob_create_from_buffer(&post_id, repo,
post_contents.ptr, post_contents.size)) < 0) post_contents.ptr, post_contents.size)) < 0)
goto done; goto done;
......
...@@ -204,7 +204,7 @@ static int normalize_options( ...@@ -204,7 +204,7 @@ static int normalize_options(
memcpy(out, in, sizeof(git_blame_options)); memcpy(out, in, sizeof(git_blame_options));
/* No newest_commit => HEAD */ /* No newest_commit => HEAD */
if (git_oid_iszero(&out->newest_commit)) { if (git_oid_is_zero(&out->newest_commit)) {
if (git_reference_name_to_id(&out->newest_commit, repo, "HEAD") < 0) { if (git_reference_name_to_id(&out->newest_commit, repo, "HEAD") < 0) {
return -1; return -1;
} }
...@@ -408,7 +408,7 @@ on_error: ...@@ -408,7 +408,7 @@ on_error:
static bool hunk_is_bufferblame(git_blame_hunk *hunk) static bool hunk_is_bufferblame(git_blame_hunk *hunk)
{ {
return git_oid_iszero(&hunk->final_commit_id); return git_oid_is_zero(&hunk->final_commit_id);
} }
static int buffer_hunk_cb( static int buffer_hunk_cb(
......
...@@ -70,7 +70,7 @@ int git_blob__parse(void *_blob, git_odb_object *odb_obj) ...@@ -70,7 +70,7 @@ int git_blob__parse(void *_blob, git_odb_object *odb_obj)
return 0; return 0;
} }
int git_blob_create_frombuffer( int git_blob_create_from_buffer(
git_oid *id, git_repository *repo, const void *buffer, size_t len) git_oid *id, git_repository *repo, const void *buffer, size_t len)
{ {
int error; int error;
...@@ -263,13 +263,13 @@ done: ...@@ -263,13 +263,13 @@ done:
return error; return error;
} }
int git_blob_create_fromworkdir( int git_blob_create_from_workdir(
git_oid *id, git_repository *repo, const char *path) git_oid *id, git_repository *repo, const char *path)
{ {
return git_blob__create_from_paths(id, NULL, repo, NULL, path, 0, true); return git_blob__create_from_paths(id, NULL, repo, NULL, path, 0, true);
} }
int git_blob_create_fromdisk( int git_blob_create_from_disk(
git_oid *id, git_repository *repo, const char *path) git_oid *id, git_repository *repo, const char *path)
{ {
int error; int error;
...@@ -325,7 +325,7 @@ static int blob_writestream_write(git_writestream *_stream, const char *buffer, ...@@ -325,7 +325,7 @@ static int blob_writestream_write(git_writestream *_stream, const char *buffer,
return git_filebuf_write(&stream->fbuf, buffer, len); return git_filebuf_write(&stream->fbuf, buffer, len);
} }
int git_blob_create_fromstream(git_writestream **out, git_repository *repo, const char *hintpath) int git_blob_create_from_stream(git_writestream **out, git_repository *repo, const char *hintpath)
{ {
int error; int error;
git_buf path = GIT_BUF_INIT; git_buf path = GIT_BUF_INIT;
...@@ -364,7 +364,7 @@ cleanup: ...@@ -364,7 +364,7 @@ cleanup:
return error; return error;
} }
int git_blob_create_fromstream_commit(git_oid *out, git_writestream *_stream) int git_blob_create_from_stream_commit(git_oid *out, git_writestream *_stream)
{ {
int error; int error;
blob_writestream *stream = (blob_writestream *) _stream; blob_writestream *stream = (blob_writestream *) _stream;
...@@ -427,3 +427,36 @@ int git_blob_filtered_content( ...@@ -427,3 +427,36 @@ int git_blob_filtered_content(
return error; return error;
} }
/* Deprecated functions */
int git_blob_create_frombuffer(
git_oid *id, git_repository *repo, const void *buffer, size_t len)
{
return git_blob_create_from_buffer(id, repo, buffer, len);
}
int git_blob_create_fromworkdir(git_oid *id, git_repository *repo, const char *relative_path)
{
return git_blob_create_from_workdir(id, repo, relative_path);
}
int git_blob_create_fromdisk(git_oid *id, git_repository *repo, const char *path)
{
return git_blob_create_from_disk(id, repo, path);
}
int git_blob_create_fromstream(
git_writestream **out,
git_repository *repo,
const char *hintpath)
{
return git_blob_create_from_stream(out, repo, hintpath);
}
int git_blob_create_fromstream_commit(
git_oid *out,
git_writestream *stream)
{
return git_blob_create_from_stream_commit(out, stream);
}
...@@ -232,7 +232,7 @@ static int diff_file_content_load_blob( ...@@ -232,7 +232,7 @@ static int diff_file_content_load_blob(
int error = 0; int error = 0;
git_odb_object *odb_obj = NULL; git_odb_object *odb_obj = NULL;
if (git_oid_iszero(&fc->file->id)) if (git_oid_is_zero(&fc->file->id))
return 0; return 0;
if (fc->file->mode == GIT_FILEMODE_COMMIT) if (fc->file->mode == GIT_FILEMODE_COMMIT)
......
...@@ -179,7 +179,7 @@ static int diff_delta__from_one( ...@@ -179,7 +179,7 @@ static int diff_delta__from_one(
delta->old_file.flags |= GIT_DIFF_FLAG_VALID_ID; delta->old_file.flags |= GIT_DIFF_FLAG_VALID_ID;
if (has_old || !git_oid_iszero(&delta->new_file.id)) if (has_old || !git_oid_is_zero(&delta->new_file.id))
delta->new_file.flags |= GIT_DIFF_FLAG_VALID_ID; delta->new_file.flags |= GIT_DIFF_FLAG_VALID_ID;
return diff_insert_delta(diff, delta, matched_pathspec); return diff_insert_delta(diff, delta, matched_pathspec);
...@@ -240,7 +240,7 @@ static int diff_delta__from_two( ...@@ -240,7 +240,7 @@ static int diff_delta__from_two(
delta->old_file.flags |= GIT_DIFF_FLAG_EXISTS; delta->old_file.flags |= GIT_DIFF_FLAG_EXISTS;
delta->new_file.flags |= GIT_DIFF_FLAG_EXISTS; delta->new_file.flags |= GIT_DIFF_FLAG_EXISTS;
if (!git_oid_iszero(&new_entry->id)) if (!git_oid_is_zero(&new_entry->id))
delta->new_file.flags |= GIT_DIFF_FLAG_VALID_ID; delta->new_file.flags |= GIT_DIFF_FLAG_VALID_ID;
} }
...@@ -797,13 +797,13 @@ static int maybe_modified( ...@@ -797,13 +797,13 @@ static int maybe_modified(
/* if oids and modes match (and are valid), then file is unmodified */ /* if oids and modes match (and are valid), then file is unmodified */
} else if (git_oid_equal(&oitem->id, &nitem->id) && } else if (git_oid_equal(&oitem->id, &nitem->id) &&
omode == nmode && omode == nmode &&
!git_oid_iszero(&oitem->id)) { !git_oid_is_zero(&oitem->id)) {
status = GIT_DELTA_UNMODIFIED; status = GIT_DELTA_UNMODIFIED;
/* if we have an unknown OID and a workdir iterator, then check some /* if we have an unknown OID and a workdir iterator, then check some
* circumstances that can accelerate things or need special handling * circumstances that can accelerate things or need special handling
*/ */
} else if (git_oid_iszero(&nitem->id) && new_is_workdir) { } else if (git_oid_is_zero(&nitem->id) && new_is_workdir) {
bool use_ctime = bool use_ctime =
((diff->diffcaps & GIT_DIFFCAPS_TRUST_CTIME) != 0); ((diff->diffcaps & GIT_DIFFCAPS_TRUST_CTIME) != 0);
git_index *index = git_iterator_index(info->new_iter); git_index *index = git_iterator_index(info->new_iter);
...@@ -843,7 +843,7 @@ static int maybe_modified( ...@@ -843,7 +843,7 @@ static int maybe_modified(
/* if we got here and decided that the files are modified, but we /* if we got here and decided that the files are modified, but we
* haven't calculated the OID of the new item, then calculate it now * haven't calculated the OID of the new item, then calculate it now
*/ */
if (modified_uncertain && git_oid_iszero(&nitem->id)) { if (modified_uncertain && git_oid_is_zero(&nitem->id)) {
const git_oid *update_check = const git_oid *update_check =
DIFF_FLAG_IS_SET(diff, GIT_DIFF_UPDATE_INDEX) && omode == nmode ? DIFF_FLAG_IS_SET(diff, GIT_DIFF_UPDATE_INDEX) && omode == nmode ?
&oitem->id : NULL; &oitem->id : NULL;
...@@ -877,7 +877,7 @@ static int maybe_modified( ...@@ -877,7 +877,7 @@ static int maybe_modified(
return diff_delta__from_two( return diff_delta__from_two(
diff, status, oitem, omode, nitem, nmode, diff, status, oitem, omode, nitem, nmode,
git_oid_iszero(&noid) ? NULL : &noid, matched_pathspec); git_oid_is_zero(&noid) ? NULL : &noid, matched_pathspec);
} }
static bool entry_is_prefixed( static bool entry_is_prefixed(
......
...@@ -325,10 +325,10 @@ static int diff_delta_format_with_paths( ...@@ -325,10 +325,10 @@ static int diff_delta_format_with_paths(
const char *oldpath, const char *oldpath,
const char *newpath) const char *newpath)
{ {
if (git_oid_iszero(&delta->old_file.id)) if (git_oid_is_zero(&delta->old_file.id))
oldpath = "/dev/null"; oldpath = "/dev/null";
if (git_oid_iszero(&delta->new_file.id)) if (git_oid_is_zero(&delta->new_file.id))
newpath = "/dev/null"; newpath = "/dev/null";
return git_buf_printf(out, template, oldpath, newpath); return git_buf_printf(out, template, oldpath, newpath);
...@@ -381,8 +381,8 @@ done: ...@@ -381,8 +381,8 @@ done:
static bool delta_is_unchanged(const git_diff_delta *delta) static bool delta_is_unchanged(const git_diff_delta *delta)
{ {
if (git_oid_iszero(&delta->old_file.id) && if (git_oid_is_zero(&delta->old_file.id) &&
git_oid_iszero(&delta->new_file.id)) git_oid_is_zero(&delta->new_file.id))
return true; return true;
if (delta->old_file.mode == GIT_FILEMODE_COMMIT || if (delta->old_file.mode == GIT_FILEMODE_COMMIT ||
......
...@@ -560,13 +560,13 @@ static int similarity_measure( ...@@ -560,13 +560,13 @@ static int similarity_measure(
/* if exact match is requested, force calculation of missing OIDs now */ /* if exact match is requested, force calculation of missing OIDs now */
if (exact_match) { if (exact_match) {
if (git_oid_iszero(&a_file->id) && if (git_oid_is_zero(&a_file->id) &&
diff->old_src == GIT_ITERATOR_TYPE_WORKDIR && diff->old_src == GIT_ITERATOR_TYPE_WORKDIR &&
!git_diff__oid_for_file(&a_file->id, !git_diff__oid_for_file(&a_file->id,
diff, a_file->path, a_file->mode, a_file->size)) diff, a_file->path, a_file->mode, a_file->size))
a_file->flags |= GIT_DIFF_FLAG_VALID_ID; a_file->flags |= GIT_DIFF_FLAG_VALID_ID;
if (git_oid_iszero(&b_file->id) && if (git_oid_is_zero(&b_file->id) &&
diff->new_src == GIT_ITERATOR_TYPE_WORKDIR && diff->new_src == GIT_ITERATOR_TYPE_WORKDIR &&
!git_diff__oid_for_file(&b_file->id, !git_diff__oid_for_file(&b_file->id,
diff, b_file->path, b_file->mode, b_file->size)) diff, b_file->path, b_file->mode, b_file->size))
......
...@@ -385,7 +385,7 @@ uint16_t git_filter_source_filemode(const git_filter_source *src) ...@@ -385,7 +385,7 @@ uint16_t git_filter_source_filemode(const git_filter_source *src)
const git_oid *git_filter_source_id(const git_filter_source *src) const git_oid *git_filter_source_id(const git_filter_source *src)
{ {
return git_oid_iszero(&src->oid) ? NULL : &src->oid; return git_oid_is_zero(&src->oid) ? NULL : &src->oid;
} }
git_filter_mode_t git_filter_source_mode(const git_filter_source *src) git_filter_mode_t git_filter_source_mode(const git_filter_source *src)
......
...@@ -1454,7 +1454,7 @@ GIT_INLINE(bool) valid_filemode(const int filemode) ...@@ -1454,7 +1454,7 @@ GIT_INLINE(bool) valid_filemode(const int filemode)
return (is_file_or_link(filemode) || filemode == GIT_FILEMODE_COMMIT); return (is_file_or_link(filemode) || filemode == GIT_FILEMODE_COMMIT);
} }
int git_index_add_frombuffer( int git_index_add_from_buffer(
git_index *index, const git_index_entry *source_entry, git_index *index, const git_index_entry *source_entry,
const void *buffer, size_t len) const void *buffer, size_t len)
{ {
...@@ -1477,7 +1477,7 @@ int git_index_add_frombuffer( ...@@ -1477,7 +1477,7 @@ int git_index_add_frombuffer(
if (index_entry_dup(&entry, index, source_entry) < 0) if (index_entry_dup(&entry, index, source_entry) < 0)
return -1; return -1;
error = git_blob_create_frombuffer(&id, INDEX_OWNER(index), buffer, len); error = git_blob_create_from_buffer(&id, INDEX_OWNER(index), buffer, len);
if (error < 0) { if (error < 0) {
index_entry_free(entry); index_entry_free(entry);
return error; return error;
...@@ -3709,3 +3709,12 @@ void git_indexwriter_cleanup(git_indexwriter *writer) ...@@ -3709,3 +3709,12 @@ void git_indexwriter_cleanup(git_indexwriter *writer)
git_index_free(writer->index); git_index_free(writer->index);
writer->index = NULL; writer->index = NULL;
} }
/* Deprecated functions */
int git_index_add_frombuffer(
git_index *index, const git_index_entry *source_entry,
const void *buffer, size_t len)
{
return git_index_add_from_buffer(index, source_entry, buffer, len);
}
...@@ -288,7 +288,7 @@ static int note_write( ...@@ -288,7 +288,7 @@ static int note_write(
/* TODO: should we apply filters? */ /* TODO: should we apply filters? */
/* create note object */ /* create note object */
if ((error = git_blob_create_frombuffer(&oid, repo, note, strlen(note))) < 0) if ((error = git_blob_create_from_buffer(&oid, repo, note, strlen(note))) < 0)
goto cleanup; goto cleanup;
if ((error = manipulate_note_in_tree_r( if ((error = manipulate_note_in_tree_r(
......
...@@ -766,7 +766,7 @@ int git_odb_exists(git_odb *db, const git_oid *id) ...@@ -766,7 +766,7 @@ int git_odb_exists(git_odb *db, const git_oid *id)
assert(db && id); assert(db && id);
if (git_oid_iszero(id)) if (git_oid_is_zero(id))
return 0; return 0;
if ((object = git_cache_get_raw(odb_cache(db), id)) != NULL) { if ((object = git_cache_get_raw(odb_cache(db), id)) != NULL) {
...@@ -994,7 +994,7 @@ int git_odb__read_header_or_object( ...@@ -994,7 +994,7 @@ int git_odb__read_header_or_object(
*out = NULL; *out = NULL;
if (git_oid_iszero(id)) if (git_oid_is_zero(id))
return error_null_oid(GIT_ENOTFOUND, "cannot read object"); return error_null_oid(GIT_ENOTFOUND, "cannot read object");
if ((object = git_cache_get_raw(odb_cache(db), id)) != NULL) { if ((object = git_cache_get_raw(odb_cache(db), id)) != NULL) {
...@@ -1099,7 +1099,7 @@ int git_odb_read(git_odb_object **out, git_odb *db, const git_oid *id) ...@@ -1099,7 +1099,7 @@ int git_odb_read(git_odb_object **out, git_odb *db, const git_oid *id)
assert(out && db && id); assert(out && db && id);
if (git_oid_iszero(id)) if (git_oid_is_zero(id))
return error_null_oid(GIT_ENOTFOUND, "cannot read object"); return error_null_oid(GIT_ENOTFOUND, "cannot read object");
*out = git_cache_get_raw(odb_cache(db), id); *out = git_cache_get_raw(odb_cache(db), id);
...@@ -1123,7 +1123,7 @@ static int odb_otype_fast(git_object_t *type_p, git_odb *db, const git_oid *id) ...@@ -1123,7 +1123,7 @@ static int odb_otype_fast(git_object_t *type_p, git_odb *db, const git_oid *id)
size_t _unused; size_t _unused;
int error; int error;
if (git_oid_iszero(id)) if (git_oid_is_zero(id))
return error_null_oid(GIT_ENOTFOUND, "cannot get object type"); return error_null_oid(GIT_ENOTFOUND, "cannot get object type");
if ((object = git_cache_get_raw(odb_cache(db), id)) != NULL) { if ((object = git_cache_get_raw(odb_cache(db), id)) != NULL) {
...@@ -1283,7 +1283,7 @@ int git_odb_write( ...@@ -1283,7 +1283,7 @@ int git_odb_write(
git_odb_hash(oid, data, len, type); git_odb_hash(oid, data, len, type);
if (git_oid_iszero(oid)) if (git_oid_is_zero(oid))
return error_null_oid(GIT_EINVALID, "cannot write object"); return error_null_oid(GIT_EINVALID, "cannot write object");
if (git_odb__freshen(db, oid)) if (git_odb__freshen(db, oid))
......
...@@ -237,7 +237,7 @@ int git_oid_streq(const git_oid *oid_a, const char *str) ...@@ -237,7 +237,7 @@ int git_oid_streq(const git_oid *oid_a, const char *str)
return git_oid_strcmp(oid_a, str) == 0 ? 0 : -1; return git_oid_strcmp(oid_a, str) == 0 ? 0 : -1;
} }
int git_oid_iszero(const git_oid *oid_a) int git_oid_is_zero(const git_oid *oid_a)
{ {
const unsigned char *a = oid_a->id; const unsigned char *a = oid_a->id;
unsigned int i; unsigned int i;
...@@ -247,6 +247,11 @@ int git_oid_iszero(const git_oid *oid_a) ...@@ -247,6 +247,11 @@ int git_oid_iszero(const git_oid *oid_a)
return 1; return 1;
} }
int git_oid_iszero(const git_oid *oid_a)
{
return git_oid_is_zero(oid_a);
}
typedef short node_index; typedef short node_index;
typedef union { typedef union {
......
...@@ -196,7 +196,7 @@ int git_push_update_tips(git_push *push, const git_remote_callbacks *callbacks) ...@@ -196,7 +196,7 @@ int git_push_update_tips(git_push *push, const git_remote_callbacks *callbacks)
continue; continue;
/* Update the remote ref */ /* Update the remote ref */
if (git_oid_iszero(&push_spec->loid)) { if (git_oid_is_zero(&push_spec->loid)) {
error = git_reference_lookup(&remote_ref, push->remote->repo, git_buf_cstr(&remote_ref_name)); error = git_reference_lookup(&remote_ref, push->remote->repo, git_buf_cstr(&remote_ref_name));
if (error >= 0) { if (error >= 0) {
...@@ -281,7 +281,7 @@ static int queue_objects(git_push *push) ...@@ -281,7 +281,7 @@ static int queue_objects(git_push *push)
git_object_t type; git_object_t type;
size_t size; size_t size;
if (git_oid_iszero(&spec->loid)) if (git_oid_is_zero(&spec->loid))
/* /*
* Delete reference on remote side; * Delete reference on remote side;
* nothing to do here. * nothing to do here.
...@@ -319,7 +319,7 @@ static int queue_objects(git_push *push) ...@@ -319,7 +319,7 @@ static int queue_objects(git_push *push)
if (!spec->refspec.force) { if (!spec->refspec.force) {
git_oid base; git_oid base;
if (git_oid_iszero(&spec->roid)) if (git_oid_is_zero(&spec->roid))
continue; continue;
if (!git_odb_exists(push->repo->_odb, &spec->roid)) { if (!git_odb_exists(push->repo->_odb, &spec->roid)) {
...@@ -346,7 +346,7 @@ static int queue_objects(git_push *push) ...@@ -346,7 +346,7 @@ static int queue_objects(git_push *push)
} }
git_vector_foreach(&push->remote->refs, i, head) { git_vector_foreach(&push->remote->refs, i, head) {
if (git_oid_iszero(&head->oid)) if (git_oid_is_zero(&head->oid))
continue; continue;
/* TODO */ /* TODO */
......
...@@ -386,7 +386,7 @@ const git_oid *git_reference_target_peel(const git_reference *ref) ...@@ -386,7 +386,7 @@ const git_oid *git_reference_target_peel(const git_reference *ref)
{ {
assert(ref); assert(ref);
if (ref->type != GIT_REFERENCE_DIRECT || git_oid_iszero(&ref->peel)) if (ref->type != GIT_REFERENCE_DIRECT || git_oid_is_zero(&ref->peel))
return NULL; return NULL;
return &ref->peel; return &ref->peel;
...@@ -1380,7 +1380,7 @@ int git_reference_peel( ...@@ -1380,7 +1380,7 @@ int git_reference_peel(
* to a commit. So we only want to use the peeled value * to a commit. So we only want to use the peeled value
* if it is not zero and the target is not a tag. * if it is not zero and the target is not a tag.
*/ */
if (target_type != GIT_OBJECT_TAG && !git_oid_iszero(&resolved->peel)) { if (target_type != GIT_OBJECT_TAG && !git_oid_is_zero(&resolved->peel)) {
error = git_object_lookup(&target, error = git_object_lookup(&target,
git_reference_owner(ref), &resolved->peel, GIT_OBJECT_ANY); git_reference_owner(ref), &resolved->peel, GIT_OBJECT_ANY);
} else { } else {
......
...@@ -86,14 +86,14 @@ static unsigned int workdir_delta2status( ...@@ -86,14 +86,14 @@ static unsigned int workdir_delta2status(
/* if OIDs don't match, we might need to calculate them now to /* if OIDs don't match, we might need to calculate them now to
* discern between RENAMED vs RENAMED+MODIFED * discern between RENAMED vs RENAMED+MODIFED
*/ */
if (git_oid_iszero(&idx2wd->old_file.id) && if (git_oid_is_zero(&idx2wd->old_file.id) &&
diff->old_src == GIT_ITERATOR_TYPE_WORKDIR && diff->old_src == GIT_ITERATOR_TYPE_WORKDIR &&
!git_diff__oid_for_file( !git_diff__oid_for_file(
&idx2wd->old_file.id, diff, idx2wd->old_file.path, &idx2wd->old_file.id, diff, idx2wd->old_file.path,
idx2wd->old_file.mode, idx2wd->old_file.size)) idx2wd->old_file.mode, idx2wd->old_file.size))
idx2wd->old_file.flags |= GIT_DIFF_FLAG_VALID_ID; idx2wd->old_file.flags |= GIT_DIFF_FLAG_VALID_ID;
if (git_oid_iszero(&idx2wd->new_file.id) && if (git_oid_is_zero(&idx2wd->new_file.id) &&
diff->new_src == GIT_ITERATOR_TYPE_WORKDIR && diff->new_src == GIT_ITERATOR_TYPE_WORKDIR &&
!git_diff__oid_for_file( !git_diff__oid_for_file(
&idx2wd->new_file.id, diff, idx2wd->new_file.path, &idx2wd->new_file.id, diff, idx2wd->new_file.path,
......
...@@ -328,7 +328,7 @@ int git_tag_create_lightweight( ...@@ -328,7 +328,7 @@ int git_tag_create_lightweight(
return git_tag_create__internal(oid, repo, tag_name, target, NULL, NULL, allow_ref_overwrite, 0); return git_tag_create__internal(oid, repo, tag_name, target, NULL, NULL, allow_ref_overwrite, 0);
} }
int git_tag_create_frombuffer(git_oid *oid, git_repository *repo, const char *buffer, int allow_ref_overwrite) int git_tag_create_from_buffer(git_oid *oid, git_repository *repo, const char *buffer, int allow_ref_overwrite)
{ {
git_tag tag; git_tag tag;
int error; int error;
...@@ -521,3 +521,10 @@ int git_tag_peel(git_object **tag_target, const git_tag *tag) ...@@ -521,3 +521,10 @@ int git_tag_peel(git_object **tag_target, const git_tag *tag)
{ {
return git_object_peel(tag_target, (const git_object *)tag, GIT_OBJECT_ANY); return git_object_peel(tag_target, (const git_object *)tag, GIT_OBJECT_ANY);
} }
/* Deprecated Functions */
int git_tag_create_frombuffer(git_oid *oid, git_repository *repo, const char *buffer, int allow_ref_overwrite)
{
return git_tag_create_from_buffer(oid, repo, buffer, allow_ref_overwrite);
}
...@@ -309,7 +309,7 @@ static int local_push_update_remote_ref( ...@@ -309,7 +309,7 @@ static int local_push_update_remote_ref(
if (lref[0] != '\0') { if (lref[0] != '\0') {
/* Create or update a ref */ /* Create or update a ref */
error = git_reference_create(NULL, remote_repo, rref, loid, error = git_reference_create(NULL, remote_repo, rref, loid,
!git_oid_iszero(roid), NULL); !git_oid_is_zero(roid), NULL);
} else { } else {
/* Delete a ref */ /* Delete a ref */
if ((error = git_reference_lookup(&remote_ref, remote_repo, rref)) < 0) { if ((error = git_reference_lookup(&remote_ref, remote_repo, rref)) < 0) {
......
...@@ -286,7 +286,7 @@ static int git_smart__connect( ...@@ -286,7 +286,7 @@ static int git_smart__connect(
if ((error = git_smart__detect_caps(first, &t->caps, &symrefs)) == 0) { if ((error = git_smart__detect_caps(first, &t->caps, &symrefs)) == 0) {
/* If the only ref in the list is capabilities^{} with OID_ZERO, remove it */ /* If the only ref in the list is capabilities^{} with OID_ZERO, remove it */
if (1 == t->refs.length && !strcmp(first->head.name, "capabilities^{}") && if (1 == t->refs.length && !strcmp(first->head.name, "capabilities^{}") &&
git_oid_iszero(&first->head.oid)) { git_oid_is_zero(&first->head.oid)) {
git_vector_clear(&t->refs); git_vector_clear(&t->refs);
git_pkt_free((git_pkt *)first); git_pkt_free((git_pkt *)first);
} }
......
...@@ -963,7 +963,7 @@ static int update_refs_from_report( ...@@ -963,7 +963,7 @@ static int update_refs_from_report(
/* Remove any refs which we updated to have a zero OID. */ /* Remove any refs which we updated to have a zero OID. */
git_vector_rforeach(refs, i, ref) { git_vector_rforeach(refs, i, ref) {
if (git_oid_iszero(&ref->head.oid)) { if (git_oid_is_zero(&ref->head.oid)) {
git_vector_remove(refs, i); git_vector_remove(refs, i);
git_pkt_free((git_pkt *)ref); git_pkt_free((git_pkt *)ref);
} }
......
...@@ -479,7 +479,7 @@ static int check_entry(git_repository *repo, const char *filename, const git_oid ...@@ -479,7 +479,7 @@ static int check_entry(git_repository *repo, const char *filename, const git_oid
if (!valid_entry_name(repo, filename)) if (!valid_entry_name(repo, filename))
return tree_error("failed to insert entry: invalid name for a tree entry", filename); return tree_error("failed to insert entry: invalid name for a tree entry", filename);
if (git_oid_iszero(id)) if (git_oid_is_zero(id))
return tree_error("failed to insert entry: invalid null OID", filename); return tree_error("failed to insert entry: invalid null OID", filename);
if (filemode != GIT_FILEMODE_COMMIT && if (filemode != GIT_FILEMODE_COMMIT &&
......
...@@ -315,7 +315,7 @@ void test_diff_blob__can_compare_against_null_blobs_with_patch(void) ...@@ -315,7 +315,7 @@ void test_diff_blob__can_compare_against_null_blobs_with_patch(void)
cl_assert_equal_i(GIT_DELTA_DELETED, delta->status); cl_assert_equal_i(GIT_DELTA_DELETED, delta->status);
cl_assert_equal_oid(git_blob_id(d), &delta->old_file.id); cl_assert_equal_oid(git_blob_id(d), &delta->old_file.id);
cl_assert_equal_sz(git_blob_rawsize(d), delta->old_file.size); cl_assert_equal_sz(git_blob_rawsize(d), delta->old_file.size);
cl_assert(git_oid_iszero(&delta->new_file.id)); cl_assert(git_oid_is_zero(&delta->new_file.id));
cl_assert_equal_sz(0, delta->new_file.size); cl_assert_equal_sz(0, delta->new_file.size);
cl_assert_equal_i(1, (int)git_patch_num_hunks(p)); cl_assert_equal_i(1, (int)git_patch_num_hunks(p));
...@@ -338,7 +338,7 @@ void test_diff_blob__can_compare_against_null_blobs_with_patch(void) ...@@ -338,7 +338,7 @@ void test_diff_blob__can_compare_against_null_blobs_with_patch(void)
delta = git_patch_get_delta(p); delta = git_patch_get_delta(p);
cl_assert(delta != NULL); cl_assert(delta != NULL);
cl_assert_equal_i(GIT_DELTA_ADDED, delta->status); cl_assert_equal_i(GIT_DELTA_ADDED, delta->status);
cl_assert(git_oid_iszero(&delta->old_file.id)); cl_assert(git_oid_is_zero(&delta->old_file.id));
cl_assert_equal_sz(0, delta->old_file.size); cl_assert_equal_sz(0, delta->old_file.size);
cl_assert_equal_oid(git_blob_id(d), &delta->new_file.id); cl_assert_equal_oid(git_blob_id(d), &delta->new_file.id);
cl_assert_equal_sz(git_blob_rawsize(d), delta->new_file.size); cl_assert_equal_sz(git_blob_rawsize(d), delta->new_file.size);
...@@ -445,9 +445,9 @@ void test_diff_blob__can_compare_identical_blobs_with_patch(void) ...@@ -445,9 +445,9 @@ void test_diff_blob__can_compare_identical_blobs_with_patch(void)
cl_assert(delta != NULL); cl_assert(delta != NULL);
cl_assert_equal_i(GIT_DELTA_UNMODIFIED, delta->status); cl_assert_equal_i(GIT_DELTA_UNMODIFIED, delta->status);
cl_assert_equal_sz(0, delta->old_file.size); cl_assert_equal_sz(0, delta->old_file.size);
cl_assert(git_oid_iszero(&delta->old_file.id)); cl_assert(git_oid_is_zero(&delta->old_file.id));
cl_assert_equal_sz(0, delta->new_file.size); cl_assert_equal_sz(0, delta->new_file.size);
cl_assert(git_oid_iszero(&delta->new_file.id)); cl_assert(git_oid_is_zero(&delta->new_file.id));
cl_assert_equal_i(0, (int)git_patch_num_hunks(p)); cl_assert_equal_i(0, (int)git_patch_num_hunks(p));
git_patch_free(p); git_patch_free(p);
......
...@@ -87,14 +87,14 @@ void test_filter_blob__ident(void) ...@@ -87,14 +87,14 @@ void test_filter_blob__ident(void)
git_buf buf = { 0 }; git_buf buf = { 0 };
cl_git_mkfile("crlf/test.ident", "Some text\n$Id$\nGoes there\n"); cl_git_mkfile("crlf/test.ident", "Some text\n$Id$\nGoes there\n");
cl_git_pass(git_blob_create_fromworkdir(&id, g_repo, "test.ident")); cl_git_pass(git_blob_create_from_workdir(&id, g_repo, "test.ident"));
cl_git_pass(git_blob_lookup(&blob, g_repo, &id)); cl_git_pass(git_blob_lookup(&blob, g_repo, &id));
cl_assert_equal_s( cl_assert_equal_s(
"Some text\n$Id$\nGoes there\n", git_blob_rawcontent(blob)); "Some text\n$Id$\nGoes there\n", git_blob_rawcontent(blob));
git_blob_free(blob); git_blob_free(blob);
cl_git_mkfile("crlf/test.ident", "Some text\n$Id: Any old just you want$\nGoes there\n"); cl_git_mkfile("crlf/test.ident", "Some text\n$Id: Any old just you want$\nGoes there\n");
cl_git_pass(git_blob_create_fromworkdir(&id, g_repo, "test.ident")); cl_git_pass(git_blob_create_from_workdir(&id, g_repo, "test.ident"));
cl_git_pass(git_blob_lookup(&blob, g_repo, &id)); cl_git_pass(git_blob_lookup(&blob, g_repo, &id));
cl_assert_equal_s( cl_assert_equal_s(
"Some text\n$Id$\nGoes there\n", git_blob_rawcontent(blob)); "Some text\n$Id$\nGoes there\n", git_blob_rawcontent(blob));
......
...@@ -23,7 +23,7 @@ static void add_blob_and_filter( ...@@ -23,7 +23,7 @@ static void add_blob_and_filter(
git_buf out = { 0 }; git_buf out = { 0 };
cl_git_mkfile("crlf/identtest", data); cl_git_mkfile("crlf/identtest", data);
cl_git_pass(git_blob_create_fromworkdir(&id, g_repo, "identtest")); cl_git_pass(git_blob_create_from_workdir(&id, g_repo, "identtest"));
cl_git_pass(git_blob_lookup(&blob, g_repo, &id)); cl_git_pass(git_blob_lookup(&blob, g_repo, &id));
cl_git_pass(git_filter_list_apply_to_blob(&out, fl, blob)); cl_git_pass(git_filter_list_apply_to_blob(&out, fl, blob));
......
...@@ -164,7 +164,7 @@ static void add_entry_and_check_mode_( ...@@ -164,7 +164,7 @@ static void add_entry_and_check_mode_(
git_index_entry new_entry; git_index_entry new_entry;
/* If old_filename exists, we copy that to the new file, and test /* If old_filename exists, we copy that to the new file, and test
* git_index_add(), otherwise create a new entry testing git_index_add_frombuffer * git_index_add(), otherwise create a new entry testing git_index_add_from_buffer
*/ */
if (from_file) if (from_file)
{ {
...@@ -189,7 +189,7 @@ static void add_entry_and_check_mode_( ...@@ -189,7 +189,7 @@ static void add_entry_and_check_mode_(
else else
{ {
const char *content = "hey there\n"; const char *content = "hey there\n";
clar__assert(!git_index_add_frombuffer(index, &new_entry, content, strlen(content)), clar__assert(!git_index_add_from_buffer(index, &new_entry, content, strlen(content)),
file, line, "Cannot add index entry from buffer", NULL, 1); file, line, "Cannot add index entry from buffer", NULL, 1);
} }
...@@ -207,7 +207,7 @@ void test_index_filemodes__explicit(void) ...@@ -207,7 +207,7 @@ void test_index_filemodes__explicit(void)
git_index *index; git_index *index;
/* These tests should run and work everywhere, as the filemode is /* These tests should run and work everywhere, as the filemode is
* given explicitly to git_index_add or git_index_add_frombuffer * given explicitly to git_index_add or git_index_add_from_buffer
*/ */
cl_repo_set_bool(g_repo, "core.filemode", false); cl_repo_set_bool(g_repo, "core.filemode", false);
...@@ -271,7 +271,7 @@ void test_index_filemodes__frombuffer_requires_files(void) ...@@ -271,7 +271,7 @@ void test_index_filemodes__frombuffer_requires_files(void)
new_entry.path = "dummy-file.txt"; new_entry.path = "dummy-file.txt";
new_entry.mode = GIT_FILEMODE_BLOB; new_entry.mode = GIT_FILEMODE_BLOB;
cl_git_pass(git_index_add_frombuffer(index, cl_git_pass(git_index_add_from_buffer(index,
&new_entry, content, strlen(content))); &new_entry, content, strlen(content)));
cl_assert((ret_entry = git_index_get_bypath(index, "dummy-file.txt", 0))); cl_assert((ret_entry = git_index_get_bypath(index, "dummy-file.txt", 0)));
...@@ -282,7 +282,7 @@ void test_index_filemodes__frombuffer_requires_files(void) ...@@ -282,7 +282,7 @@ void test_index_filemodes__frombuffer_requires_files(void)
new_entry.path = "dummy-file.txt"; new_entry.path = "dummy-file.txt";
new_entry.mode = GIT_FILEMODE_BLOB_EXECUTABLE; new_entry.mode = GIT_FILEMODE_BLOB_EXECUTABLE;
cl_git_pass(git_index_add_frombuffer(index, cl_git_pass(git_index_add_from_buffer(index,
&new_entry, content, strlen(content))); &new_entry, content, strlen(content)));
cl_assert((ret_entry = git_index_get_bypath(index, "dummy-file.txt", 0))); cl_assert((ret_entry = git_index_get_bypath(index, "dummy-file.txt", 0)));
...@@ -293,7 +293,7 @@ void test_index_filemodes__frombuffer_requires_files(void) ...@@ -293,7 +293,7 @@ void test_index_filemodes__frombuffer_requires_files(void)
new_entry.path = "dummy-link.txt"; new_entry.path = "dummy-link.txt";
new_entry.mode = GIT_FILEMODE_LINK; new_entry.mode = GIT_FILEMODE_LINK;
cl_git_pass(git_index_add_frombuffer(index, cl_git_pass(git_index_add_from_buffer(index,
&new_entry, content, strlen(content))); &new_entry, content, strlen(content)));
cl_assert((ret_entry = git_index_get_bypath(index, "dummy-link.txt", 0))); cl_assert((ret_entry = git_index_get_bypath(index, "dummy-link.txt", 0)));
...@@ -304,7 +304,7 @@ void test_index_filemodes__frombuffer_requires_files(void) ...@@ -304,7 +304,7 @@ void test_index_filemodes__frombuffer_requires_files(void)
new_entry.path = "invalid_mode.txt"; new_entry.path = "invalid_mode.txt";
new_entry.mode = GIT_FILEMODE_TREE; new_entry.mode = GIT_FILEMODE_TREE;
cl_git_fail(git_index_add_frombuffer(index, cl_git_fail(git_index_add_from_buffer(index,
&new_entry, content, strlen(content))); &new_entry, content, strlen(content)));
cl_assert_equal_p(NULL, git_index_get_bypath(index, "invalid_mode.txt", 0)); cl_assert_equal_p(NULL, git_index_get_bypath(index, "invalid_mode.txt", 0));
...@@ -312,7 +312,7 @@ void test_index_filemodes__frombuffer_requires_files(void) ...@@ -312,7 +312,7 @@ void test_index_filemodes__frombuffer_requires_files(void)
new_entry.path = "invalid_mode.txt"; new_entry.path = "invalid_mode.txt";
new_entry.mode = GIT_FILEMODE_COMMIT; new_entry.mode = GIT_FILEMODE_COMMIT;
cl_git_fail(git_index_add_frombuffer(index, cl_git_fail(git_index_add_from_buffer(index,
&new_entry, content, strlen(content))); &new_entry, content, strlen(content)));
cl_assert_equal_p(NULL, git_index_get_bypath(index, "invalid_mode.txt", 0)); cl_assert_equal_p(NULL, git_index_get_bypath(index, "invalid_mode.txt", 0));
......
...@@ -193,7 +193,7 @@ static void setup_uptodate_files(void) ...@@ -193,7 +193,7 @@ static void setup_uptodate_files(void)
/* Put 'C' into the index */ /* Put 'C' into the index */
new_entry.path = "C"; new_entry.path = "C";
new_entry.mode = GIT_FILEMODE_BLOB; new_entry.mode = GIT_FILEMODE_BLOB;
cl_git_pass(git_index_add_frombuffer(index, &new_entry, "hello!\n", 7)); cl_git_pass(git_index_add_from_buffer(index, &new_entry, "hello!\n", 7));
git_index_free(index); git_index_free(index);
git_buf_dispose(&path); git_buf_dispose(&path);
......
...@@ -310,7 +310,7 @@ void test_index_tests__add_frombuffer(void) ...@@ -310,7 +310,7 @@ void test_index_tests__add_frombuffer(void)
memset(&entry, 0x0, sizeof(git_index_entry)); memset(&entry, 0x0, sizeof(git_index_entry));
entry.mode = GIT_FILEMODE_BLOB; entry.mode = GIT_FILEMODE_BLOB;
entry.path = "test.txt"; entry.path = "test.txt";
cl_git_pass(git_index_add_frombuffer(index, &entry, cl_git_pass(git_index_add_from_buffer(index, &entry,
content, strlen(content))); content, strlen(content)));
/* Wow... it worked! */ /* Wow... it worked! */
...@@ -352,7 +352,7 @@ void test_index_tests__dirty_and_clean(void) ...@@ -352,7 +352,7 @@ void test_index_tests__dirty_and_clean(void)
/* Index is dirty after adding an entry */ /* Index is dirty after adding an entry */
entry.mode = GIT_FILEMODE_BLOB; entry.mode = GIT_FILEMODE_BLOB;
entry.path = "test.txt"; entry.path = "test.txt";
cl_git_pass(git_index_add_frombuffer(index, &entry, "Hi.\n", 4)); cl_git_pass(git_index_add_from_buffer(index, &entry, "Hi.\n", 4));
cl_assert(git_index_entrycount(index) == 1); cl_assert(git_index_entrycount(index) == 1);
cl_assert(git_index_is_dirty(index)); cl_assert(git_index_is_dirty(index));
...@@ -374,7 +374,7 @@ void test_index_tests__dirty_and_clean(void) ...@@ -374,7 +374,7 @@ void test_index_tests__dirty_and_clean(void)
cl_assert(!git_index_is_dirty(index)); cl_assert(!git_index_is_dirty(index));
/* Index is dirty when we do an unforced read with dirty content */ /* Index is dirty when we do an unforced read with dirty content */
cl_git_pass(git_index_add_frombuffer(index, &entry, "Hi.\n", 4)); cl_git_pass(git_index_add_from_buffer(index, &entry, "Hi.\n", 4));
cl_assert(git_index_entrycount(index) == 1); cl_assert(git_index_entrycount(index) == 1);
cl_assert(git_index_is_dirty(index)); cl_assert(git_index_is_dirty(index));
...@@ -402,7 +402,7 @@ void test_index_tests__dirty_fails_optionally(void) ...@@ -402,7 +402,7 @@ void test_index_tests__dirty_fails_optionally(void)
/* Index is dirty after adding an entry */ /* Index is dirty after adding an entry */
entry.mode = GIT_FILEMODE_BLOB; entry.mode = GIT_FILEMODE_BLOB;
entry.path = "test.txt"; entry.path = "test.txt";
cl_git_pass(git_index_add_frombuffer(index, &entry, "Hi.\n", 4)); cl_git_pass(git_index_add_from_buffer(index, &entry, "Hi.\n", 4));
cl_assert(git_index_is_dirty(index)); cl_assert(git_index_is_dirty(index));
cl_git_pass(git_checkout_head(repo, NULL)); cl_git_pass(git_checkout_head(repo, NULL));
...@@ -410,7 +410,7 @@ void test_index_tests__dirty_fails_optionally(void) ...@@ -410,7 +410,7 @@ void test_index_tests__dirty_fails_optionally(void)
/* Index is dirty (again) after adding an entry */ /* Index is dirty (again) after adding an entry */
entry.mode = GIT_FILEMODE_BLOB; entry.mode = GIT_FILEMODE_BLOB;
entry.path = "test.txt"; entry.path = "test.txt";
cl_git_pass(git_index_add_frombuffer(index, &entry, "Hi.\n", 4)); cl_git_pass(git_index_add_from_buffer(index, &entry, "Hi.\n", 4));
cl_assert(git_index_is_dirty(index)); cl_assert(git_index_is_dirty(index));
cl_git_pass(git_libgit2_opts(GIT_OPT_ENABLE_UNSAVED_INDEX_SAFETY, 1)); cl_git_pass(git_libgit2_opts(GIT_OPT_ENABLE_UNSAVED_INDEX_SAFETY, 1));
...@@ -455,7 +455,7 @@ void test_index_tests__add_frombuffer_reset_entry(void) ...@@ -455,7 +455,7 @@ void test_index_tests__add_frombuffer_reset_entry(void)
memset(&entry, 0x0, sizeof(git_index_entry)); memset(&entry, 0x0, sizeof(git_index_entry));
entry.mode = GIT_FILEMODE_BLOB; entry.mode = GIT_FILEMODE_BLOB;
entry.path = "test.txt"; entry.path = "test.txt";
cl_git_pass(git_index_add_frombuffer(index, &entry, cl_git_pass(git_index_add_from_buffer(index, &entry,
content, strlen(content))); content, strlen(content)));
/* Wow... it worked! */ /* Wow... it worked! */
......
...@@ -55,7 +55,7 @@ void test_index_version__can_write_v4(void) ...@@ -55,7 +55,7 @@ void test_index_version__can_write_v4(void)
memset(&entry, 0, sizeof(entry)); memset(&entry, 0, sizeof(entry));
entry.path = paths[i]; entry.path = paths[i];
entry.mode = GIT_FILEMODE_BLOB; entry.mode = GIT_FILEMODE_BLOB;
cl_git_pass(git_index_add_frombuffer(index, &entry, paths[i], cl_git_pass(git_index_add_from_buffer(index, &entry, paths[i],
strlen(paths[i]) + 1)); strlen(paths[i]) + 1));
} }
cl_assert_equal_sz(git_index_entrycount(index), ARRAY_SIZE(paths)); cl_assert_equal_sz(git_index_entrycount(index), ARRAY_SIZE(paths));
...@@ -100,7 +100,7 @@ void test_index_version__v4_uses_path_compression(void) ...@@ -100,7 +100,7 @@ void test_index_version__v4_uses_path_compression(void)
path[ARRAY_SIZE(path) - 3] = i; path[ARRAY_SIZE(path) - 3] = i;
path[ARRAY_SIZE(path) - 2] = j; path[ARRAY_SIZE(path) - 2] = j;
path[ARRAY_SIZE(path) - 1] = '\0'; path[ARRAY_SIZE(path) - 1] = '\0';
cl_git_pass(git_index_add_frombuffer(index, &entry, buf, sizeof(buf))); cl_git_pass(git_index_add_from_buffer(index, &entry, buf, sizeof(buf)));
} }
} }
......
...@@ -352,7 +352,7 @@ int merge_test_workdir(git_repository *repo, const struct merge_index_entry expe ...@@ -352,7 +352,7 @@ int merge_test_workdir(git_repository *repo, const struct merge_index_entry expe
return 0; return 0;
for (i = 0; i < expected_len; i++) { for (i = 0; i < expected_len; i++) {
git_blob_create_fromworkdir(&actual_oid, repo, expected[i].path); git_blob_create_from_workdir(&actual_oid, repo, expected[i].path);
git_oid_fromstr(&expected_oid, expected[i].oid_str); git_oid_fromstr(&expected_oid, expected[i].oid_str);
if (git_oid_cmp(&actual_oid, &expected_oid) != 0) if (git_oid_cmp(&actual_oid, &expected_oid) != 0)
......
...@@ -59,7 +59,7 @@ void test_object_blob_filter__initialize(void) ...@@ -59,7 +59,7 @@ void test_object_blob_filter__initialize(void)
if (g_crlf_raw_len[i] < 0) if (g_crlf_raw_len[i] < 0)
g_crlf_raw_len[i] = strlen(g_crlf_raw[i]); g_crlf_raw_len[i] = strlen(g_crlf_raw[i]);
cl_git_pass(git_blob_create_frombuffer( cl_git_pass(git_blob_create_from_buffer(
&g_crlf_oids[i], g_repo, g_crlf_raw[i], (size_t)g_crlf_raw_len[i])); &g_crlf_oids[i], g_repo, g_crlf_raw[i], (size_t)g_crlf_raw_len[i]));
} }
} }
......
...@@ -29,12 +29,12 @@ void test_object_blob_fromstream__multiple_write(void) ...@@ -29,12 +29,12 @@ void test_object_blob_fromstream__multiple_write(void)
cl_git_fail_with(GIT_ENOTFOUND, cl_git_fail_with(GIT_ENOTFOUND,
git_object_lookup(&blob, repo, &expected_id, GIT_OBJECT_ANY)); git_object_lookup(&blob, repo, &expected_id, GIT_OBJECT_ANY));
cl_git_pass(git_blob_create_fromstream(&stream, repo, NULL)); cl_git_pass(git_blob_create_from_stream(&stream, repo, NULL));
for (i = 0; i < howmany; i++) for (i = 0; i < howmany; i++)
cl_git_pass(stream->write(stream, textual_content, strlen(textual_content))); cl_git_pass(stream->write(stream, textual_content, strlen(textual_content)));
cl_git_pass(git_blob_create_fromstream_commit(&id, stream)); cl_git_pass(git_blob_create_from_stream_commit(&id, stream));
cl_assert_equal_oid(&expected_id, &id); cl_assert_equal_oid(&expected_id, &id);
cl_git_pass(git_object_lookup(&blob, repo, &expected_id, GIT_OBJECT_BLOB)); cl_git_pass(git_object_lookup(&blob, repo, &expected_id, GIT_OBJECT_BLOB));
...@@ -67,12 +67,12 @@ static void assert_named_chunked_blob(const char *expected_sha, const char *fake ...@@ -67,12 +67,12 @@ static void assert_named_chunked_blob(const char *expected_sha, const char *fake
cl_git_pass(git_oid_fromstr(&expected_id, expected_sha)); cl_git_pass(git_oid_fromstr(&expected_id, expected_sha));
cl_git_pass(git_blob_create_fromstream(&stream, repo, fake_name)); cl_git_pass(git_blob_create_from_stream(&stream, repo, fake_name));
for (i = 0; i < howmany; i++) for (i = 0; i < howmany; i++)
cl_git_pass(stream->write(stream, textual_content, strlen(textual_content))); cl_git_pass(stream->write(stream, textual_content, strlen(textual_content)));
cl_git_pass(git_blob_create_fromstream_commit(&id, stream)); cl_git_pass(git_blob_create_from_stream_commit(&id, stream));
cl_assert_equal_oid(&expected_id, &id); cl_assert_equal_oid(&expected_id, &id);
} }
......
...@@ -33,7 +33,7 @@ void test_object_blob_write__can_create_a_blob_in_a_standard_repo_from_a_file_lo ...@@ -33,7 +33,7 @@ void test_object_blob_write__can_create_a_blob_in_a_standard_repo_from_a_file_lo
{ {
repo = cl_git_sandbox_init(WORKDIR); repo = cl_git_sandbox_init(WORKDIR);
assert_blob_creation(WORKDIR "/test.txt", "test.txt", &git_blob_create_fromworkdir); assert_blob_creation(WORKDIR "/test.txt", "test.txt", &git_blob_create_from_workdir);
} }
void test_object_blob_write__can_create_a_blob_in_a_standard_repo_from_a_absolute_filepath_pointing_outside_of_the_working_directory(void) void test_object_blob_write__can_create_a_blob_in_a_standard_repo_from_a_absolute_filepath_pointing_outside_of_the_working_directory(void)
...@@ -46,7 +46,7 @@ void test_object_blob_write__can_create_a_blob_in_a_standard_repo_from_a_absolut ...@@ -46,7 +46,7 @@ void test_object_blob_write__can_create_a_blob_in_a_standard_repo_from_a_absolut
cl_must_pass(git_path_prettify_dir(&full_path, ELSEWHERE, NULL)); cl_must_pass(git_path_prettify_dir(&full_path, ELSEWHERE, NULL));
cl_must_pass(git_buf_puts(&full_path, "test.txt")); cl_must_pass(git_buf_puts(&full_path, "test.txt"));
assert_blob_creation(ELSEWHERE "/test.txt", git_buf_cstr(&full_path), &git_blob_create_fromdisk); assert_blob_creation(ELSEWHERE "/test.txt", git_buf_cstr(&full_path), &git_blob_create_from_disk);
git_buf_dispose(&full_path); git_buf_dispose(&full_path);
cl_must_pass(git_futils_rmdir_r(ELSEWHERE, NULL, GIT_RMDIR_REMOVE_FILES)); cl_must_pass(git_futils_rmdir_r(ELSEWHERE, NULL, GIT_RMDIR_REMOVE_FILES));
...@@ -62,7 +62,7 @@ void test_object_blob_write__can_create_a_blob_in_a_bare_repo_from_a_absolute_fi ...@@ -62,7 +62,7 @@ void test_object_blob_write__can_create_a_blob_in_a_bare_repo_from_a_absolute_fi
cl_must_pass(git_path_prettify_dir(&full_path, ELSEWHERE, NULL)); cl_must_pass(git_path_prettify_dir(&full_path, ELSEWHERE, NULL));
cl_must_pass(git_buf_puts(&full_path, "test.txt")); cl_must_pass(git_buf_puts(&full_path, "test.txt"));
assert_blob_creation(ELSEWHERE "/test.txt", git_buf_cstr(&full_path), &git_blob_create_fromdisk); assert_blob_creation(ELSEWHERE "/test.txt", git_buf_cstr(&full_path), &git_blob_create_from_disk);
git_buf_dispose(&full_path); git_buf_dispose(&full_path);
cl_must_pass(git_futils_rmdir_r(ELSEWHERE, NULL, GIT_RMDIR_REMOVE_FILES)); cl_must_pass(git_futils_rmdir_r(ELSEWHERE, NULL, GIT_RMDIR_REMOVE_FILES));
......
...@@ -98,7 +98,7 @@ void test_object_tree_read__largefile(void) ...@@ -98,7 +98,7 @@ void test_object_tree_read__largefile(void)
ie.path = BIGFILE; ie.path = BIGFILE;
cl_git_pass(git_repository_index(&index, g_repo)); cl_git_pass(git_repository_index(&index, g_repo));
cl_git_pass(git_index_add_frombuffer(index, &ie, buf, BIGFILE_SIZE)); cl_git_pass(git_index_add_from_buffer(index, &ie, buf, BIGFILE_SIZE));
cl_repo_commit_from_index(&oid, g_repo, NULL, 0, BIGFILE); cl_repo_commit_from_index(&oid, g_repo, NULL, 0, BIGFILE);
cl_git_pass(git_commit_lookup(&commit, g_repo, &oid)); cl_git_pass(git_commit_lookup(&commit, g_repo, &oid));
......
...@@ -51,10 +51,10 @@ void test_odb_backend_mempack__exists_with_existing_objects_succeeds(void) ...@@ -51,10 +51,10 @@ void test_odb_backend_mempack__exists_with_existing_objects_succeeds(void)
cl_assert(git_odb_exists(_odb, &_oid) == 1); cl_assert(git_odb_exists(_odb, &_oid) == 1);
} }
void test_odb_backend_mempack__blob_create_frombuffer_succeeds(void) void test_odb_backend_mempack__blob_create_from_buffer_succeeds(void)
{ {
const char *data = "data"; const char *data = "data";
cl_git_pass(git_blob_create_frombuffer(&_oid, _repo, data, strlen(data) + 1)); cl_git_pass(git_blob_create_from_buffer(&_oid, _repo, data, strlen(data) + 1));
cl_assert(git_odb_exists(_odb, &_oid) == 1); cl_assert(git_odb_exists(_odb, &_oid) == 1);
} }
...@@ -47,7 +47,7 @@ void test_odb_freshen__loose_blob(void) ...@@ -47,7 +47,7 @@ void test_odb_freshen__loose_blob(void)
set_time_wayback(&before, LOOSE_BLOB_FN); set_time_wayback(&before, LOOSE_BLOB_FN);
/* make sure we freshen a blob */ /* make sure we freshen a blob */
cl_git_pass(git_blob_create_frombuffer(&id, repo, LOOSE_STR, CONST_STRLEN(LOOSE_STR))); cl_git_pass(git_blob_create_from_buffer(&id, repo, LOOSE_STR, CONST_STRLEN(LOOSE_STR)));
cl_assert_equal_oid(&expected_id, &id); cl_assert_equal_oid(&expected_id, &id);
cl_must_pass(p_lstat("testrepo.git/objects/" LOOSE_BLOB_FN, &after)); cl_must_pass(p_lstat("testrepo.git/objects/" LOOSE_BLOB_FN, &after));
...@@ -66,13 +66,13 @@ void test_odb_freshen__readonly_object(void) ...@@ -66,13 +66,13 @@ void test_odb_freshen__readonly_object(void)
cl_git_pass(git_oid_fromstr(&expected_id, UNIQUE_BLOB_ID)); cl_git_pass(git_oid_fromstr(&expected_id, UNIQUE_BLOB_ID));
cl_git_pass(git_blob_create_frombuffer(&id, repo, UNIQUE_STR, CONST_STRLEN(UNIQUE_STR))); cl_git_pass(git_blob_create_from_buffer(&id, repo, UNIQUE_STR, CONST_STRLEN(UNIQUE_STR)));
cl_assert_equal_oid(&expected_id, &id); cl_assert_equal_oid(&expected_id, &id);
set_time_wayback(&before, UNIQUE_BLOB_FN); set_time_wayback(&before, UNIQUE_BLOB_FN);
cl_assert((before.st_mode & S_IWUSR) == 0); cl_assert((before.st_mode & S_IWUSR) == 0);
cl_git_pass(git_blob_create_frombuffer(&id, repo, UNIQUE_STR, CONST_STRLEN(UNIQUE_STR))); cl_git_pass(git_blob_create_from_buffer(&id, repo, UNIQUE_STR, CONST_STRLEN(UNIQUE_STR)));
cl_assert_equal_oid(&expected_id, &id); cl_assert_equal_oid(&expected_id, &id);
cl_must_pass(p_lstat("testrepo.git/objects/" UNIQUE_BLOB_FN, &after)); cl_must_pass(p_lstat("testrepo.git/objects/" UNIQUE_BLOB_FN, &after));
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment