Skip to content

Further component bugfixes and improvements - #481

Merged
ksh8281 merged 1 commit into
Samsung:mainfrom
kulcsaradam:additional_component_fixes
Aug 27, 2026
Merged

Further component bugfixes and improvements#481
ksh8281 merged 1 commit into
Samsung:mainfrom
kulcsaradam:additional_component_fixes

Conversation

@kulcsaradam

Copy link
Copy Markdown
Contributor

Remove file type from WasiRefCounted and fix bugs.

@kulcsaradam
kulcsaradam force-pushed the additional_component_fixes branch 2 times, most recently from dca3823 to eaf4664 Compare August 24, 2026 08:21
Remove file type from WasiRefCounted and fix bugs.

Signed-off-by: Ádám László Kulcsár <adam.kulcsar@szteszoftver.hu>
@kulcsaradam
kulcsaradam force-pushed the additional_component_fixes branch from eaf4664 to e18f5ff Compare August 25, 2026 07:35

@zherczeg zherczeg left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Nice patch!

Comment thread src/wasi/WASI02Impl.cpp
options->memory()->buffer()[offset] = resultError;
}

uint32_t offset = argv[4].asI32();

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Is this ok to remove handling of argv[5] and argv[6]?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

stat-at doesnt actaully have those arguments, they were left in by mistake. It only has a self pointer, pathflags, a path and a path length.

@kulcsaradam
kulcsaradam requested a review from zherczeg August 27, 2026 09:36

@zherczeg zherczeg left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@ksh8281
ksh8281 merged commit 14fca82 into Samsung:main Aug 27, 2026
89 checks passed
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