Update docstrings for aget/aset to be consistent with Clojure

Description

The docstrings can be updated to match Clojure in these respects:

  1. Reflect variadic nature.

  2. Mention that it works on JavaScript arrays

  3. For aset, indicate the return value.

Additionally, the parameter i can be renamed idx to match Clojure.

Environment

None

Activity

Show:
Mike Fikes
July 7, 2017, 4:56 PM

Attached a patch which updates the docstrings and also the param names. Since param names were changed, added unit tests (including covering the runtime and macro forms).

Completed

Assignee

David Nolen

Reporter

Mike Fikes

Labels

Approval

Accepted

Patch

Code and Test