-
Notifications
You must be signed in to change notification settings - Fork 5
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
Add the CHERITech demo and fixes necessary for it to work #20
base: main
Are you sure you want to change the base?
Conversation
demos/2024-04-23-cheritech/demo.cc
Outdated
char clientID[clientIDlength] = "cheriot-XXXXXXXXXXXXXX"; | ||
|
||
/** | ||
* Turn an LED on. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this comment be different from the one for led_on
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You seem to be reviewing something that was removed in a later commit.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I assume you copied the files in this commit from somewhere. If so, can you point to the originals if they are in another open repo?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
They are 'copied' from my Morello machine. I'm not sure what this is asking for.
e49e28c
to
419af6b
Compare
For larger buffers, this does not fit in the maximum size of sealed objects.
JavaScript VM is linked but not yet used. 16 KiB free.
No description provided.