Skip to content

Commit e70c785

Browse files
committed
Use py_str_to_c_str instead of "es"
1 parent a50aabc commit e70c785

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

src/pygit2.c

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -188,17 +188,18 @@ PyDoc_STRVAR(reference_is_valid_name__doc__,
188188
"\n"
189189
"Check if the passed string is a valid reference name.");
190190
PyObject *
191-
reference_is_valid_name(PyObject *self, PyObject *args)
191+
reference_is_valid_name(PyObject *self, PyObject *py_refname)
192192
{
193193
char* refname;
194194
int result;
195195

196-
if (!PyArg_ParseTuple(args, "es", "utf-8", &refname)) {
196+
refname = py_str_to_c_str(py_refname, NULL);
197+
if (refname == NULL) {
197198
return NULL;
198199
}
199200

200201
result = git_reference_is_valid_name(refname);
201-
PyMem_Free(refname);
202+
free(refname);
202203
return PyBool_FromLong(result);
203204
}
204205

@@ -210,7 +211,7 @@ PyMethodDef module_methods[] = {
210211
discover_repository__doc__},
211212
{"hashfile", hashfile, METH_VARARGS, hashfile__doc__},
212213
{"hash", hash, METH_VARARGS, hash__doc__},
213-
{"reference_is_valid_name", reference_is_valid_name, METH_VARARGS, reference_is_valid_name__doc__},
214+
{"reference_is_valid_name", reference_is_valid_name, METH_O, reference_is_valid_name__doc__},
214215
{"option", option, METH_VARARGS, option__doc__},
215216
{NULL}
216217
};

0 commit comments

Comments
 (0)