Apply bluetech suggestions on PR

src/_pytest/python.py

@@ -1457,7 +1457,9 @@ def __init__(
         #: parametrization adds a ``"[...]"`` suffix to function names).
         #:
         #: .. versionadded:: 3.0
-        self.originalname = originalname if originalname is not None else name
+        self.originalname = originalname or name
+        # note: when FunctionDefinition is introduced, we should change this to a readonly
+        # property that returns FunctionDefinition.name
 
         self.keywords.update(self.obj.__dict__)
         self.own_markers.extend(get_unpacked_marks(self.obj))

testing/python/collect.py

@@ -680,7 +680,7 @@ def test_no_param():
         ]
 
     def test_function_with_square_brackets(self, testdir: Testdir) -> None:
-        """Check that Function._getobj uses originalname."""
+        """Check that functions with square brackets don't cause trouble."""
         p1 = testdir.makepyfile(
             """
             locals()["test_foo[name]"] = lambda: None