Skip to content
This repository has been archived by the owner on Aug 5, 2022. It is now read-only.

Modify docs2 #1898

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Modify docs2 #1898

wants to merge 3 commits into from

Conversation

t-harvey
Copy link

In my role of fixing the WebIDL in the documentation, I've made what should be the last of the changes to WebIDL inconsistencies:

aio.md and ble.md: each had their own version of ReadCallback, so they couldn't be compiled together; we changed the types to include the package name to uniquely identify each one

buffer.md: Javascript objects have a method called toString, so changed the operation name to to_string; changed "Uint8" to the WebIDL type "octet"; set a default value for the to_string operation and specified it as "optional" to match the corresponding zephyr code

gpio.md and sensors.md: like aio/ble, these each declared their own version of "ChangeCallback", so we prepended each instance with its package name

pme.md: set JSON to all caps, because that's the way that WebIDL defines it

Timothy Harvey added 2 commits July 24, 2018 14:23
somehow missed on the first go around.

Signed-off-by: Timothy Harvey <[email protected]>
aio.md and ble.md: each had their own version of ReadCallback, so they couldn't be compiled together; we changed the types to include the package name to uniquely identify each one
buffer.md: Javascript objects have a method called toString, so changed the operation name to to_string; changed "Uint8" to the WebIDL type "octet"; set a default value for the to_string operation and specified it as "optional" to match the corresponding zephry code
gpio.md and sensors.md: like aio/ble, these each declared their own version of "ChangeCallback", so we prepended each instance with its package name
pme.md: set JSON to all caps, because that's the way that WebIDL defines it

Signed-off-by: Timothy Harvey <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant