toshok / toshok/echojs

Builtin function 'length' properties

Open
#47 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

easy spec
Dominant language
JavaScript
Stars
431
Forks
20
PR merge metrics
No merged PRs in 30d

Description

The builtin functions provided in the environment have 'length' properties spelled out in ecma262. we should do a pass over all methods and use new macro EJS_INSTALL_ATOM_FUNCTION_LEN_FLAGS (check in ejs-array.c or ejs-string.c for usage) everywhere. then likely change the macro's name and get rid of the other macros.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by comparing the uses of EJS_INSTALL_ATOM_FUNCTION_LEN_FLAGS in ejs-array.c and ejs-string.c, then inspect the other builtin method installations. The work is done when builtin functions use the new length-and-flags macro consistently and the older macros have been renamed or removed as appropriate.

Written by the indexing model from the issue text.

Assessment

Tech stack
c, javascript
Domain
compilers
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.