Skip to content

Rework the function and local names section. #596

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

Rework the function and local names section. #596

wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented Mar 9, 2016

A proposal to address:

  1. The naming of the imports.
  2. Deal with duplicate names by design, by defining them to be prefixes. The runtime would be expected to add a unique suffix to resolve duplicates, and this might be taken advantage of by the producer to reduce the compressed size.
  3. Allow zero length names to support the use case of the file size increasing more gracefully as names are added. This property depends on blocks of zero compressing well.
  4. Move the name length first to help address the use case of having a directory to lookup the name strings.
  5. Split function names and local names into two section. Seems to be a better fit to an 'extension' model. E.g. A runtime could ignore the local but still name functions. If other names are added in future they can go in additional sections without breaking the existing name sections.

@sunfishcode sunfishcode added this to the Discussion milestone Jul 12, 2016
@jfbastien jfbastien closed this May 12, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants