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

obtaining a promise and related terminology #65

Closed
marcoscaceres opened this issue Jun 5, 2013 · 6 comments
Closed

obtaining a promise and related terminology #65

marcoscaceres opened this issue Jun 5, 2013 · 6 comments
Assignees

Comments

@marcoscaceres
Copy link
Contributor

The spec says:

"Obtains a Future interface representing a request for access to MIDI devices on the user's system."

Please remove this. It's already in step 1 of the "algorithm to request MIDI Access".

Also, the spec says:

Create a Future object that will run the following steps

It's not the "Future" that runs the steps. It's the user agent.

Please change the text to:

  1. Let future be a new Future object and resolver its associated resolver.
  2. Return future.
  3. ...
@marcoscaceres
Copy link
Contributor Author

BTW, please see the following link for how to write an algorithm that users futures:
http://web-alarms.sysapps.org/#taskscheduler-getpendingtasks

We had Anne and Alex check the above and they said it was written correctly. It would be good to model any Promise related text after the above (I think Future's resolver handles a bunch of stuff that doesn't need to be in the MIDI spec).

@ghost ghost assigned cwilso Jun 7, 2013
@cwilso
Copy link
Contributor

cwilso commented Jun 7, 2013

Thanks for the example.

@marcoscaceres
Copy link
Contributor Author

Please note that .done() got nuked also. It's used in an example. Please change it for .then() to be safe.

@cwilso
Copy link
Contributor

cwilso commented Jun 7, 2013

I know. We had a long internal back and forth on all the changes from the
last TC39 meeting. :)

On Fri, Jun 7, 2013 at 10:36 AM, Marcos Caceres notifications@github.comwrote:

Please note that .done() got nuked also. It's used in an example. Please
change it for .then() to be safe.


Reply to this email directly or view it on GitHubhttps://github.com//issues/65#issuecomment-19121549
.

@marcoscaceres
Copy link
Contributor Author

heh, good to know someone knows what the hell is going on with all that stuff :)

@cwilso
Copy link
Contributor

cwilso commented Jun 7, 2013

I did not make THAT claim. :) Much like Thomas Edison, I just found one
more way that was wrong. :)

On Fri, Jun 7, 2013 at 10:52 AM, Marcos Caceres notifications@github.comwrote:

heh, good to know someone knows what the hell is going on with all that
stuff :)


Reply to this email directly or view it on GitHubhttps://github.com//issues/65#issuecomment-19122417
.

@cwilso cwilso closed this as completed in fb3f9b6 Jun 7, 2013
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

No branches or pull requests

2 participants