Skip to content

Conversation

@PradeepKiruvale
Copy link

@PradeepKiruvale PradeepKiruvale commented May 23, 2024

I have separated the guest import and export calls into separate files in this PR.
I also did some code formatting

Restructured the code as below

The directory strucutre is as below:

├── cpp_host (Native host code)
├── guest (Guest code)
├── guest_imported_fns.cpp ( host application imported calls)
├── main.cpp (Application)
└── wit (Wit file that is used for generating the code)

Added a native string rust code

0vygvn added 4 commits May 23, 2024 10:32
Signed-off-by: Pradeep Kumar K J <[email protected]>
Signed-off-by: Pradeep Kumar K J <[email protected]>
Signed-off-by: Pradeep Kumar K J <[email protected]>
Signed-off-by: Pradeep Kumar K J <[email protected]>
@PradeepKiruvale PradeepKiruvale force-pushed the refactor-native-string branch from 5acd4c8 to eb016b7 Compare May 27, 2024 06:20
Signed-off-by: Pradeep Kumar K J <[email protected]>
__attribute__((import_name("c"))) void
cabi_post_fooX3AfooX2FstringsX23c(uint8_t *);
extern "C" void fooX3AfooX2FstringsX00a(uint8_t *arg0, size_t arg1) {
extern "C" void fooX3AfooX2FstringsX00a(uint8_t *arg0, size_t arg1) {
Copy link
Owner

Choose a reason for hiding this comment

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

adding trailing whitespace?

clean:
-rm *.o libstrings.so app-strings

run:
Copy link
Owner

Choose a reason for hiding this comment

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

I fear these targets don't make much sense for the guest directory

```

# Call-graph for a function
Copy link
Owner

Choose a reason for hiding this comment

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

Thank you for creating this!

Pradeep Kumar K J added 3 commits June 5, 2024 18:06
Signed-off-by: Pradeep Kumar K J <[email protected]>
Signed-off-by: Pradeep Kumar K J <[email protected]>
Signed-off-by: Pradeep Kumar K J <[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.

2 participants