Skip to content
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

Update API portability paragraph #152

Merged
merged 4 commits into from
Jun 12, 2015
Merged

Conversation

lukewagner
Copy link
Member

Capturing discussion from #146, though discussion can continue here. I made some further tweaks to clarify.

WebAssembly does not specify any APIs or syscalls, only an
[import mechanism](MVP.md#modules) where the set of available imports is defined
by the host environment. In a [Web](Web.md) environment, functionality is
accessed through the Web APIs defined by the [Web](https://whatwg.org)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Confusing to have two "Web" links lead to different places.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done, changed link to wikipedia "Open Web Platform" instead which gives a more comprehensive list anyhow.

@jfbastien
Copy link
Member

IIUC one of @sunfishcode's objections was that this paragraph speaks to a different audience. Developers and implementors understand "portability" differently.

To address this, maybe this paragraph could have its own sub-header? And the rest of Portability.md above also have a sub-header?

@jfbastien jfbastien added this to the Public Announcement milestone Jun 12, 2015
@lukewagner
Copy link
Member Author

That's a good suggestion @jfbastien. @sunfishcode any better?

@@ -4,6 +4,8 @@ WebAssembly's [binary format](BinaryEncoding.md) is designed to be executable
efficiently on a variety of operating systems and instruction set architectures,
[on the Web](Web.md) and [off the Web](NonWeb.md).

## Execution
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How about "What platforms can WebAssembly run on?"

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How about "Assumptions for efficient execution"?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works for me.

@sunfishcode
Copy link
Member

Yes, this is better. I think we can do more, but this is good for now.

@jfbastien
Copy link
Member

lgtm

1 similar comment
@sunfishcode
Copy link
Member

lgtm

@lukewagner
Copy link
Member Author

Thanks, all.

lukewagner added a commit that referenced this pull request Jun 12, 2015
@lukewagner lukewagner merged commit bba8bdb into master Jun 12, 2015
@lukewagner lukewagner deleted the update-portable-api-para branch June 12, 2015 16:15
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.

3 participants