[tag] add foreach function

Author: chobie

php_git2.c

@@ -558,6 +558,28 @@ static zend_function_entry php_git2_functions[] = {
 	PHP_FE(git_merge_result_fastforward_oid, arginfo_git_merge_result_fastforward_oid)
 	PHP_FE(git_merge_result_free, arginfo_git_merge_result_free)
 
+	/* tag */
+	PHP_FE(git_tag_lookup, arginfo_git_tag_lookup)
+	PHP_FE(git_tag_lookup_prefix, arginfo_git_tag_lookup_prefix)
+	PHP_FE(git_tag_free, arginfo_git_tag_free)
+	PHP_FE(git_tag_id, arginfo_git_tag_id)
+	PHP_FE(git_tag_owner, arginfo_git_tag_owner)
+	PHP_FE(git_tag_target, arginfo_git_tag_target)
+	PHP_FE(git_tag_target_id, arginfo_git_tag_target_id)
+	PHP_FE(git_tag_target_type, arginfo_git_tag_target_type)
+	PHP_FE(git_tag_name, arginfo_git_tag_name)
+	PHP_FE(git_tag_tagger, arginfo_git_tag_tagger)
+	PHP_FE(git_tag_message, arginfo_git_tag_message)
+	PHP_FE(git_tag_create, arginfo_git_tag_create)
+	PHP_FE(git_tag_annotation_create, arginfo_git_tag_annotation_create)
+	PHP_FE(git_tag_create_frombuffer, arginfo_git_tag_create_frombuffer)
+	PHP_FE(git_tag_create_lightweight, arginfo_git_tag_create_lightweight)
+	PHP_FE(git_tag_delete, arginfo_git_tag_delete)
+	PHP_FE(git_tag_list, arginfo_git_tag_list)
+	PHP_FE(git_tag_list_match, arginfo_git_tag_list_match)
+	PHP_FE(git_tag_foreach, arginfo_git_tag_foreach)
+	PHP_FE(git_tag_peel, arginfo_git_tag_peel)
+
 	/* misc */
 	PHP_FE(git_resource_type, arginfo_git_resource_type)
 	PHP_FE_END

tag.c

@@ -2,6 +2,38 @@
 #include "php_git2_priv.h"
 #include "tag.h"
 
+static int php_git2_tag_foreach_cb(const char *name, git_oid *oid, void *payload)
+{
+	php_git2_t *result;
+	zval *param_name, *param_oid, *retval_ptr = NULL;
+	php_git2_cb_t *p = (php_git2_cb_t*)payload;
+	int i = 0;
+	long retval = 0;
+	char buffer[41] = {0};
+	GIT2_TSRMLS_SET(p->tsrm_ls)
+
+	git_oid_fmt(buffer, oid);
+
+	Z_ADDREF_P(p->payload);
+	MAKE_STD_ZVAL(param_name);
+	MAKE_STD_ZVAL(param_oid);
+	ZVAL_STRING(param_name, name, 1);
+	ZVAL_STRING(param_oid, buffer, 1);
+
+	if (php_git2_call_function_v(p->fci, p->fcc TSRMLS_CC, &retval_ptr, 3, &param_name, &param_oid, &p->payload)) {
+		zval_ptr_dtor(&retval_ptr);
+		zend_list_delete(result->resource_id);
+		return GIT_EUSER;
+	}
+
+	retval = Z_LVAL_P(retval_ptr);
+	zval_ptr_dtor(&retval_ptr);
+	zend_list_delete(result->resource_id);
+
+	return retval;
+}
+
+
 /* {{{ proto resource git_tag_lookup(resource $repo, string $id)
  */
 PHP_FUNCTION(git_tag_lookup)
@@ -482,26 +514,33 @@ PHP_FUNCTION(git_tag_list_match)
 /* }}} */
 
 
-/* {{{ proto long git_tag_foreach(repo, callback, payload)
-*/
+/* {{{ proto long git_tag_foreach(resource $repo,  $callback,  $payload)
+ */
 PHP_FUNCTION(git_tag_foreach)
 {
-	zval *repo;
-	php_git2_t *_repo;
-	zval *callback;
-	php_git2_t *_callback;
-	zval *payload;
-
-	/* TODO(chobie): implement this */
-	php_error_docref(NULL TSRMLS_CC, E_WARNING, "git_tag_foreach not implemented yet");
-	return;
+	int result = 0, error = 0;
+	zval *repo = NULL, *callback = NULL;
+	php_git2_t *_repo = NULL;
+	zend_fcall_info fci = empty_fcall_info;
+	zend_fcall_info_cache fcc = empty_fcall_info_cache;
+	php_git2_cb_t *cb;
+	zval *payload = NULL;
 
 	if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC,
-		"rrz", &repo, &callback, &payload) == FAILURE) {
+		"rfz", &repo, &fci, &fcc, &payload) == FAILURE) {
 		return;
 	}
+
 	ZEND_FETCH_RESOURCE(_repo, php_git2_t*, &repo, -1, PHP_GIT2_RESOURCE_NAME, git2_resource_handle);
+	if (php_git2_cb_init(&cb, &fci, &fcc, payload TSRMLS_CC)) {
+		RETURN_FALSE;
+	}
+	result = git_tag_foreach(PHP_GIT2_V(_repo, repository), php_git2_tag_foreach_cb, cb);
+	php_git2_cb_free(cb);
+	RETURN_LONG(result);
 }
+/* }}} */
+
 
 /* {{{ proto resource git_tag_peel(resource $tag)
  */