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

added Number.toFixed function #621

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

Conversation

JackTam1993
Copy link

I added an empty toFixed function in packages/core/src/number.ts and test case in devs/run-tests/02numbers.ts. when I run the test case withyarn build and yarn test, the console will return an exception

WASM> Exception: TypeError
WASM> message: setting field 'toFixed' of a builtin frozen object
WASM> at prototype [as F1_pc16] (node_modules/@devicescript/core/src/number.ts:1:1)
WASM> at main [as F0_pc3] (devs/run-tests/02numbers.ts:1:1)

I think some settings occured this error?

@JackTam1993 JackTam1993 mentioned this pull request Sep 15, 2023
@JackTam1993
Copy link
Author

@microsoft-github-policy-service agree

@mmoskal
Copy link
Member

mmoskal commented Nov 1, 2023

We only allow adding properties to objects listed in builtin_proto_idx. Try this:

diff --git a/runtime/devicescript/objects.c b/runtime/devicescript/objects.c
index b75762b7..1e80756c 100644
--- a/runtime/devicescript/objects.c
+++ b/runtime/devicescript/objects.c
@@ -261,8 +261,9 @@ static const uint8_t builtin_proto_idx[] = {
     [DEVS_BUILTIN_OBJECT_BUFFER] = 10,
     [DEVS_BUILTIN_OBJECT_GPIO_PROTOTYPE] = 11,
     [DEVS_BUILTIN_OBJECT_GPIO] = 12,
+    [DEVS_BUILTIN_OBJECT_NUMBER_PROTOTYPE] = 13,
 };
-#define MAX_PROTO 12
+#define MAX_PROTO 13
 
 devs_maplike_t *devs_get_builtin_object(devs_ctx_t *ctx, unsigned idx) {
     if (idx < sizeof(builtin_proto_idx)) {

@JackTam1993 JackTam1993 closed this Nov 7, 2023
@JackTam1993 JackTam1993 deleted the toFixed branch November 7, 2023 09:09
@mmoskal
Copy link
Member

mmoskal commented Nov 7, 2023

It didn't work?

@JackTam1993
Copy link
Author

It didn't work?

No, it didn't.

@JackTam1993 JackTam1993 restored the toFixed branch November 8, 2023 08:27
@JackTam1993 JackTam1993 reopened this Nov 8, 2023
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.

None yet

2 participants