remade the method to be VARARGS insted of FASTCALL

Author: XFajk

.clangd

@@ -0,0 +1,2 @@
+CompileFlags:
+  CompilationDatabase: ./.mesonpy-build/

.my_pygame_test/main.py

@@ -0,0 +1,27 @@
+import pygame
+from pygame.geometry import Line
+
+pygame.init()
+
+window = pygame.display.set_mode((600, 600))
+
+l = Line(10, 10, 200, 200)
+p = (1, 1)
+
+running = True
+while running:
+
+    window.fill((255, 255, 255))
+
+    pygame.draw.line(window, (255, 0, 0), l.a, l.b, 5)
+    
+    pygame.draw.circle(window, (0, 255, 0), p, 5)
+    pygame.draw.circle(window, (0, 255, 0), l.project(p, do_clamp=True), 5)
+
+    pygame.display.flip()
+
+    for event in pygame.event.get():
+        if event.type == pygame.QUIT:
+            running = False
+
+pygame.quit()

src_c/line.c

@@ -1,5 +1,7 @@
 #include "doc/geometry_doc.h"
 #include "geometry_common.h"
+#include "methodobject.h"
+#include "pytypedefs.h"
 
 static double
 pgLine_Length(pgLineBase *line)
@@ -179,27 +181,6 @@ _line_scale_helper(pgLineBase *line, double factor, double origin)
     return 1;
 }
 
-void
-_normalize_vector(double *vector)
-{
-    double length = sqrt(vector[0] * vector[0] + vector[1] * vector[1]);
-    // check to see if the vector is zero
-    if (length == 0) {
-        vector[0] = 0;
-        vector[1] = 0;
-    }
-    else {
-        vector[0] /= length;
-        vector[1] /= length;
-    }
-}
-
-double
-_length_of_vector(double *vector)
-{
-    return sqrt(vector[0] * vector[0] + vector[1] * vector[1]);
-}
-
 static PyObject *
 _line_project_helper(pgLineBase *line, double *point, int clamp)
 {
@@ -281,21 +262,25 @@ pg_line_scale_ip(pgLineObject *self, PyObject *const *args, Py_ssize_t nargs)
 }
 
 static PyObject *
-pg_line_project(pgLineObject *self, PyObject *const *args, Py_ssize_t nargs,
-                PyObject *kwnames)
+pg_line_project(pgLineObject *self, PyObject *args, PyObject *kwnames)
 {
     double point[2] = {0.f, 0.f};
     int clamp = 0;
 
-    if (nargs >= 1) {
-        if (!pg_TwoDoublesFromObj(args[0], &point[0], &point[1])) {
-            return RAISE(PyExc_TypeError,
-                         "project requires a sequence of two numbers");
-        }
+    PyObject *point_obj = NULL;
+
+    static char *kwlist[] = {"point", "clamp", NULL};
+
+    if (!PyArg_ParseTupleAndKeywords(args, kwnames, "O|p:project", kwlist,
+                                     &point_obj, &clamp)) {
+        return RAISE(
+            PyExc_TypeError,
+            "project requires a sequence(point) and an optional clamp flag");
     }
 
-    if (kwnames != NULL) {
-        clamp = PyObject_IsTrue(args[nargs]);
+    if (!pg_TwoDoublesFromObj(point_obj, &point[0], &point[1])) {
+        return RAISE(PyExc_TypeError,
+                     "project requires a sequence of two numbers");
     }
 
     PyObject *projected_point;
@@ -319,7 +304,7 @@ static struct PyMethodDef pg_line_methods[] = {
     {"scale", (PyCFunction)pg_line_scale, METH_FASTCALL, DOC_LINE_SCALE},
     {"scale_ip", (PyCFunction)pg_line_scale_ip, METH_FASTCALL,
      DOC_LINE_SCALEIP},
-    {"project", (PyCFunction)pg_line_project, METH_FASTCALL | METH_KEYWORDS,
+    {"project", (PyCFunction)pg_line_project, METH_VARARGS | METH_KEYWORDS,
      DOC_LINE_PROJECT},
     {NULL, NULL, 0, NULL}};