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

feat(ohos): add jsi module #120

Merged
merged 15 commits into from
Dec 11, 2024
Merged

feat(ohos): add jsi module #120

merged 15 commits into from
Dec 11, 2024

Conversation

zealotchen0
Copy link
Collaborator

Before submitting a new pull request, please make sure:

  • Test cases have been added/updated/passed for the code you will submit.
  • Documentation has added or updated.
  • Commit message is following the Convention Commit guideline with maximum 72 characters.
  • Squash the repeat code commits, short patches are welcome.

static JSVM_Value GetPropertyCbInfo(JSVM_Env env, JSVM_Value name, JSVM_Value thisArg, JSVM_Value data) {
char strValue[100];
size_t size;
OH_JSVM_GetValueStringUtf8(env, name, strValue, 300, &size);
Copy link
Owner

Choose a reason for hiding this comment

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

临时代码?


JSVM_Value res = nullptr;
OH_JSVM_CreateBigintWords(env_, 1, 2, reinterpret_cast<const uint64_t *>(constructor_wrapper.get()), &res);
JSVM_PropertyHandlerConfigurationStruct* propertyHandlerCfg = new JSVM_PropertyHandlerConfigurationStruct();
Copy link
Owner

Choose a reason for hiding this comment

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

没delete

@sohotz sohotz merged commit 2915362 into sohotz:main Dec 11, 2024
17 of 22 checks passed
sohotz pushed a commit that referenced this pull request Jan 15, 2025
* feat(ohos): add jsi init

* add turboMoudule with cpp version

* feat(ohos): add turbo module

* feat(vue): update vue bundle

* feat(ohos): add jsi for jsvm

* feat(ohos): add jsi for jsvm

* feat(ohos): fix crash

* feat(ohos): jsi clean code

* feat(ohos): jsi clean code

---------

Co-authored-by: PC-GRRPM211\Administrator <[email protected]>
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