InputSystem icon indicating copy to clipboard operation
InputSystem copied to clipboard

FIX: remove the misleading params parameter

Open LovorDev opened this issue 1 year ago • 1 comments

Description

The params parameter can be misleading and the method does not make sense. Additionally, checking for a null value is not necessary, as with params, it is always an array.empty. If you pass null, an error will occur one way or another.

Changes made

Remove params parameter and useless null checks

Checklist

Before review:

  • [ ] Changelog entry added.
    • Explains the change in Changed, Fixed, Added sections.
    • For API change contains an example snippet and/or migration example.
    • FogBugz ticket attached, example ([case %number%](https://issuetracker.unity3d.com/issues/...)).
    • FogBugz is marked as "Resolved" with next release version correctly set.
  • [ ] Tests added/changed, if applicable.
    • Functional tests Area_CanDoX, Area_CanDoX_EvenIfYIsTheCase, Area_WhenIDoX_AndYHappens_ThisIsTheResult.
    • Performance tests.
    • Integration tests.
  • [ ] Docs for new/changed API's.
    • Xmldoc cross references are set correctly.
    • Added explanation how the API works.
    • Usage code examples added.
    • The manual is updated, if needed.

During merge:

  • [ ] Commit message for squash-merge is prefixed with one of the list:
    • NEW: ___.
    • FIX: ___.
    • DOCS: ___.
    • CHANGE: ___.
    • RELEASE: 1.1.0-preview.3.

LovorDev avatar Feb 11 '24 21:02 LovorDev

CLA assistant check
All committers have signed the CLA.

unity-cla-assistant avatar Feb 11 '24 21:02 unity-cla-assistant