Don't bother using slice when creating handlers

2023-10-16 16:29:36 +01:00
parent d937934737
commit 46e7dda6a6
1 changed files with 5 additions and 7 deletions
--- a/extensions/ql-vscode/src/view/model-editor/MethodRow.tsx
+++ b/extensions/ql-vscode/src/view/model-editor/MethodRow.tsx
@@ -128,13 +128,11 @@ const ModelableMethodRow = forwardRef<HTMLElement | undefined, MethodRowProps>(

    const removeModelClickedHandlers = useMemo(
      () =>
-        modeledMethods
-          .slice(0, modeledMethods.length - 1)
-          .map((_, index) => () => {
-            const newModeledMethods = [...modeledMethods];
-            newModeledMethods.splice(index, 1);
-            onChange(method.signature, newModeledMethods);
-          }),
+        modeledMethods.map((_, index) => () => {
+          const newModeledMethods = [...modeledMethods];
+          newModeledMethods.splice(index, 1);
+          onChange(method.signature, newModeledMethods);
+        }),
      [method, modeledMethods, onChange],
    );